summaryrefslogtreecommitdiff
path: root/src/providers
diff options
context:
space:
mode:
authorStef Walter <stefw@gnome.org>2012-04-24 11:32:04 +0200
committerStephen Gallagher <sgallagh@redhat.com>2012-05-01 14:22:50 -0400
commitc8a124ac1b03d83016bda02cad5a369ce6fb1cd7 (patch)
tree9f5ea0df345b3ae8008d1a3cd6c3398dfc67bf13 /src/providers
parent24ba5b87c9a2c01deddc9be867e4951e2a397146 (diff)
downloadsssd-c8a124ac1b03d83016bda02cad5a369ce6fb1cd7.tar.gz
sssd-c8a124ac1b03d83016bda02cad5a369ce6fb1cd7.tar.bz2
sssd-c8a124ac1b03d83016bda02cad5a369ce6fb1cd7.zip
execv, excvp and exec_child never return EOK
* So don't need to handle that case
Diffstat (limited to 'src/providers')
-rw-r--r--src/providers/ipa/ipa_dyndns.c8
-rw-r--r--src/providers/ldap/sdap_child_helpers.c8
2 files changed, 6 insertions, 10 deletions
diff --git a/src/providers/ipa/ipa_dyndns.c b/src/providers/ipa/ipa_dyndns.c
index 71fc27bc..4224919b 100644
--- a/src/providers/ipa/ipa_dyndns.c
+++ b/src/providers/ipa/ipa_dyndns.c
@@ -1015,11 +1015,9 @@ fork_nsupdate_send(struct ipa_nsupdate_ctx *ctx)
}
errno = 0;
- ret = execv(NSUPDATE_PATH, args);
- if(ret == -1) {
- err = errno;
- DEBUG(1, ("execv failed [%d][%s].\n", err, strerror(err)));
- }
+ execv(NSUPDATE_PATH, args);
+ err = errno;
+ DEBUG(SSSDBG_CRIT_FAILURE, ("execv failed [%d][%s].\n", err, strerror(err)));
return NULL;
}
diff --git a/src/providers/ldap/sdap_child_helpers.c b/src/providers/ldap/sdap_child_helpers.c
index 704c89ec..eeb5e5f0 100644
--- a/src/providers/ldap/sdap_child_helpers.c
+++ b/src/providers/ldap/sdap_child_helpers.c
@@ -107,11 +107,9 @@ static errno_t sdap_fork_child(struct tevent_context *ev,
err = exec_child(child,
pipefd_to_child, pipefd_from_child,
LDAP_CHILD, ldap_child_debug_fd);
- if (err != EOK) {
- DEBUG(1, ("Could not exec LDAP child: [%d][%s].\n",
- err, strerror(err)));
- return err;
- }
+ DEBUG(SSSDBG_CRIT_FAILURE, ("Could not exec LDAP child: [%d][%s].\n",
+ err, strerror(err)));
+ return err;
} else if (pid > 0) { /* parent */
child->pid = pid;
child->read_from_child_fd = pipefd_from_child[0];