s4:auth: kinit_to_ccache() should always use the canonicalized principal
authorStefan Metzmacher <metze@samba.org>
Fri, 13 Sep 2019 14:04:30 +0000 (16:04 +0200)
committerKarolin Seeger <kseeger@samba.org>
Wed, 25 Sep 2019 23:37:33 +0000 (23:37 +0000)
We should always use krb5_get_init_creds_opt_set_canonicalize()
and krb5_get_init_creds_opt_set_win2k() for heimdal
and expect the client principal to be changed.

There's no reason to have a different logic between MIT and Heimdal.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14124

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Guenther Deschner <gd@samba.org>
(cherry picked from commit 162b4199493c1f179e775a325a19ae7a136c418b)

source4/auth/kerberos/kerberos_util.c

index 50bf8feec9680c75158cc98d00b286d847532f2d..950d91f17378bc405aa8745bfa71117fbbc103df 100644 (file)
@@ -313,6 +313,8 @@ done:
         */
        krb5_get_init_creds_opt_set_win2k(smb_krb5_context->krb5_context,
                                          krb_options, true);
+       krb5_get_init_creds_opt_set_canonicalize(smb_krb5_context->krb5_context,
+                                                krb_options, true);
 #else /* MIT */
        krb5_get_init_creds_opt_set_canonicalize(krb_options, true);
 #endif