Use an empty signal handler for SIGPIPE instead of SIG_IGN
authorSimon Marlow <marlowsd@gmail.com>
Sat, 25 Sep 2010 19:35:48 +0000 (19:35 +0000)
committerSimon Marlow <marlowsd@gmail.com>
Sat, 25 Sep 2010 19:35:48 +0000 (19:35 +0000)
This is so that the SIGPIPE handler gets reset to the default
automatically on exec().

rts/posix/Signals.c

index b045289..e723b8f 100644 (file)
@@ -471,6 +471,15 @@ shutdown_handler(int sig STG_UNUSED)
 }
 
 /* -----------------------------------------------------------------------------
+ * An empty signal handler, currently used for SIGPIPE
+ * -------------------------------------------------------------------------- */
+static void
+empty_handler (int sig STG_UNUSED)
+{
+    // nothing
+}
+
+/* -----------------------------------------------------------------------------
  * Install default signal handlers.
  *
  * The RTS installs a default signal handler for catching
@@ -526,7 +535,9 @@ initDefaultHandlers(void)
 #endif
 
     // ignore SIGPIPE; see #1619
-    action.sa_handler = SIG_IGN;
+    // actually, we use an empty signal handler rather than SIG_IGN,
+    // so that SIGPIPE gets reset to its default behaviour on exec.
+    action.sa_handler = empty_handler;
     sigemptyset(&action.sa_mask);
     action.sa_flags = 0;
     if (sigaction(SIGPIPE, &action, &oact) != 0) {