This is usually what you want; I didn't even add a flag to stop it.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
goto child_errno_fail;
close(errfromchild[1]);
}
goto child_errno_fail;
close(errfromchild[1]);
}
+
+ /* Make (fairly!) sure all other fds are closed. */
+ int max = sysconf(_SC_OPEN_MAX);
+ for (int i = 3; i < max; i++)
+ if (i != execfail[1])
+ close(i);
+
execvp(arr[0], arr);
child_errno_fail:
execvp(arr[0], arr);
child_errno_fail:
* If @errfd == @outfd (and non-NULL) they will be shared.
* If @infd, @outfd or @errfd is &pipecmd_preserve, it is unchanged.
*
* If @errfd == @outfd (and non-NULL) they will be shared.
* If @infd, @outfd or @errfd is &pipecmd_preserve, it is unchanged.
*
- * The return value is the pid of the child, or -1.
+ * The return value is the pid of the child, or -1. All other file-descriptors
+ * are closed in the child.
*/
pid_t pipecmd(int *infd, int *outfd, int *errfd, const char *cmd, ...);
*/
pid_t pipecmd(int *infd, int *outfd, int *errfd, const char *cmd, ...);