]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s3:auth: make_user_info_map() should not set mapped_state
authorStefan Metzmacher <metze@samba.org>
Thu, 3 Mar 2022 22:24:25 +0000 (23:24 +0100)
committerJule Anger <janger@samba.org>
Wed, 16 Mar 2022 14:27:12 +0000 (14:27 +0000)
mapped_state is only evaluated in authsam_check_password_internals()
of auth_sam.c in source4, so setting it in the auth3 code
doesn't make any difference. I've proved that with
an SMB_ASSERT() and a full pipeline not triggering it.

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

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
(cherry picked from commit c56cb12f347b7582290ce1d4dfe3959d69050bd9)

source3/auth/auth_util.c

index 9d9f2234f210525adb101a04a147635b6ce6eb99..a0260b67c19308884976dbc101c594a3d9019917 100644 (file)
@@ -136,8 +136,6 @@ NTSTATUS make_user_info_map(TALLOC_CTX *mem_ctx,
                                lm_interactive_pwd, nt_interactive_pwd,
                                plaintext, password_state);
        if (NT_STATUS_IS_OK(result)) {
-               /* We have tried mapping */
-               (*user_info)->mapped_state = true;
                /* did we actually map the user to a different name? */
                (*user_info)->was_mapped = was_mapped;
        }