From 42bc7cb28858f8affa5bc7586f8d39b3afe4c387 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Fri, 28 Nov 2014 12:32:25 +0100 Subject: 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 --- src/providers/ad/ad_id.c | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'src') diff --git a/src/providers/ad/ad_id.c b/src/providers/ad/ad_id.c index ddb10837..995a7127 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; } -- cgit