summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2015-05-15 13:36:03 +0200
committerJakub Hrozek <jhrozek@redhat.com>2015-06-14 21:47:23 +0200
commit0c37b025b3da6bed26d7c84c4254f8ecc05bfc77 (patch)
treed8a3263507fe2d52c1ef8c0dac325f896b3dc15b
parentde2bad8ae08f09964834bda0f88db9de39f47c5c (diff)
downloadsssd-0c37b025b3da6bed26d7c84c4254f8ecc05bfc77.tar.gz
sssd-0c37b025b3da6bed26d7c84c4254f8ecc05bfc77.tar.xz
sssd-0c37b025b3da6bed26d7c84c4254f8ecc05bfc77.zip
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 <sbose@redhat.com>
-rw-r--r--src/providers/ipa/ipa_subdomains_server.c8
1 files changed, 0 insertions, 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) {