From: Jeremy Allison <jra@samba.org>
Date: Mon, 23 Jan 2012 22:09:32 +0000 (-0800)
Subject: Another fix for bug #8556 - ACL permissions ignored when SMBsetatr is requested.
X-Git-Tag: tevent-0.9.15~271
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=869fd8eeba00bd62e89de344184ef748c5c0f9c0;p=thirdparty%2Fsamba.git

Another fix for bug #8556 - ACL permissions ignored when SMBsetatr is requested.

Remove erroneous check on FILE_WRITE_ATTRIBUTES when changing POSIX
permissions - this isn't an attribute set call (unless you're storing
attributes in POSIX permissions, which is not recommended).

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Jan 24 00:44:24 CET 2012 on sn-devel-104
---

diff --git a/source3/smbd/trans2.c b/source3/smbd/trans2.c
index ba6bcbbb9e2..cee6c351890 100644
--- a/source3/smbd/trans2.c
+++ b/source3/smbd/trans2.c
@@ -6970,11 +6970,6 @@ static NTSTATUS smb_set_file_unix_basic(connection_struct *conn,
 	}
 #endif
 
-	status = check_access(conn, fsp, smb_fname, FILE_WRITE_ATTRIBUTES);
-	if (!NT_STATUS_IS_OK(status)) {
-		return status;
-	}
-
 	/*
 	 * Deal with the UNIX specific mode set.
 	 */