diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2014-11-28 12:32:25 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-12-02 10:46:37 +0100 |
commit | 42bc7cb28858f8affa5bc7586f8d39b3afe4c387 (patch) | |
tree | e0390fdc789e08b214d65b9ebbeb477cbe001021 /src/providers/ad | |
parent | a5623363d6042290fe652a1ca5ce5a85a821236f (diff) | |
download | sssd-42bc7cb28858f8affa5bc7586f8d39b3afe4c387.tar.gz sssd-42bc7cb28858f8affa5bc7586f8d39b3afe4c387.tar.xz sssd-42bc7cb28858f8affa5bc7586f8d39b3afe4c387.zip |
AD: Set dp_error if gc was not used
Global catalog was not used in ipa server mode and request failed then
dp_error was not set (default is zero). dp_error should not be OK
on failed request.
[ipa_get_ad_acct_ad_part_done] (0x0040): AD lookup failed: 11
[ipa_subdomain_account_done] (0x0040): ipa_get_*_acct request failed: 11
[sdap_id_op_destroy] (0x4000): releasing operation connection
[ipa_account_info_error_text] (0x0020): Bug: dp_error is OK on failed request
[acctinfo_callback] (0x0100): Request processed. Returned 3,11,Account info lookup failed
Reviewed-by: Sumit Bose <sbose@redhat.com>
Diffstat (limited to 'src/providers/ad')
-rw-r--r-- | src/providers/ad/ad_id.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/providers/ad/ad_id.c b/src/providers/ad/ad_id.c index ddb108370..995a71277 100644 --- a/src/providers/ad/ad_id.c +++ b/src/providers/ad/ad_id.c @@ -155,6 +155,10 @@ ad_handle_acct_info_done(struct tevent_req *subreq) } talloc_zfree(subreq); if (ret != EOK) { + /* if GC was not used dp error should be set */ + state->dp_error = dp_error; + state->err = err; + tevent_req_error(req, ret); return; } |