[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