sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2025-01-06 13:18:07 +03:00
Code

dcesrv_core: a failure from gensec_update results in NAK_REASON_INVALID_CHECKSUM

Browse Source 
We already report that for gensec_start_mech_by_authtype() failures,
but we also need to do that for any invalid authentication.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14356

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
(cherry picked from commit 97545873eb)
This commit is contained in:
Stefan Metzmacher 2020-11-12 16:41:05 +01:00 committed by Jule Anger
parent f27161ef53
commit aa0e68958c
2 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
librpc/rpc/dcesrv_core.c
View File

@ -1362,7 +1362,7 @@ static void dcesrv_bind_done(struct tevent_req *subreq)
status = dcesrv_auth_complete(call, status);
if (!NT_STATUS_IS_OK(status)) {
status = dcesrv_bind_nak(call, 0);
status = dcesrv_bind_nak(call, DCERPC_BIND_NAK_REASON_INVALID_CHECKSUM);
dcesrv_conn_auth_wait_finished(conn, status);
return;
}

2
selftest/knownfail.d/dcerpc-auth-pad
View File

@ -11,5 +11,3 @@
^samba.tests.dcerpc.raw_protocol.samba.tests.dcerpc.raw_protocol.TestDCERPC_BIND.test_auth_tail_pad_spnego_auth3
^samba.tests.dcerpc.raw_protocol.samba.tests.dcerpc.raw_protocol.TestDCERPC_BIND.test_schannel_invalid_alter_no_padding.*chgdcpass
^samba.tests.dcerpc.raw_protocol.samba.tests.dcerpc.raw_protocol.TestDCERPC_BIND.test_schannel_invalid_alter_tail_padding.*chgdcpass
^samba.tests.dcerpc.raw_protocol.samba.tests.dcerpc.raw_protocol.TestDCERPC_BIND.test_schannel_invalid_bind_no_padding.*chgdcpass
^samba.tests.dcerpc.raw_protocol.samba.tests.dcerpc.raw_protocol.TestDCERPC_BIND.test_schannel_invalid_bind_tail_padding.*chgdcpass