From 85427a949aefae40a57ee940ad90488f6e0965c5 Mon Sep 17 00:00:00 2001 From: Christopher Faylor Date: Fri, 26 Mar 2004 16:15:23 +0000 Subject: [PATCH] * sigproc.cc (wait_sig): Make sure that SIGCHLD is handled regardless of whether a signal is queued. --- winsup/cygwin/ChangeLog | 5 +++++ winsup/cygwin/sigproc.cc | 17 ++++++++++------- 2 files changed, 15 insertions(+), 7 deletions(-) diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 07c9a1b8d..7dcefe0b9 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,8 @@ +2004-03-26 Christopher Faylor + + * sigproc.cc (wait_sig): Make sure that SIGCHLD is handled regardless + of whether a signal is queued. + 2004-03-26 Christopher Faylor * sigproc.cc (wait_sig): Report if not trying to send signal due to diff --git a/winsup/cygwin/sigproc.cc b/winsup/cygwin/sigproc.cc index 4dc9bdb9e..055029979 100644 --- a/winsup/cygwin/sigproc.cc +++ b/winsup/cygwin/sigproc.cc @@ -1162,19 +1162,22 @@ wait_sig (VOID *self) default: if (pack.si.si_signo < 0) sig_clear (-pack.si.si_signo); - else if (sigq.sigs[pack.si.si_signo].si.si_signo) - sigproc_printf ("sig %d already queued", pack.si.si_signo); else { int sig = pack.si.si_signo; - int sigres = pack.process (); - if (sigres <= 0) + if (sigq.sigs[sig].si.si_signo) + sigproc_printf ("sig %d already queued", pack.si.si_signo); + else { + int sigres = pack.process (); + if (sigres <= 0) + { #ifdef DEBUGGING2 - if (!sigres) - system_printf ("Failed to arm signal %d from pid %d", pack.sig, pack.pid); + if (!sigres) + system_printf ("Failed to arm signal %d from pid %d", pack.sig, pack.pid); #endif - sigq.add (pack); // FIXME: Shouldn't add this in !sh condition + sigq.add (pack); // FIXME: Shouldn't add this in !sh condition + } } if (sig == SIGCHLD) proc_subproc (PROC_CLEARWAIT, 0);