Reenter the fatal_signal_handler in case of nested signal.

diff --git a/ChangeLog b/ChangeLog
index 70743d2545de1beced961cbbb0b9d847fe4747d9..5a9b26ead80b11a695de2fca670b708a2cce3d5d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2008-06-23  Bruno Haible  <bruno@clisp.org>
+
+       * lib/fatal-signal.c (fatal_signal_handler): Update comment.
+       (install_handlers): Don't set the SA_RESETHAND flag.
+
 2008-06-23  Bruno Haible  <bruno@clisp.org>
 
        * m4/sigaction.m4: Comment fixes.

diff --git a/lib/fatal-signal.c b/lib/fatal-signal.c
index eca46dba0c69fcf0d642b676b2caa35337e03890..64b346d8747b4637bcc684bc74a706ad8445e2e6 100644 (file)
--- a/lib/fatal-signal.c
+++ b/lib/fatal-signal.c
@@ -160,9 +160,10 @@ fatal_signal_handler (int sig)
     }
 
   /* Now execute the signal's default action.
-     If any cleanup action blocks the signal that triggered the cleanup, the
-     re-raised signal is delivered when this handler returns; otherwise it
-     is delivered already during raise().  */
+     If the signal being delivered was blocked, the re-raised signal would be
+     delivered when this handler returns.  But the way we install this handler,
+     no signal is blocked, and the re-raised signal is delivered already
+     during raise().  */
   uninstall_handlers ();
   raise (sig);
 }
@@ -176,9 +177,10 @@ install_handlers ()
   struct sigaction action;
 
   action.sa_handler = &fatal_signal_handler;
-  /* One-shot handling - if we fault while handling a fault, the
-     cleanup actions are intentionally cut short.  */
-  action.sa_flags = SA_NODEFER | SA_RESETHAND;
+  /* If we get a fatal signal while executing fatal_signal_handler, enter
+     fatal_signal_handler recursively, since it is reentrant.  Hence no
+     SA_RESETHAND.  */
+  action.sa_flags = SA_NODEFER;
   sigemptyset (&action.sa_mask);
   for (i = 0; i < num_fatal_signals; i++)
     if (fatal_signals[i] >= 0)