>From 72f34bbe3f75200eeff652a245466372433958ca Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Fri, 28 Nov 2014 12:32:25 +0100 Subject: [PATCH 1/2] 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 --- src/providers/ad/ad_id.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/providers/ad/ad_id.c b/src/providers/ad/ad_id.c index ddb1083703561ddf99c88762b8f5cf9d28da7e08..995a7127795b8fe0839b05d3dcbe56cca531598a 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; } -- 2.1.0