summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPavel Reichl <preichl@redhat.com>2014-06-26 16:42:53 +0100
committerJakub Hrozek <jhrozek@redhat.com>2014-06-27 10:28:08 +0200
commit0d47aef7577f8cf651255cf59df87b3847dbe1ad (patch)
tree281f2c878ee64a3dae30c78ece9e51c37f8c11ac
parent979f969abe7a75a2f41f6fddabec94674ca3c722 (diff)
downloadsssd-0d47aef7577f8cf651255cf59df87b3847dbe1ad.tar.gz
sssd-0d47aef7577f8cf651255cf59df87b3847dbe1ad.tar.xz
sssd-0d47aef7577f8cf651255cf59df87b3847dbe1ad.zip
SDAP: return after tevent_req_error
Don't call tevent_req_done after tevent_req_error (for the same request). Reviewed-by: Sumit Bose <sbose@redhat.com>
-rw-r--r--src/providers/ldap/sdap_async_nested_groups.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap_async_nested_groups.c b/src/providers/ldap/sdap_async_nested_groups.c
index 040b836df..b17946fc3 100644
--- a/src/providers/ldap/sdap_async_nested_groups.c
+++ b/src/providers/ldap/sdap_async_nested_groups.c
@@ -1391,8 +1391,9 @@ static void sdap_nested_group_single_done(struct tevent_req *subreq)
talloc_zfree(subreq);
if (ret != EOK) {
DEBUG(SSSDBG_CRIT_FAILURE, "Error processing nested groups "
- "[%d]: %s", ret, strerror(ret));
+ "[%d]: %s\n.", ret, strerror(ret));
tevent_req_error(req, ret);
+ return;
}
tevent_req_done(req);