summaryrefslogtreecommitdiff
path: root/src/tools/sss_userdel.c
diff options
context:
space:
mode:
authorMichal Zidek <mzidek@redhat.com>2013-04-03 17:04:41 +0200
committerJakub Hrozek <jhrozek@redhat.com>2013-04-05 17:22:29 +0200
commite391eb27c711a5d51f7a5ad92928212fdcb3133a (patch)
treefcd26f591b4e83f269971abe2f6a0310696c646d /src/tools/sss_userdel.c
parent4d09e8abf9f4306d3d6add88ee9100373c421c74 (diff)
downloadsssd-e391eb27c711a5d51f7a5ad92928212fdcb3133a.tar.gz
sssd-e391eb27c711a5d51f7a5ad92928212fdcb3133a.tar.bz2
sssd-e391eb27c711a5d51f7a5ad92928212fdcb3133a.zip
Check for waitpid failure at wrong place.
Coverity bugs. https://fedorahosted.org/sssd/ticket/1865
Diffstat (limited to 'src/tools/sss_userdel.c')
-rw-r--r--src/tools/sss_userdel.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/tools/sss_userdel.c b/src/tools/sss_userdel.c
index 2cf77596..ac87a736 100644
--- a/src/tools/sss_userdel.c
+++ b/src/tools/sss_userdel.c
@@ -101,15 +101,15 @@ static int kick_user(struct tools_ctx *tctx)
}
while((child_pid = waitpid(pid, &status, 0)) > 0) {
- if (child_pid == -1) {
- DEBUG(1, ("waitpid failed\n"));
- return errno;
- }
if (WIFEXITED(status)) {
break;
}
}
+ if (child_pid == -1) {
+ DEBUG(SSSDBG_CRIT_FAILURE, ("waitpid failed\n"));
+ return errno;
+ }
}
return EOK;