diff options
| author | Stefan Metzmacher <metze@samba.org> | 2017-05-12 09:10:19 +0200 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2017-06-30 10:50:22 +0200 |
| commit | d2bf63c93f2717a405727c4e1024d4c8347a142d (patch) | |
| tree | ebd1e07f700a5f32f7b241e1bef6d7b69f285738 /auth | |
| parent | 545b0c420d567f1f50dac734a97b478fd09bd985 (diff) | |
| download | samba-d2bf63c93f2717a405727c4e1024d4c8347a142d.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>
(cherry picked from commit 31691963b3d9ac460df0c56d36f81ec815db0225)
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 f063f7b358b..21c6cfb422a 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; |