Attachment 12112 Details for Bug 11910 – Patch for v4-3-test

[patch] Patch for v4-3-test

tmp43.diff.txt (text/plain), 2.43 KB, created by Stefan Metzmacher on 2016-05-19 11:40:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Stefan Metzmacher

Created: 2016-05-19 11:40:29 UTC

Size: 2.43 KB

patch

obsolete

>From fd1cd381da9a0c973325c72ccfea854eec85c07b Mon Sep 17 00:00:00 2001
>From: Stefan Metzmacher <metze@samba.org>
>Date: Wed, 18 May 2016 09:56:02 +0200
>Subject: [PATCH] s3:smbd: fix anonymous authentication if signing is mandatory
>
>BUG: https://bugzilla.samba.org/show_bug.cgi?id=11910
>
>Signed-off-by: Stefan Metzmacher <metze@samba.org>
>Reviewed-by: Andreas Schneider <asn@samba.org>
>
>Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
>Autobuild-Date(master): Wed May 18 15:49:46 CEST 2016 on sn-devel-144
>
>(cherry picked from commit aa5760433b219de7b41d35ca7ad6d4d702b89adf)
>---
> source3/smbd/sesssetup.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
>diff --git a/source3/smbd/sesssetup.c b/source3/smbd/sesssetup.c
>index 7774b66..e00704c 100644
>--- a/source3/smbd/sesssetup.c
>+++ b/source3/smbd/sesssetup.c
>@@ -135,6 +135,7 @@ static void reply_sesssetup_and_X_spnego(struct smb_request *req)
> 	struct smbXsrv_connection *xconn = req->xconn;
> 	struct smbd_server_connection *sconn = req->sconn;
> 	uint16_t action = 0;
>+	bool is_authenticated = false;
> 	NTTIME now = timeval_to_nttime(&req->request_time);
> 	struct smbXsrv_session *session = NULL;
> 	uint16_t smb_bufsize = SVAL(req->vwv+2, 0);
>@@ -328,12 +329,13 @@ static void reply_sesssetup_and_X_spnego(struct smb_request *req)
> 		sconn->num_users++;
> 
> 		if (security_session_user_level(session_info, NULL) >= SECURITY_USER) {
>+			is_authenticated = true;
> 			session->compat->homes_snum =
> 				register_homes_share(session_info->unix_info->unix_name);
> 		}
> 
> 		if (srv_is_signing_negotiated(xconn) &&
>-		    action == 0 &&
>+		    is_authenticated &&
> 		    session->global->signing_key.length > 0)
> 		{
> 			/*
>@@ -593,6 +595,7 @@ void reply_sesssetup_and_X(struct smb_request *req)
> 	struct auth_session_info *session_info = NULL;
> 	uint16_t smb_flag2 = req->flags2;
> 	uint16_t action = 0;
>+	bool is_authenticated = false;
> 	NTTIME now = timeval_to_nttime(&req->request_time);
> 	struct smbXsrv_session *session = NULL;
> 	NTSTATUS nt_status;
>@@ -1030,12 +1033,13 @@ void reply_sesssetup_and_X(struct smb_request *req)
> 	sconn->num_users++;
> 
> 	if (security_session_user_level(session_info, NULL) >= SECURITY_USER) {
>+		is_authenticated = true;
> 		session->compat->homes_snum =
> 			register_homes_share(session_info->unix_info->unix_name);
> 	}
> 
> 	if (srv_is_signing_negotiated(xconn) &&
>-	    action == 0 &&
>+	    is_authenticated &&
> 	    session->global->signing_key.length > 0)
> 	{
> 		/*
>-- 
>1.9.1
>

Flags:

asn: review+
metze: review? (gd)

Actions: View

Attachments on bug 11910: 12107 | 12111 | 12112 | 12113