diff options
author | dimitri staessens <dimitri.staessens@intec.ugent.be> | 2016-08-09 18:39:03 +0200 |
---|---|---|
committer | dimitri staessens <dimitri.staessens@intec.ugent.be> | 2016-08-09 18:40:54 +0200 |
commit | b1d2e01f1c7c49bd2e1f926f4adb23b21ba36a4a (patch) | |
tree | 58ff91a7f218cd4d6d9795cf37c822620dd8b504 /src/lib/sockets.c | |
parent | e302f7c490e8ffc9a5fe74069115340230a80266 (diff) | |
download | ouroboros-b1d2e01f1c7c49bd2e1f926f4adb23b21ba36a4a.tar.gz ouroboros-b1d2e01f1c7c49bd2e1f926f4adb23b21ba36a4a.zip |
lib: Fix cleanup for sending messages
Cleanup handlers were not always popped.
Diffstat (limited to 'src/lib/sockets.c')
-rw-r--r-- | src/lib/sockets.c | 21 |
1 files changed, 4 insertions, 17 deletions
diff --git a/src/lib/sockets.c b/src/lib/sockets.c index 5d861cdf..0d93c0a9 100644 --- a/src/lib/sockets.c +++ b/src/lib/sockets.c @@ -134,25 +134,12 @@ irm_msg_t * send_recv_irm_msg(irm_msg_t * msg) irm_msg__pack(msg, buf.data); - if (write(sockfd, buf.data, buf.len) == -1) { - free(buf.data); - close(sockfd); - return NULL; - } + if (write(sockfd, buf.data, buf.len) != -1) + count = read(sockfd, buf.data, IRM_MSG_BUF_SIZE); - count = read(sockfd, buf.data, IRM_MSG_BUF_SIZE); - if (count <= 0) { - free(buf.data); - close(sockfd); - return NULL; - } - recv_msg = irm_msg__unpack(NULL, count, buf.data); - if (recv_msg == NULL) { - free(buf.data); - close(sockfd); - return NULL; - } + if (count > 0) + recv_msg = irm_msg__unpack(NULL, count, buf.data); pthread_cleanup_pop(true); pthread_cleanup_pop(true); |