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 ba6bcbbb9e288cfac6f3684844bd0f09b4bcae29..cee6c35189002d4a7a1136df59238130b3bb7f9f 100644 (file)
--- 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.
         */