diff options
author | Pavel Březina <pbrezina@redhat.com> | 2012-11-06 12:24:15 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-11-08 11:47:16 +0100 |
commit | b6d5f2a91fbce15c7ef4d382fa6b52407adb26dd (patch) | |
tree | 8edb0175474400fb31ea0b31a55078e3fa7769df | |
parent | e9086c9178b06ff396b958dcc2a1e5baa0693f5c (diff) | |
download | sssd-b6d5f2a91fbce15c7ef4d382fa6b52407adb26dd.tar.gz sssd-b6d5f2a91fbce15c7ef4d382fa6b52407adb26dd.tar.bz2 sssd-b6d5f2a91fbce15c7ef4d382fa6b52407adb26dd.zip |
sss_dp_get_domains_send(): handle subreq error correctly
If force is true, ret may stay uninitialized and if ret == 0
after the subrequest is send, we will go to immediate label.
Data provider request is sent, but the answer is never processed.
This prohibited subdomain from working correctly.
-rw-r--r-- | src/responder/common/responder_get_domains.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/responder/common/responder_get_domains.c b/src/responder/common/responder_get_domains.c index f4bcbf14..468dd761 100644 --- a/src/responder/common/responder_get_domains.c +++ b/src/responder/common/responder_get_domains.c @@ -203,7 +203,8 @@ struct tevent_req *sss_dp_get_domains_send(TALLOC_CTX *mem_ctx, subreq = get_subdomains_send(req, rctx, state->dom, state->force, state->hint); - if (ret != EAGAIN) { + if (subreq == NULL) { + ret = ENOMEM; goto immediately; } tevent_req_set_callback(subreq, sss_dp_get_domains_process, req); |