Use FD_CLOEXEC fcntl for WEB-server socket

free() is not safe in fork()ed child when using threads. Don't close
WEB-server socket manually, use FD_CLOEXEC fcntl instead.

Author: sysoleg

libhttpd/api.c

@@ -262,6 +262,11 @@ int port;
         free(new);
         return (NULL);
     }
+    if (fcntl(sock, F_SETFD, FD_CLOEXEC) < 0) {
+        close(sock);
+        free(new);
+        return (NULL);
+    }
 #	ifdef SO_REUSEADDR
     opt = 1;
     if (setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, (char *)&opt, sizeof(int)) < 0) {

src/gateway.c

@@ -397,7 +397,6 @@ main_loop(void)
         debug(LOG_ERR, "Could not create web server: %s", strerror(errno));
         exit(1);
     }
-    register_fd_cleanup_on_fork(webserver->serverSock);
 
     debug(LOG_DEBUG, "Assigning callbacks to web server");
     httpdAddCContent(webserver, "/", "wifidog", 0, NULL, http_callback_wifidog);