common-main: call restore_sigpipe_to_default()
[gitweb.git] / common-main.c
index 57c912a78e3f12665ee5afbacb8c5e69a5f4a53c..20e55ef7d7681ddda6a29e26d196e484c8720b7a 100644 (file)
@@ -1,6 +1,27 @@
-#include "git-compat-util.h"
+#include "cache.h"
 #include "exec_cmd.h"
 
+/*
+ * Many parts of Git have subprograms communicate via pipe, expect the
+ * upstream of a pipe to die with SIGPIPE when the downstream of a
+ * pipe does not need to read all that is written.  Some third-party
+ * programs that ignore or block SIGPIPE for their own reason forget
+ * to restore SIGPIPE handling to the default before spawning Git and
+ * break this carefully orchestrated machinery.
+ *
+ * Restore the way SIGPIPE is handled to default, which is what we
+ * expect.
+ */
+static void restore_sigpipe_to_default(void)
+{
+       sigset_t unblock;
+
+       sigemptyset(&unblock);
+       sigaddset(&unblock, SIGPIPE);
+       sigprocmask(SIG_UNBLOCK, &unblock, NULL);
+       signal(SIGPIPE, SIG_DFL);
+}
+
 int main(int argc, char **av)
 {
        /*
@@ -9,7 +30,16 @@ int main(int argc, char **av)
         */
        const char **argv = (const char **)av;
 
+       /*
+        * Always open file descriptors 0/1/2 to avoid clobbering files
+        * in die().  It also avoids messing up when the pipes are dup'ed
+        * onto stdin/stdout/stderr in the child processes we spawn.
+        */
+       sanitize_stdfds();
+
        argv[0] = git_extract_argv0_path(argv[0]);
 
+       restore_sigpipe_to_default();
+
        return cmd_main(argc, argv);
 }