diff options
| author | Stefan Metzmacher <metze@samba.org> | 2022-03-03 23:24:25 +0100 |
|---|---|---|
| committer | Jule Anger <janger@samba.org> | 2022-03-16 14:27:12 +0000 |
| commit | 57401a170aaa097e42c6e310e7ed76d9ae5a0b60 (patch) | |
| tree | ad2b383e408ca7224b4e80e202dd562e06aa8670 | |
| parent | 311a4cc141acaae8a10084f56e23efa352518ff3 (diff) | |
| download | samba-57401a170aaa097e42c6e310e7ed76d9ae5a0b60.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 9d9f2234f21..a0260b67c19 100644 --- a/source3/auth/auth_util.c +++ b/source3/auth/auth_util.c @@ -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; } |