From 2696ceb4d4796f9de78c80982ecb13fc4b658b3c Mon Sep 17 00:00:00 2001 From: Mukund Sivaraman Date: Tue, 3 Feb 2015 11:43:34 +0530 Subject: [PATCH] Fix a Valgrind warning about use of uninitialized memory (as part of #38454) --- lib/isc/unix/socket.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/isc/unix/socket.c b/lib/isc/unix/socket.c index 99f26d85f0..f869464664 100644 --- a/lib/isc/unix/socket.c +++ b/lib/isc/unix/socket.c @@ -2446,7 +2446,7 @@ set_rcvbuf(void) { return; len = sizeof(min); - if (getsockopt(fd, SOL_SOCKET, SO_RCVBUF, (void *)&min, &len) >= 0 && + if (getsockopt(fd, SOL_SOCKET, SO_RCVBUF, (void *)&min, &len) == 0 && min < rcvbuf) { again: if (setsockopt(fd, SOL_SOCKET, SO_RCVBUF, (void *)&rcvbuf, @@ -2547,7 +2547,7 @@ opensocket(isc__socketmgr_t *manager, isc__socket_t *sock, #endif #if defined(SO_RCVBUF) ISC_SOCKADDR_LEN_T optlen; - int size; + int size = 0; #endif again: @@ -2843,7 +2843,7 @@ opensocket(isc__socketmgr_t *manager, isc__socket_t *sock, #if defined(SO_RCVBUF) optlen = sizeof(size); if (getsockopt(sock->fd, SOL_SOCKET, SO_RCVBUF, - (void *)&size, &optlen) >= 0 && size < rcvbuf) { + (void *)&size, &optlen) == 0 && size < rcvbuf) { RUNTIME_CHECK(isc_once_do(&rcvbuf_once, set_rcvbuf) == ISC_R_SUCCESS); if (setsockopt(sock->fd, SOL_SOCKET, SO_RCVBUF, @@ -5875,7 +5875,7 @@ internal_connect(isc_task_t *me, isc_event_t *ev) { */ optlen = sizeof(cc); if (getsockopt(sock->fd, SOL_SOCKET, SO_ERROR, - (void *)&cc, (void *)&optlen) < 0) + (void *)&cc, (void *)&optlen) != 0) cc = errno; else errno = cc;