From 66d7553b0e008601846c1fc7b3a82052cbb5d66d Mon Sep 17 00:00:00 2001 From: Stefan Metzmacher Date: Mon, 16 Apr 2012 12:14:07 +0200 Subject: [PATCH] s3:libsmb: fix anonymous session setups against windows servers Windows server doesn't set the SMB2_SESSION_FLAG_IS_GUEST nor SMB2_SESSION_FLAG_IS_NULL flag. This fix makes sure we don't try to verify a signature on the final session setup response. metze --- source3/libsmb/cliconnect.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/source3/libsmb/cliconnect.c b/source3/libsmb/cliconnect.c index 9d4e4e1f996..6e057a5fef4 100644 --- a/source3/libsmb/cliconnect.c +++ b/source3/libsmb/cliconnect.c @@ -1710,6 +1710,22 @@ static void cli_session_setup_ntlmssp_done(struct tevent_req *subreq) if (cli_state_protocol(state->cli) >= PROTOCOL_SMB2_02) { struct smbXcli_session *session = state->cli->smb2.session; + + if (state->ntlmssp_state->nt_hash == NULL) { + /* + * Windows server does not set the + * SMB2_SESSION_FLAG_IS_GUEST nor + * SMB2_SESSION_FLAG_IS_NULL flag. + * + * This fix makes sure we do not try + * to verify a signature on the final + * session setup response. + */ + TALLOC_FREE(state->ntlmssp_state); + tevent_req_done(req); + return; + } + status = smb2cli_session_set_session_key(session, state->ntlmssp_state->session_key, recv_iov); -- 2.11.4.GIT