The handle function has the signature
static void handle(int incoming, struct sockaddr *addr, socklen_t addrlen);
so the addr pointer may be passed as null syntactically.

However there is just one place in daemon.c, which calls the handle
function in function service_loop in daemon.c:1044:
handle(incoming, &, sslen);
so there is always a valid pointer for the sockaddr passed to the handle

The code in question was to its place in f9c87be6, when following the
git blame it originates from 5b276ee4.
That commit 5b276ee4 also seems to copy around the ipv6 code block
and the sockaddr pointer check being null is introduced.
However that commit has already a redundant check on addr being a
null pointer before.

I am including the authors of said patches to the receivers.
 daemon.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/daemon.c b/daemon.c
index 6aeddcb..5e48c1e 100644
--- a/daemon.c
+++ b/daemon.c
@@ -760,7 +760,7 @@ static void handle(int incoming, struct sockaddr *addr, 
socklen_t addrlen)
                snprintf(portbuf, sizeof(portbuf), "REMOTE_PORT=%d",
 #ifndef NO_IPV6
-       } else if (addr && addr->sa_family == AF_INET6) {
+       } else if (addr->sa_family == AF_INET6) {
                struct sockaddr_in6 *sin6_addr = (void *) addr;
                char *buf = addrbuf + 12;

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at

Reply via email to