Fix bug 9678 - Windows 8 Roaming profiles fail
authorGregor Beck <gbeck@sernet.de>
Thu, 1 Aug 2013 12:16:24 +0000 (14:16 +0200)
committerKarolin Seeger <kseeger@samba.org>
Mon, 5 Aug 2013 18:04:11 +0000 (20:04 +0200)
Windows 8 tries to set 'ATTRIBUTE_SECURITY_INFORMATION' on some
dirs. Ignoring it makes roaming profiles work again.
Just like w2k3 gracefully ignore all the other bits.

Signed-off-by: Gregor Beck <gbeck@sernet.de>
source3/smbd/nttrans.c

index 5fc3a09784d86b734253b49e8512a42c3bcc1463..a884b2f38f424b070c4c7aaf5e2763ea7f8be275 100644 (file)
@@ -885,13 +885,8 @@ NTSTATUS set_sd(files_struct *fsp, struct security_descriptor *psd,
 
        /* Ensure we have at least one thing set. */
        if ((security_info_sent & (SECINFO_OWNER|SECINFO_GROUP|SECINFO_DACL|SECINFO_SACL)) == 0) {
-               if (security_info_sent & SECINFO_LABEL) {
-                       /* Only consider SECINFO_LABEL if no other
-                          bits are set. Just like W2K3 we don't
-                          store this. */
-                       return NT_STATUS_OK;
-               }
-               return NT_STATUS_INVALID_PARAMETER;
+               /* Just like W2K3 */
+               return NT_STATUS_OK;
        }
 
        /* Ensure we have the rights to do this. */