From: Stefan Metzmacher Date: Fri, 12 May 2017 07:10:19 +0000 (+0200) Subject: auth/spnego: fix gensec_update_ev() argument order for the SPNEGO_FALLBACK case X-Git-Tag: samba-4.5.11~14 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=48a697132f0c80d3ee2b9926bf6e551f0a4e6a15;p=thirdparty%2Fsamba.git 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 Reviewed-by: Andrew Bartlett (cherry picked from commit 31691963b3d9ac460df0c56d36f81ec815db0225) --- diff --git a/auth/gensec/spnego.c b/auth/gensec/spnego.c index 5f5047a808c..962e1fc4f5b 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")); @@ -756,8 +756,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;