Fix the time we set of SO_REUSEADDR for listening sockets.
authorMichael Poole <mdpoole@troilus.org>
Fri, 8 Jan 2010 03:01:26 +0000 (22:01 -0500)
committerMichael Poole <mdpoole@troilus.org>
Fri, 8 Jan 2010 03:01:26 +0000 (22:01 -0500)
src/ioset.c (ioset_listen): Linux -- reasonably -- requires that
    SO_REUSEADDR be set on a socket before binding it to an in-use socket.
    So do that.

src/ioset.c

index 30eb359f816bac41731e7c093146de7326379eec..a7563d30398e90aca346eddc6b21b3e92a66a293 100644 (file)
@@ -220,18 +220,18 @@ struct io_fd *ioset_listen(struct sockaddr *local, unsigned int sa_size, void *d
     }
 
     if (local && sa_size) {
+        opt = 1;
+        res = setsockopt(fd, SOL_SOCKET, SO_REUSEADDR, (const char*)&opt, sizeof(opt));
+        if (res < 0) {
+            log_module(MAIN_LOG, LOG_WARNING, "Unable to mark listener address as re-usable: %s", strerror(errno));
+        }
+
         res = bind(fd, local, sa_size);
         if (res < 0) {
             log_module(MAIN_LOG, LOG_ERROR, "Unable to bind listening socket %d: %s", fd, strerror(errno));
             close(fd);
             return NULL;
         }
-
-        opt = 1;
-        res = setsockopt(fd, SOL_SOCKET, SO_REUSEADDR, (const char*)&opt, sizeof(opt));
-        if (res < 0) {
-            log_module(MAIN_LOG, LOG_WARNING, "Unable to mark listener address as re-usable: %s", strerror(errno));
-        }
     }
 
     res = listen(fd, 1);