From: Ulrich Drepper <drepper@redhat.com>
Date: Sat, 12 Jan 2002 01:36:35 +0000 (+0000)
Subject: (sighandler): Initialize all elements to SIG_ERR. (__sigaction): Don't use value... 
X-Git-Tag: upstream/2.30~22182
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=db298de5c5059585bcbd0d461c337ea92aa62336;p=external%2Fglibc.git

(sighandler): Initialize all elements to SIG_ERR. (__sigaction): Don't use value from sighandler if it is SIG_ERR.
---

diff --git a/linuxthreads/signals.c b/linuxthreads/signals.c
index 8a0dabf..5a1025e 100644
--- a/linuxthreads/signals.c
+++ b/linuxthreads/signals.c
@@ -74,7 +74,7 @@ static union
 {
   arch_sighandler_t old;
   void (*rt) (int, struct siginfo *, struct ucontext *);
-} sighandler[NSIG];
+} sighandler[NSIG] = { [1 ... NSIG - 1] = { (arch_sighandler_t) SIG_ERR } };
 
 /* The wrapper around user-provided signal handlers */
 static void pthread_sighandler(int signo, SIGCONTEXT ctx)
@@ -157,10 +157,14 @@ int __sigaction(int sig, const struct sigaction * act,
     return -1;
   if (sig > 0 && sig < NSIG)
     {
-      if (oact != NULL)
+      if (oact != NULL
+	  /* We may have inherited SIG_IGN from the parent, so return the
+	     kernel's idea of the signal handler the first time
+	     through.  */
+	  && (__sighandler_t) sighandler[sig].old != SIG_ERR)
 	oact->sa_handler = (__sighandler_t) sighandler[sig].old;
       if (act)
-	/* For the assignment is does not matter whether it's a normal
+	/* For the assignment it does not matter whether it's a normal
 	   or real-time signal.  */
 	sighandler[sig].old = (arch_sighandler_t) act->sa_handler;
     }