Signals whose handler is set to SIG_IGN are not fatal.

diff --git a/lib/ChangeLog b/lib/ChangeLog
index e0497acb3c667a997c8b8e81ee7909111b0ad632..6601ce55d247ba89464faa56c0f8d70a6c2543fb 100644 (file)
--- a/lib/ChangeLog
+++ b/lib/ChangeLog
@@ -1,3 +1,14 @@
+2004-12-18  Bruno Haible  <bruno@clisp.org>
+
+       * fatal-signal.c (fatal_signals): Make non-const.
+       (init_fatal_signals): New function.
+       (uninstall_handlers, install_handlers): Ignore signals that were set to
+       SIG_IGN.
+       (at_fatal_signal): Call init_fatal_signals.
+       (init_fatal_signal_set): Likewise. Ignore signals that were set to
+       SIG_IGN.
+       Reported by Paul Eggert.
+
 2004-12-08  Paul Eggert  <eggert@cs.ucla.edu>
 
        * getdate.y (textint): New member "negative".

diff --git a/lib/fatal-signal.c b/lib/fatal-signal.c
index 8718fd606deb0698bcd93b6ee67baab041a6f1d4..ebba498e991db483147bbb1748a26395489f17f8 100644 (file)
--- a/lib/fatal-signal.c
+++ b/lib/fatal-signal.c
@@ -1,5 +1,5 @@
 /* Emergency actions in case of a fatal signal.
-   Copyright (C) 2003 Free Software Foundation, Inc.
+   Copyright (C) 2003-2004 Free Software Foundation, Inc.
    Written by Bruno Haible <bruno@clisp.org>, 2003.
 
    This program is free software; you can redistribute it and/or modify
@@ -54,7 +54,7 @@
    plus
      SIGXCPU, SIGXFSZ - because they are quite similar to SIGTERM.  */
 
-static const int fatal_signals[] =
+static int fatal_signals[] =
   {
     /* ISO C 99 signals.  */
 #ifdef SIGINT
@@ -82,6 +82,29 @@ static const int fatal_signals[] =
 
 #define num_fatal_signals (SIZEOF (fatal_signals) - 1)
 
+/* Eliminate signals whose signal handler is SIG_IGN.  */
+
+static void
+init_fatal_signals (void)
+{
+  static bool fatal_signals_initialized = false;
+  if (!fatal_signals_initialized)
+    {
+      size_t i;
+
+      for (i = 0; i < num_fatal_signals; i++)
+       {
+         struct sigaction action;
+
+         if (sigaction (fatal_signals[i], NULL, &action) >= 0
+             && action.sa_handler == SIG_IGN)
+           fatal_signals[i] = -1;
+       }
+
+      fatal_signals_initialized = true;
+    }
+}
+
 
 /* ========================================================================= */
 
@@ -111,7 +134,8 @@ uninstall_handlers ()
   size_t i;
 
   for (i = 0; i < num_fatal_signals; i++)
-    signal (fatal_signals[i], SIG_DFL);
+    if (fatal_signals[i] >= 0)
+      signal (fatal_signals[i], SIG_DFL);
 }
 
 
@@ -153,7 +177,8 @@ install_handlers ()
   size_t i;
 
   for (i = 0; i < num_fatal_signals; i++)
-    signal (fatal_signals[i], &fatal_signal_handler);
+    if (fatal_signals[i] >= 0)
+      signal (fatal_signals[i], &fatal_signal_handler);
 }
 
 
@@ -165,6 +190,7 @@ at_fatal_signal (action_t action)
   static bool cleanup_initialized = false;
   if (!cleanup_initialized)
     {
+      init_fatal_signals ();
       install_handlers ();
       cleanup_initialized = true;
     }
@@ -211,9 +237,12 @@ init_fatal_signal_set ()
     {
       size_t i;
 
+      init_fatal_signals ();
+
       sigemptyset (&fatal_signal_set);
       for (i = 0; i < num_fatal_signals; i++)
-       sigaddset (&fatal_signal_set, fatal_signals[i]);
+       if (fatal_signals[i] >= 0)
+         sigaddset (&fatal_signal_set, fatal_signals[i]);
 
       fatal_signal_set_initialized = true;
     }

diff --git a/lib/fatal-signal.h b/lib/fatal-signal.h
index 16e4f72437d490c5ee72d38d763c43178268f9ed..62c4e90e504e911010c5bf1cf27f42403741731a 100644 (file)
--- a/lib/fatal-signal.h
+++ b/lib/fatal-signal.h
@@ -1,5 +1,5 @@
 /* Emergency actions in case of a fatal signal.
-   Copyright (C) 2003 Free Software Foundation, Inc.
+   Copyright (C) 2003-2004 Free Software Foundation, Inc.
    Written by Bruno Haible <bruno@clisp.org>, 2003.
 
    This program is free software; you can redistribute it and/or modify
@@ -26,7 +26,12 @@ extern "C" {
    terminates the process, like removing a temporary file or killing a
    subprocess that may be stuck waiting for a device, pipe or network input.
    Such signals are SIGHUP, SIGINT, SIGPIPE, SIGTERM, and possibly others.
-   The limitation of this facility is that it cannot work for SIGKILL.  */
+   The limitation of this facility is that it cannot work for SIGKILL.
+
+   Signals with a SIG_IGN handler are considered to be non-fatal.  The
+   functions in this file assume that when a SIG_IGN handler is installed
+   for a signal, it was installed before any functions in this file were
+   called and it stays so for the whole lifetime of the process.  */
 
 /* Register a cleanup function to be executed when a catchable fatal signal
    occurs.