diff options
author | Stefan Metzmacher <metze@samba.org> | 2017-05-12 09:10:19 +0200 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2017-05-21 21:05:08 +0200 |
commit | 31691963b3d9ac460df0c56d36f81ec815db0225 (patch) | |
tree | 0a513f9e4de961289316f7f21170e975a98dff31 /auth | |
parent | 1116ee1c5c47b405666a0d52f646d05613a803ab (diff) | |
download | samba-31691963b3d9ac460df0c56d36f81ec815db0225.tar.gz |
auth/spnego: fix gensec_update_ev() argument order for the SPNEGO_FALLBACK case
This went unnoticed so long as we don't use -Wc++-compat
and gensec_update_ev() used the sync update() hook for all
NTLMSSP and Kerberos.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=12788
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Diffstat (limited to 'auth')
-rw-r--r-- | auth/gensec/spnego.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/auth/gensec/spnego.c b/auth/gensec/spnego.c index 484f49175fc..645c8b2e087 100644 --- a/auth/gensec/spnego.c +++ b/auth/gensec/spnego.c @@ -366,7 +366,7 @@ static NTSTATUS gensec_spnego_server_try_fallback(struct gensec_security *gensec return nt_status; } nt_status = gensec_update_ev(spnego_state->sub_sec_security, - ev, out_mem_ctx, in, out); + out_mem_ctx, ev, in, out); return nt_status; } DEBUG(1, ("Failed to parse SPNEGO request\n")); @@ -804,8 +804,8 @@ static NTSTATUS gensec_spnego_update(struct gensec_security *gensec_security, TA switch (spnego_state->state_position) { case SPNEGO_FALLBACK: - return gensec_update_ev(spnego_state->sub_sec_security, ev, - out_mem_ctx, in, out); + return gensec_update_ev(spnego_state->sub_sec_security, + out_mem_ctx, ev, in, out); case SPNEGO_SERVER_START: { NTSTATUS nt_status; |