svn commit: r391580 - in /apr/apr/trunk: CHANGES threadproc/unix/signals.c

[jo...@apache.org]

Author: jorton
Date: Wed Apr  5 03:54:26 2006
New Revision: 391580

URL: http://svn.apache.org/viewcvs?rev=391580&view=rev
Log:
* threadproc/unix/signals.c (apr_signal, avoid_zombies): Use the
Darwin zombie-avoidance hack on NetBSD too.

PR: 36750
Submitted by: Todd Vierling <tv pobox.com>

Modified:
    apr/apr/trunk/CHANGES
    apr/apr/trunk/threadproc/unix/signals.c

Modified: apr/apr/trunk/CHANGES
URL: http://svn.apache.org/viewcvs/apr/apr/trunk/CHANGES?rev=391580&r1=391579&r2=391580&view=diff
==============================================================================
--- apr/apr/trunk/CHANGES (original)
+++ apr/apr/trunk/CHANGES Wed Apr  5 03:54:26 2006
@@ -1,5 +1,8 @@
 Changes for APR 1.3.0
 
+  *) NetBSD: Avoid leaving zombie process when using apr_signal() 
+     to ignore SIGCHLD.  PR 36750.  [Todd Vierling <tv pobox.com>]
+
   *) Correct bug in kqueue backend for apr_pollset where we would
      erroneously indicate that a socket was readable or writeable.
      [Garrett Rooney]

Modified: apr/apr/trunk/threadproc/unix/signals.c
URL: http://svn.apache.org/viewcvs/apr/apr/trunk/threadproc/unix/signals.c?rev=391580&r1=391579&r2=391580&view=diff
==============================================================================
--- apr/apr/trunk/threadproc/unix/signals.c (original)
+++ apr/apr/trunk/threadproc/unix/signals.c Wed Apr  5 03:54:26 2006
@@ -55,7 +55,7 @@
 
 #if APR_HAVE_SIGACTION
 
-#ifdef DARWIN
+#if defined(__NetBSD__) || defined(DARWIN)
 static void avoid_zombies(int signo)
 {
     int exit_status;
@@ -91,7 +91,7 @@
         act.sa_flags |= SA_NOCLDWAIT;
     }
 #endif
-#ifdef DARWIN
+#if defined(__NetBSD__) || defined(DARWIN)
     /* ignoring SIGCHLD or leaving the default disposition doesn't avoid zombies,
      * and there is no SA_NOCLDWAIT flag, so catch the signal and reap status in 
      * the handler to avoid zombies