changeset 10221:8c0d36d59e97

Add a comment.
author Bruno Haible <bruno@clisp.org>
date Thu, 19 Jun 2008 03:34:29 +0200
parents 1ccb0de40324
children 8c1fb8a7e62d
files ChangeLog lib/fatal-signal.c
diffstat 2 files changed, 9 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2008-06-18  Bruno Haible  <bruno@clisp.org>
+
+	* lib/fatal-signal.c (init_fatal_signals): Add comment.
+	Reported by Eric Blake.
+
 2008-06-18  Eric Blake  <ebb9@byu.net>
 
 	Work around cygwin 1.5.25 strsignal bug.
--- a/lib/fatal-signal.c
+++ b/lib/fatal-signal.c
@@ -96,6 +96,10 @@
 	  struct sigaction action;
 
 	  if (sigaction (fatal_signals[i], NULL, &action) >= 0
+	      /* POSIX says that SIG_IGN can only occur when action.sa_flags
+		 does not contain SA_SIGINFO.  But in Linux 2.4, for example,
+		 SA_SIGINFO can actually be set and is ignored when sa_handler
+		 is SIG_IGN.  So don't bother testing for SA_SIGINFO.  */
 	      && action.sa_handler == SIG_IGN)
 	    fatal_signals[i] = -1;
 	}