process: Don't close all FDs on new spawn

Fixes #40990

Signed-off-by: David Goulet <dgoulet@torproject.org>

diff --git a/changes/ticket40990 b/changes/ticket40990
new file mode 100644 (file)
index 0000000..af61308
--- /dev/null
+++ b/changes/ticket40990
@@ -0,0 +1,4 @@
+  o Minor bugfix (process):
+    - Avoid closing all possible FDs when spawning a process (PT). On some systems, this could
+      lead to 3+ minutes hang. Fixes bug 40990; bugfix on 0.3.5.1-alpha.
+

diff --git a/src/lib/process/process_unix.c b/src/lib/process/process_unix.c
index 15ae03eadf0cc9135dbae49184d5bb80d7f2ae53..932cdf2e8ca8c26a73ef9714458299c8f2b860a4 100644 (file)
--- a/src/lib/process/process_unix.c
+++ b/src/lib/process/process_unix.c
@@ -137,7 +137,7 @@ process_unix_exec(process_t *process)
   int stdin_pipe[2];
   int stdout_pipe[2];
   int stderr_pipe[2];
-  int retval, fd;
+  int retval;
 
   unix_process = process_get_unix_process(process);
 
@@ -240,11 +240,9 @@ process_unix_exec(process_t *process)
     close(stdin_pipe[0]);
     close(stdin_pipe[1]);
 
-    /* Close all other fds, including the read end of the pipe.  XXX: We should
-     * now be doing enough FD_CLOEXEC setting to make this needless.
-     */
-    for (fd = STDERR_FILENO + 1; fd < max_fd; fd++)
-      close(fd);
+    /* Note that we don't close all FDs from here, which we used to do, because
+     * all our open are CLOEXEC. With a very large maximum number of FDs, the
+     * loop was taking a long time: #40990 */
 
     /* Create the argv value for our new process. */
     char **argv = process_get_argv(process);