SMB: Validate negotiate (to protect against downgrade) even if signing off
authorSteve French <smfrench@gmail.com>
Thu, 21 Sep 2017 00:57:18 +0000 (19:57 -0500)
committerpopcornmix <popcornmix@gmail.com>
Fri, 6 Oct 2017 12:06:50 +0000 (13:06 +0100)
commit7e97781cc17392343dac4406cc42e95d9c1693a4
treea467346bf0888a1c17e306e531117ec2539e0056
parent68256369b18995603e6665f42f171b98281557e7
SMB: Validate negotiate (to protect against downgrade) even if signing off

commit 0603c96f3af50e2f9299fa410c224ab1d465e0f9 upstream.

As long as signing is supported (ie not a guest user connection) and
connection is SMB3 or SMB3.02, then validate negotiate (protect
against man in the middle downgrade attacks).  We had been doing this
only when signing was required, not when signing was just enabled,
but this more closely matches recommended SMB3 behavior and is
better security.  Suggested by Metze.

Signed-off-by: Steve French <smfrench@gmail.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
Acked-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Ronnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/cifs/smb2pdu.c