[SSSD] [PATCH] SERVER: Check the return value of waitpid

Jakub Hrozek jhrozek at redhat.com
Sun Nov 18 23:05:44 UTC 2012


We should at least print an error message and error out if waitpid()
fails.

https://fedorahosted.org/sssd/ticket/1651
-------------- next part --------------
>From 90b2dddf4b902fe3a83e141a8a9330bd16403906 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhrozek at redhat.com>
Date: Sun, 18 Nov 2012 20:54:28 +0100
Subject: [PATCH] SERVER: Check the return value of waitpid

We should at least print an error message and error out if waitpid()
fails.

https://fedorahosted.org/sssd/ticket/1651
---
 src/util/server.c | 35 +++++++++++++++++++++++++----------
 1 file changed, 25 insertions(+), 10 deletions(-)

diff --git a/src/util/server.c b/src/util/server.c
index 3dc9bcc0becbafc440a8d8ccf1a1bf1bbaadc622..964be3d1725663d5196d22c2093ccbcb57eae1c5 100644
--- a/src/util/server.c
+++ b/src/util/server.c
@@ -79,9 +79,9 @@ static void deamon_parent_sigterm(int sig)
 
 void become_daemon(bool Fork)
 {
-    pid_t pid;
+    pid_t pid, cpid;
     int status;
-    int ret;
+    int ret, error;
 
     if (Fork) {
         pid = fork();
@@ -93,15 +93,30 @@ void become_daemon(bool Fork)
             CatchSignal(SIGTERM, deamon_parent_sigterm);
 
             /* or exit when sssd monitor is terminated */
-            waitpid(pid, &status, 0);
+            do {
+                errno = 0;
+                cpid = waitpid(pid, &status, 0);
+                if (cpid == 1) {
+                    /* An error occurred while waiting */
+                    error = errno;
+                    if (error != EINTR) {
+                        DEBUG(SSSDBG_CRIT_FAILURE,
+                              ("Error [%d][%s] while waiting for child\n",
+                               error, strerror(error)));
+                        /* Forcibly kill this child */
+                        kill(pid, SIGKILL);
+                    }
+                } else if (pid != 0) {
+                    error = 0;
+                    /* return error if we didn't exited normally */
+                    ret = 1;
 
-            /* return error if we didn't exited normally */
-            ret = 1;
-
-            if (WIFEXITED(status)) {
-                /* but return our exit code otherwise */
-                ret = WEXITSTATUS(status);
-            }
+                    if (WIFEXITED(status)) {
+                        /* but return our exit code otherwise */
+                        ret = WEXITSTATUS(status);
+                    }
+                }
+            } while (error == EINTR);
 
             _exit(ret);
         }
-- 
1.8.0



More information about the sssd-devel mailing list