mirror of
https://github.com/samba-team/samba.git
synced 2025-08-26 01:49:31 +03:00
auth/ntlmssp: don't send domain and workstation in the NEGOTIATE_MESSAGE
We don't set NTLMSSP_NEGOTIATE_OEM_{DOMAIN,WORKSTATION}_SUPPLIED anyway.
This matches modern Windows clients.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11804
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Günther Deschner <gd@samba.org>
(cherry picked from commit 4fca8eaaae
)
This commit is contained in:
@ -58,28 +58,16 @@ NTSTATUS ntlmssp_client_initial(struct gensec_security *gensec_security,
|
||||
talloc_get_type_abort(gensec_security->private_data,
|
||||
struct gensec_ntlmssp_context);
|
||||
struct ntlmssp_state *ntlmssp_state = gensec_ntlmssp->ntlmssp_state;
|
||||
const char *domain = ntlmssp_state->client.netbios_domain;
|
||||
const char *workstation = ntlmssp_state->client.netbios_name;
|
||||
NTSTATUS status;
|
||||
|
||||
/* These don't really matter in the initial packet, so don't panic if they are not set */
|
||||
if (!domain) {
|
||||
domain = "";
|
||||
}
|
||||
|
||||
if (!workstation) {
|
||||
workstation = "";
|
||||
}
|
||||
|
||||
/* generate the ntlmssp negotiate packet */
|
||||
status = msrpc_gen(out_mem_ctx,
|
||||
out, "CddAA",
|
||||
"NTLMSSP",
|
||||
NTLMSSP_NEGOTIATE,
|
||||
ntlmssp_state->neg_flags,
|
||||
domain,
|
||||
workstation);
|
||||
|
||||
"", /* domain */
|
||||
""); /* workstation */
|
||||
if (!NT_STATUS_IS_OK(status)) {
|
||||
DEBUG(0, ("ntlmssp_client_initial: failed to generate "
|
||||
"ntlmssp negotiate packet\n"));
|
||||
|
Reference in New Issue
Block a user