Attachment 10755 Details for Bug 11103 – git-am fix for 4.2.0

[patch] git-am fix for 4.2.0

bug-11103-4-2.fix (text/plain), 1.88 KB, created by Jeremy Allison on 2015-02-19 22:31:45 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Allison

Created: 2015-02-19 22:31:45 UTC

Size: 1.88 KB

patch

obsolete

>From 9b96ec668f3f9d841ad21f8a39452bfb09e4c8fd Mon Sep 17 00:00:00 2001
>From: Jeremy Allison <jra@samba.org>
>Date: Wed, 18 Feb 2015 21:27:37 -0800
>Subject: [PATCH] s3: smbd: signing. Ensure we respond correctly to an SMB2
> negprot with SMB2_NEGOTIATE_SIGNING_REQUIRED.
>
>Bug 11103:  - Samba does not set the required flags in the SMB2/SMB3 Negotiate Protocol Response when signing required by client
>
>https://bugzilla.samba.org/show_bug.cgi?id=11103
>
>Signed-off-by: Jeremy Allison <jra@samba.org>
>Reviewed-by: Steve French <smfrench@gmail.com>
>(cherry picked from commit 1cea6e5b6f8c0e28d5ba2d296c831c4878fca304)
>---
> source3/smbd/smb2_negprot.c   | 3 ++-
> source3/smbd/smb2_sesssetup.c | 4 +++-
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
>diff --git a/source3/smbd/smb2_negprot.c b/source3/smbd/smb2_negprot.c
>index 1f7f1a9..438756b 100644
>--- a/source3/smbd/smb2_negprot.c
>+++ b/source3/smbd/smb2_negprot.c
>@@ -221,7 +221,8 @@ NTSTATUS smbd_smb2_request_process_negprot(struct smbd_smb2_request *req)
> 	}
> 
> 	security_mode = SMB2_NEGOTIATE_SIGNING_ENABLED;
>-	if (lp_server_signing() == SMB_SIGNING_REQUIRED) {
>+	if (lp_server_signing() == SMB_SIGNING_REQUIRED ||
>+			(in_security_mode & SMB2_NEGOTIATE_SIGNING_REQUIRED)) {
> 		security_mode |= SMB2_NEGOTIATE_SIGNING_REQUIRED;
> 	}
> 
>diff --git a/source3/smbd/smb2_sesssetup.c b/source3/smbd/smb2_sesssetup.c
>index 57f623a..1511761 100644
>--- a/source3/smbd/smb2_sesssetup.c
>+++ b/source3/smbd/smb2_sesssetup.c
>@@ -186,7 +186,9 @@ static NTSTATUS smbd_smb2_auth_generic_return(struct smbXsrv_session *session,
> 	struct smbXsrv_connection *xconn = smb2req->xconn;
> 
> 	if ((in_security_mode & SMB2_NEGOTIATE_SIGNING_REQUIRED) ||
>-	    lp_server_signing() == SMB_SIGNING_REQUIRED) {
>+	    lp_server_signing() == SMB_SIGNING_REQUIRED ||
>+	    (xconn->smb2.server.security_mode &
>+			SMB2_NEGOTIATE_SIGNING_REQUIRED)) {
> 		x->global->signing_required = true;
> 	}
> 
>-- 
>1.9.1
>

Actions: View

Attachments on bug 11103: 10744 | 10745 | 10746 | 10755 | 10756 | 10759