Also exit waitpid() loop when pid == 0. Fixes a problem where the sudo

process would spin eating up CPU until sendmail finished when it has
to send mail.

diff --git a/logging.c b/logging.c
index b16c507bf37728621fc46604b7947da3718ff866..8b48caad833123c27fdc3174c2753757351add46 100644 (file)
--- a/logging.c
+++ b/logging.c
@@ -592,10 +592,12 @@ reapchild(sig)
     int sig;
 {
     int status, serrno = errno;
-
 #ifdef sudo_waitpid
-    while (sudo_waitpid(-1, &status, WNOHANG) != -1 || errno == EINTR)
-       ;
+    pid_t pid;
+
+    do {
+       pid = sudo_waitpid(-1, &status, WNOHANG);
+    } while (pid != 0 && (pid != -1 || errno == EINTR));
 #else
     (void) wait(&status);
 #endif