diff options
| author | Stefan Metzmacher <metze@samba.org> | 2022-03-03 23:24:25 +0100 |
|---|---|---|
| committer | Jule Anger <janger@samba.org> | 2022-03-16 13:41:14 +0000 |
| commit | c46c341016d82566d97829d67b40a0e7bed25f36 (patch) | |
| tree | 8a8977945c8afa369bafa96e99dc5bdb13e9526b | |
| parent | a219a81ff89912cbd55050e9e1fa78731f66e3bc (diff) | |
| download | samba-c46c341016d82566d97829d67b40a0e7bed25f36.tar.gz | |
s3:auth: make_user_info_map() should not set mapped_state
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)
| -rw-r--r-- | source3/auth/auth_util.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/source3/auth/auth_util.c b/source3/auth/auth_util.c index d7b3b032572..b60dd2647c8 100644 --- a/source3/auth/auth_util.c +++ b/source3/auth/auth_util.c @@ -137,8 +137,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; } |