diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2015-05-15 13:36:03 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-06-14 21:47:23 +0200 |
commit | 0c37b025b3da6bed26d7c84c4254f8ecc05bfc77 (patch) | |
tree | d8a3263507fe2d52c1ef8c0dac325f896b3dc15b /src/providers/ipa | |
parent | de2bad8ae08f09964834bda0f88db9de39f47c5c (diff) | |
download | sssd-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>
Diffstat (limited to 'src/providers/ipa')
-rw-r--r-- | src/providers/ipa/ipa_subdomains_server.c | 8 |
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) { |