summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSebastien Braun2010-10-04 00:12:21 +0200
committerSebastien Braun2010-10-05 22:12:26 +0200
commitccf4d3cff3dde8976c9a29fc5cd25c38229c75e3 (patch)
treec9b05dba053c34c436c2fb2bdf32a2562f34b66d
parentAdd Network Configuration icon (diff)
downloadpvs-ccf4d3cff3dde8976c9a29fc5cd25c38229c75e3.tar.gz
pvs-ccf4d3cff3dde8976c9a29fc5cd25c38229c75e3.tar.xz
pvs-ccf4d3cff3dde8976c9a29fc5cd25c38229c75e3.zip
Fix OpenPGM error handling bug resulting in spurious warnings and/or memory leaks.
-rw-r--r--src/net/mcast/McastPGMSocket.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/net/mcast/McastPGMSocket.cpp b/src/net/mcast/McastPGMSocket.cpp
index bf52bd7..fa86d3b 100644
--- a/src/net/mcast/McastPGMSocket.cpp
+++ b/src/net/mcast/McastPGMSocket.cpp
@@ -447,7 +447,7 @@ void McastPGMSocket::handleData()
{
char buf[4096];
size_t size;
- pgm_error_t* err;
+ pgm_error_t* err = 0;
status = pgm_recv(_priv->socket, buf, sizeof(buf), MSG_DONTWAIT, &size, &err);
@@ -496,6 +496,7 @@ void McastPGMSocket::handleData()
qDebug() << " connection reset";
emit connectionReset();
qCritical() << "Connection Reset: PGM Error: " << (err ? err->message : "(null)");
+ pgm_error_free(err);
break;
}
else if (status == PGM_IO_STATUS_FIN)