From 0c37b025b3da6bed26d7c84c4254f8ecc05bfc77 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Fri, 15 May 2015 13:36:03 +0200 Subject: IPA: Do not set AD_KRB5_REALM twice Related: https://fedorahosted.org/sssd/ticket/2638 Both the AD common code and ipa_ad_ctx_new() used set AD_KRB5_REALM. As verified by unit tests, we don't need to set the parameter twice. Reviewed-by: Sumit Bose --- src/providers/ipa/ipa_subdomains_server.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/providers/ipa/ipa_subdomains_server.c b/src/providers/ipa/ipa_subdomains_server.c index c2e6f567e..caa3821c3 100644 --- a/src/providers/ipa/ipa_subdomains_server.c +++ b/src/providers/ipa/ipa_subdomains_server.c @@ -156,14 +156,6 @@ ipa_ad_ctx_new(struct be_ctx *be_ctx, return ENOMEM; } - ret = dp_opt_set_string(ad_options->basic, AD_KRB5_REALM, - id_ctx->server_mode->realm); - if (ret != EOK) { - DEBUG(SSSDBG_OP_FAILURE, "Cannot set AD realm\n"); - talloc_free(ad_options); - return ret; - } - extra_attrs = dp_opt_get_string(id_ctx->sdap_id_ctx->opts->basic, SDAP_USER_EXTRA_ATTRS); if (extra_attrs != NULL) { -- cgit