diff options
| author | Sumit Bose <sbose@redhat.com> | 2017-06-08 11:06:02 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2017-06-08 16:25:03 +0200 |
| commit | 614057ea85c05d3a6d4b62217a41b8b5db8d5d38 (patch) | |
| tree | d2673739d91f2a676e2acb9dec36bbf735543abf | |
| parent | 13205258cc17d3833558244251f5adbc98cf34e5 (diff) | |
| download | sssd-614057ea85c05d3a6d4b62217a41b8b5db8d5d38.tar.gz sssd-614057ea85c05d3a6d4b62217a41b8b5db8d5d38.tar.xz sssd-614057ea85c05d3a6d4b62217a41b8b5db8d5d38.zip | |
krb5: disable enterprise principals during password changes
Currently using enterprise principals during password changes does not
work reliable.
First there is a special behavior if canonicalization, which in general
should be used together with enterprise principals, is enabled with AD,
see https://pagure.io/SSSD/sssd/issue/1405 and
https://pagure.io/SSSD/sssd/issue/1615 for details. As a result of this
SSSD currently disables canonicalization during password changes.
Additionally it looks like MIT Kerberos does not handle canonicalized
principals well, even if canonicalization is enabled, if not the default
krbtgt/REALM@REALM but kadmin/changepw@REALM is requested. Since it is
currently not clear what is the expected behavior here it make sense to
completely disable enterprise principals during password changes for the
time being.
Resolves https://pagure.io/SSSD/sssd/issue/3426
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
| -rw-r--r-- | src/providers/krb5/krb5_child_handler.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/providers/krb5/krb5_child_handler.c b/src/providers/krb5/krb5_child_handler.c index 11ac867e6..0007f92a6 100644 --- a/src/providers/krb5/krb5_child_handler.c +++ b/src/providers/krb5/krb5_child_handler.c @@ -143,7 +143,8 @@ static errno_t create_send_buffer(struct krb5child_req *kr, return EINVAL; } - if (kr->pd->cmd == SSS_CMD_RENEW || kr->is_offline) { + if (kr->pd->cmd == SSS_CMD_RENEW || kr->pd->cmd == SSS_PAM_CHAUTHTOK_PRELIM + || kr->pd->cmd == SSS_PAM_CHAUTHTOK || kr->is_offline) { use_enterprise_principal = false; } else { use_enterprise_principal = dp_opt_get_bool(kr->krb5_ctx->opts, |