Code

Clean up file descriptors when calling hooks.
authorDaniel Barkalow <barkalow@iabervon.org>
Thu, 8 Dec 2005 02:04:38 +0000 (21:04 -0500)
committerJunio C Hamano <junkio@cox.net>
Thu, 8 Dec 2005 05:05:39 +0000 (21:05 -0800)
When calling post-update hook, don't leave stdin and stdout connected to
the pushing connection.

Signed-off-by: Daniel Barkalow <barkalow@iabervon.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
receive-pack.c
run-command.c
run-command.h

index 18735061206a813d523fdf29dd3e5b06899f1a1d..cbe37e72391c5f22fb21a48fe64232c212ed8709 100644 (file)
@@ -173,7 +173,7 @@ static void run_update_post_hook(struct command *cmd)
                argc++;
        }
        argv[argc] = NULL;
-       run_command_v(argc, argv);
+       run_command_v_opt(argc, argv, RUN_COMMAND_NO_STDIO);
 }
 
 /*
index 5787a5095548c6530a4d987705a3189ed57356c4..8bf5922fc3b30d06ca7336a8df18a12c389c8681 100644 (file)
@@ -2,13 +2,19 @@
 #include "run-command.h"
 #include <sys/wait.h>
 
-int run_command_v(int argc, char **argv)
+int run_command_v_opt(int argc, char **argv, int flags)
 {
        pid_t pid = fork();
 
        if (pid < 0)
                return -ERR_RUN_COMMAND_FORK;
        if (!pid) {
+               if (flags & RUN_COMMAND_NO_STDIO) {
+                       int fd = open("/dev/null", O_RDWR);
+                       dup2(fd, 0);
+                       dup2(fd, 1);
+                       close(fd);                      
+               }
                execvp(argv[0], (char *const*) argv);
                die("exec %s failed.", argv[0]);
        }
@@ -36,6 +42,11 @@ int run_command_v(int argc, char **argv)
        }
 }
 
+int run_command_v(int argc, char **argv)
+{
+       return run_command_v_opt(argc, argv, 0);
+}
+
 int run_command(const char *cmd, ...)
 {
        int argc;
@@ -54,5 +65,5 @@ int run_command(const char *cmd, ...)
        va_end(param);
        if (MAX_RUN_COMMAND_ARGS <= argc)
                return error("too many args to run %s", cmd);
-       return run_command_v(argc, argv);
+       return run_command_v_opt(argc, argv, 0);
 }
index 5ee0972241030d2f6d87a465c4726e6cc3ab803a..2469eeaefbc19424d5ec61f173cc17b68a183bc5 100644 (file)
@@ -11,6 +11,9 @@ enum {
        ERR_RUN_COMMAND_WAITPID_NOEXIT,
 };
 
+#define RUN_COMMAND_NO_STDIO 1
+
+int run_command_v_opt(int argc, char **argv, int opt);
 int run_command_v(int argc, char **argv);
 int run_command(const char *cmd, ...);