From 9a9bfb6443b16316ff124456cdb6d1e2d0281c53 Mon Sep 17 00:00:00 2001 From: hitstergtd Date: Mon, 18 Apr 2016 14:44:54 +0100 Subject: [PATCH] Problem: test_udp does not release resources properly Solution: - call zmq_msg_close() if there is an error - call free() to release resources if there is an error (CWE-404) --- tests/test_udp.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/test_udp.cpp b/tests/test_udp.cpp index 6a48b562..b5b1981f 100644 --- a/tests/test_udp.cpp +++ b/tests/test_udp.cpp @@ -57,8 +57,10 @@ int msg_recv_cmp (zmq_msg_t *msg_, void *s_, const char* group_, const char* bod return -1; int recv_rc = zmq_msg_recv (msg_, s_, 0); - if (recv_rc == -1) + if (recv_rc == -1) { + zmq_msg_close(msg_); return -1; + } if (strcmp (zmq_msg_group (msg_), group_) != 0) { @@ -73,6 +75,7 @@ int msg_recv_cmp (zmq_msg_t *msg_, void *s_, const char* group_, const char* bod if (strcmp (body, body_) != 0) { zmq_msg_close (msg_); + free(body); return -1; }