Changeset b9c7424

Timestamp:

Mar 25, 2011, 3:55:41 AM (13 years ago)

Author:

David Benjamin <davidben@mit.edu>

Children:

8eb6068

Parents:

b929e35

git-author:

David Benjamin <davidben@mit.edu> (02/26/11 20:35:26)

git-committer:

David Benjamin <davidben@mit.edu> (03/25/11 03:55:41)

Message:

Just ignore SIGPIPE altogether

There's not much point in switching back and forth from ignoring it and
mostly ignoring it. Also, we don't handle SIGCHLD so remove the code for
it.

(We actually also disable it in the Jabber module because XML::Stream
sets $SIG{PIPE}. Meh.)

Files:

• filterproc.c (modified) (3 diffs)
• owl.c (modified) (2 diffs)

filterproc.c

@@ -17 +17 @@
   int err = 0;
   struct pollfd fds[2];
-  struct sigaction sig = {.sa_handler = SIG_IGN}, old;
 
   fcntl(rfd, F_SETFL, O_NONBLOCK | fcntl(rfd, F_GETFL));
@@ -27 +26 @@
   fds[1].events = POLLOUT;
 
-  sigaction(SIGPIPE, &sig, &old);
-  
   while(1) {
     if(out && *out) {
@@ -67 +64 @@
 
   *in = g_string_free(str, err < 0);
-  sigaction(SIGPIPE, &old, NULL);
   return err;
 }

owl.c

@@ -358 +358 @@
 
   owl_function_debugmsg("Got signal %d", sig);
-  /* TODO: These don't need to be re-entrant anymore! */
   if (sig == SIGWINCH) {
-    /* we can't inturrupt a malloc here, so it just sets a flag
-     * schedulding a resize for later
-     */
     owl_function_resize();
-  } else if (sig == SIGPIPE || sig == SIGCHLD) {
-    owl_function_error("Got unexpected signal: %d %s",
-                       sig, (sig == SIGPIPE) ? "SIGPIPE" : "SIGCHLD");
   } else if (sig == SIGTERM || sig == SIGHUP) {
     owl_function_quit();
@@ -397 +390 @@
 
 void owl_register_signal_handlers(void) {
+  struct sigaction ignore = { .sa_handler = SIG_IGN };
   sigset_t sigset;
 
+  /* Turn off SIGPIPE; we check the return value of write. */
+  sigaction(SIGPIPE, &ignore, NULL);
+
+  /* Register some signals with the signal thread. */
   sigemptyset(&sigset);
   sigaddset(&sigset, SIGWINCH);
   sigaddset(&sigset, SIGALRM);
-  sigaddset(&sigset, SIGPIPE);
   sigaddset(&sigset, SIGTERM);
   sigaddset(&sigset, SIGHUP);
   sigaddset(&sigset, SIGINT);
-
   owl_signal_init(&sigset, sig_handler, g_main_context_default());
 }