s3: VFS: Modify SMB_VFS_GET_NT_ACL to take a const struct smb_filename * instead...
[garming/samba-autobuild/.git] / source3 / modules / vfs_acl_common.c
index 6c57acb13dddbe423e5dae8fa20908e1349024d0..30574e0e06fcb9e146039be7287a7350721b00de 100644 (file)
@@ -23,6 +23,7 @@
 #include "system/filesys.h"
 #include "../libcli/security/security.h"
 #include "../librpc/gen_ndr/ndr_security.h"
+#include "../lib/util/bitmap.h"
 
 static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
                        DATA_BLOB *pblob,
@@ -48,11 +49,28 @@ static NTSTATUS store_acl_blob_fsp(vfs_handle_struct *handle,
  Hash a security descriptor.
 *******************************************************************/
 
+static NTSTATUS hash_blob_sha256(DATA_BLOB blob,
+                                uint8_t *hash)
+{
+       SHA256_CTX tctx;
+
+       memset(hash, '\0', XATTR_SD_HASH_SIZE);
+
+       samba_SHA256_Init(&tctx);
+       samba_SHA256_Update(&tctx, blob.data, blob.length);
+       samba_SHA256_Final(hash, &tctx);
+
+       return NT_STATUS_OK;
+}
+
+/*******************************************************************
+ Hash a security descriptor.
+*******************************************************************/
+
 static NTSTATUS hash_sd_sha256(struct security_descriptor *psd,
                        uint8_t *hash)
 {
        DATA_BLOB blob;
-       SHA256_CTX tctx;
        NTSTATUS status;
 
        memset(hash, '\0', XATTR_SD_HASH_SIZE);
@@ -60,12 +78,7 @@ static NTSTATUS hash_sd_sha256(struct security_descriptor *psd,
        if (!NT_STATUS_IS_OK(status)) {
                return status;
        }
-
-       samba_SHA256_Init(&tctx);
-       samba_SHA256_Update(&tctx, blob.data, blob.length);
-       samba_SHA256_Final(hash, &tctx);
-
-       return NT_STATUS_OK;
+       return hash_blob_sha256(blob, hash);
 }
 
 /*******************************************************************
@@ -73,27 +86,45 @@ static NTSTATUS hash_sd_sha256(struct security_descriptor *psd,
 *******************************************************************/
 
 static NTSTATUS parse_acl_blob(const DATA_BLOB *pblob,
-                               struct security_descriptor **ppdesc,
-                               uint16_t *p_hash_type,
-                               uint8_t hash[XATTR_SD_HASH_SIZE])
+                              TALLOC_CTX *mem_ctx,
+                              struct security_descriptor **ppdesc,
+                              uint16_t *p_hash_type,
+                              uint16_t *p_version,
+                              uint8_t hash[XATTR_SD_HASH_SIZE],
+                              uint8_t sys_acl_hash[XATTR_SD_HASH_SIZE])
 {
-       TALLOC_CTX *ctx = talloc_tos();
        struct xattr_NTACL xacl;
        enum ndr_err_code ndr_err;
        size_t sd_size;
+       TALLOC_CTX *frame = talloc_stackframe();
 
-       ndr_err = ndr_pull_struct_blob(pblob, ctx, &xacl,
+       ndr_err = ndr_pull_struct_blob(pblob, frame, &xacl,
                        (ndr_pull_flags_fn_t)ndr_pull_xattr_NTACL);
 
        if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
                DEBUG(5, ("parse_acl_blob: ndr_pull_xattr_NTACL failed: %s\n",
                        ndr_errstr(ndr_err)));
+               TALLOC_FREE(frame);
                return ndr_map_error2ntstatus(ndr_err);
        }
 
+       *p_version = xacl.version;
+
        switch (xacl.version) {
+               case 1:
+                       *ppdesc = make_sec_desc(mem_ctx, SD_REVISION,
+                                       xacl.info.sd->type | SEC_DESC_SELF_RELATIVE,
+                                       xacl.info.sd->owner_sid,
+                                       xacl.info.sd->group_sid,
+                                       xacl.info.sd->sacl,
+                                       xacl.info.sd->dacl,
+                                       &sd_size);
+                       /* No hash - null out. */
+                       *p_hash_type = XATTR_SD_HASH_TYPE_NONE;
+                       memset(hash, '\0', XATTR_SD_HASH_SIZE);
+                       break;
                case 2:
-                       *ppdesc = make_sec_desc(ctx, SD_REVISION,
+                       *ppdesc = make_sec_desc(mem_ctx, SD_REVISION,
                                        xacl.info.sd_hs2->sd->type | SEC_DESC_SELF_RELATIVE,
                                        xacl.info.sd_hs2->sd->owner_sid,
                                        xacl.info.sd_hs2->sd->group_sid,
@@ -105,7 +136,7 @@ static NTSTATUS parse_acl_blob(const DATA_BLOB *pblob,
                        memset(hash, '\0', XATTR_SD_HASH_SIZE);
                        break;
                case 3:
-                       *ppdesc = make_sec_desc(ctx, SD_REVISION,
+                       *ppdesc = make_sec_desc(mem_ctx, SD_REVISION,
                                        xacl.info.sd_hs3->sd->type | SEC_DESC_SELF_RELATIVE,
                                        xacl.info.sd_hs3->sd->owner_sid,
                                        xacl.info.sd_hs3->sd->group_sid,
@@ -113,20 +144,35 @@ static NTSTATUS parse_acl_blob(const DATA_BLOB *pblob,
                                        xacl.info.sd_hs3->sd->dacl,
                                        &sd_size);
                        *p_hash_type = xacl.info.sd_hs3->hash_type;
-                       /* Current version 3. */
+                       /* Current version 3 (if no sys acl hash available). */
                        memcpy(hash, xacl.info.sd_hs3->hash, XATTR_SD_HASH_SIZE);
                        break;
+               case 4:
+                       *ppdesc = make_sec_desc(mem_ctx, SD_REVISION,
+                                       xacl.info.sd_hs4->sd->type | SEC_DESC_SELF_RELATIVE,
+                                       xacl.info.sd_hs4->sd->owner_sid,
+                                       xacl.info.sd_hs4->sd->group_sid,
+                                       xacl.info.sd_hs4->sd->sacl,
+                                       xacl.info.sd_hs4->sd->dacl,
+                                       &sd_size);
+                       *p_hash_type = xacl.info.sd_hs4->hash_type;
+                       /* Current version 4. */
+                       memcpy(hash, xacl.info.sd_hs4->hash, XATTR_SD_HASH_SIZE);
+                       memcpy(sys_acl_hash, xacl.info.sd_hs4->sys_acl_hash, XATTR_SD_HASH_SIZE);
+                       break;
                default:
+                       TALLOC_FREE(frame);
                        return NT_STATUS_REVISION_MISMATCH;
        }
 
-       TALLOC_FREE(xacl.info.sd);
+       TALLOC_FREE(frame);
 
        return (*ppdesc != NULL) ? NT_STATUS_OK : NT_STATUS_NO_MEMORY;
 }
 
 /*******************************************************************
- Create a DATA_BLOB from a security descriptor.
+ Create a DATA_BLOB from a hash of the security descriptor storead at
+ the system layer and the NT ACL we wish to preserve
 *******************************************************************/
 
 static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
@@ -144,7 +190,7 @@ static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
 
        xacl.version = 3;
        xacl.info.sd_hs3 = &sd_hs3;
-       xacl.info.sd_hs3->sd = CONST_DISCARD(struct security_descriptor *, psd);
+       xacl.info.sd_hs3->sd = discard_const_p(struct security_descriptor, psd);
        xacl.info.sd_hs3->hash_type = hash_type;
        memcpy(&xacl.info.sd_hs3->hash[0], hash, XATTR_SD_HASH_SIZE);
 
@@ -161,12 +207,58 @@ static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
        return NT_STATUS_OK;
 }
 
+/*******************************************************************
+ Create a DATA_BLOB from a hash of the security descriptors 
+ (system and NT) stored at the system layer and the NT ACL we wish 
+ to preserve.
+*******************************************************************/
+
+static NTSTATUS create_sys_acl_blob(const struct security_descriptor *psd,
+                                   DATA_BLOB *pblob,
+                                   uint16_t hash_type,
+                                   uint8_t hash[XATTR_SD_HASH_SIZE],
+                                   const char *description,
+                                   uint8_t sys_acl_hash[XATTR_SD_HASH_SIZE])
+{
+       struct xattr_NTACL xacl;
+       struct security_descriptor_hash_v4 sd_hs4;
+       enum ndr_err_code ndr_err;
+       TALLOC_CTX *ctx = talloc_tos();
+       NTTIME nttime_now;
+       struct timeval now = timeval_current();
+       nttime_now = timeval_to_nttime(&now);
+
+       ZERO_STRUCT(xacl);
+       ZERO_STRUCT(sd_hs4);
+
+       xacl.version = 4;
+       xacl.info.sd_hs4 = &sd_hs4;
+       xacl.info.sd_hs4->sd = discard_const_p(struct security_descriptor, psd);
+       xacl.info.sd_hs4->hash_type = hash_type;
+       memcpy(&xacl.info.sd_hs4->hash[0], hash, XATTR_SD_HASH_SIZE);
+       xacl.info.sd_hs4->description = description;
+       xacl.info.sd_hs4->time = nttime_now;
+       memcpy(&xacl.info.sd_hs4->sys_acl_hash[0], sys_acl_hash, XATTR_SD_HASH_SIZE);
+
+       ndr_err = ndr_push_struct_blob(
+                       pblob, ctx, &xacl,
+                       (ndr_push_flags_fn_t)ndr_push_xattr_NTACL);
+
+       if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
+               DEBUG(5, ("create_acl_blob: ndr_push_xattr_NTACL failed: %s\n",
+                       ndr_errstr(ndr_err)));
+               return ndr_map_error2ntstatus(ndr_err);
+       }
+
+       return NT_STATUS_OK;
+}
+
 /*******************************************************************
  Add in 3 inheritable components for a non-inheritable directory ACL.
  CREATOR_OWNER/CREATOR_GROUP/WORLD.
 *******************************************************************/
 
-static void add_directory_inheritable_components(vfs_handle_struct *handle,
+static NTSTATUS add_directory_inheritable_components(vfs_handle_struct *handle,
                                 const char *name,
                                SMB_STRUCT_STAT *psbuf,
                                struct security_descriptor *psd)
@@ -179,18 +271,30 @@ static void add_directory_inheritable_components(vfs_handle_struct *handle,
        mode_t dir_mode;
        mode_t file_mode;
        mode_t mode;
-       struct security_ace *new_ace_list = TALLOC_ZERO_ARRAY(talloc_tos(),
-                                               struct security_ace,
-                                               num_aces + 3);
+       struct security_ace *new_ace_list;
+
+       if (psd->dacl) {
+               new_ace_list = talloc_zero_array(psd->dacl,
+                                                struct security_ace,
+                                                num_aces + 3);
+       } else {
+               /*
+                * make_sec_acl() at the bottom of this function
+                * dupliates new_ace_list
+                */
+               new_ace_list = talloc_zero_array(talloc_tos(),
+                                                struct security_ace,
+                                                num_aces + 3);
+       }
 
        if (new_ace_list == NULL) {
-               return;
+               return NT_STATUS_NO_MEMORY;
        }
 
        /* Fake a quick smb_filename. */
        ZERO_STRUCT(smb_fname);
        smb_fname.st = *psbuf;
-       smb_fname.base_name = CONST_DISCARD(char *, name);
+       smb_fname.base_name = discard_const_p(char, name);
 
        dir_mode = unix_mode(conn,
                        FILE_ATTRIBUTE_DIRECTORY, &smb_fname, NULL);
@@ -236,8 +340,22 @@ static void add_directory_inheritable_components(vfs_handle_struct *handle,
                        SEC_ACE_FLAG_CONTAINER_INHERIT|
                                SEC_ACE_FLAG_OBJECT_INHERIT|
                                SEC_ACE_FLAG_INHERIT_ONLY);
-       psd->dacl->aces = new_ace_list;
-       psd->dacl->num_aces += 3;
+       if (psd->dacl) {
+               psd->dacl->aces = new_ace_list;
+               psd->dacl->num_aces += 3;
+               psd->dacl->size += new_ace_list[num_aces].size +
+                       new_ace_list[num_aces+1].size +
+                       new_ace_list[num_aces+2].size;
+       } else {
+               psd->dacl = make_sec_acl(psd,
+                               NT4_ACL_REVISION,
+                               3,
+                               new_ace_list);
+               if (psd->dacl == NULL) {
+                       return NT_STATUS_NO_MEMORY;
+               }
+       }
+       return NT_STATUS_OK;
 }
 
 /*******************************************************************
@@ -247,109 +365,242 @@ static void add_directory_inheritable_components(vfs_handle_struct *handle,
 *******************************************************************/
 
 static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
-                               files_struct *fsp,
-                               const char *name,
-                               uint32_t security_info,
-                               struct security_descriptor **ppdesc)
+                                   files_struct *fsp,
+                                   const struct smb_filename *smb_fname,
+                                   uint32_t security_info,
+                                   TALLOC_CTX *mem_ctx,
+                                   struct security_descriptor **ppdesc)
 {
-       DATA_BLOB blob;
+       DATA_BLOB blob = data_blob_null;
        NTSTATUS status;
        uint16_t hash_type = XATTR_SD_HASH_TYPE_NONE;
+       uint16_t xattr_version = 0;
        uint8_t hash[XATTR_SD_HASH_SIZE];
+       uint8_t sys_acl_hash[XATTR_SD_HASH_SIZE];
        uint8_t hash_tmp[XATTR_SD_HASH_SIZE];
+       uint8_t sys_acl_hash_tmp[XATTR_SD_HASH_SIZE];
        struct security_descriptor *psd = NULL;
        struct security_descriptor *pdesc_next = NULL;
+       const char *name = NULL;
        bool ignore_file_system_acl = lp_parm_bool(SNUM(handle->conn),
                                                ACL_MODULE_NAME,
                                                "ignore system acls",
                                                false);
+       TALLOC_CTX *frame = talloc_stackframe();
 
-       if (fsp && name == NULL) {
+       if (fsp && smb_fname == NULL) {
                name = fsp->fsp_name->base_name;
+       } else {
+               name = smb_fname->base_name;
        }
 
        DEBUG(10, ("get_nt_acl_internal: name=%s\n", name));
 
-       /* Get the full underlying sd for the hash
-          or to return as backup. */
-       if (fsp) {
-               status = SMB_VFS_NEXT_FGET_NT_ACL(handle,
-                               fsp,
-                               HASH_SECURITY_INFO,
-                               &pdesc_next);
-       } else {
-               status = SMB_VFS_NEXT_GET_NT_ACL(handle,
-                               name,
-                               HASH_SECURITY_INFO,
-                               &pdesc_next);
-       }
-
+       status = get_acl_blob(frame, handle, fsp, name, &blob);
        if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10, ("get_nt_acl_internal: get_next_acl for file %s "
-                       "returned %s\n",
-                       name,
+               DEBUG(10, ("get_nt_acl_internal: get_acl_blob returned %s\n",
                        nt_errstr(status)));
-               return status;
+               psd = NULL;
+               goto out;
+       } else {
+               status = parse_acl_blob(&blob, mem_ctx, &psd,
+                                       &hash_type, &xattr_version, &hash[0], &sys_acl_hash[0]);
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(10, ("parse_acl_blob returned %s\n",
+                                  nt_errstr(status)));
+                       psd = NULL;
+                       goto out;
+               }
        }
 
-       status = get_acl_blob(talloc_tos(), handle, fsp, name, &blob);
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10, ("get_nt_acl_internal: get_acl_blob returned %s\n",
-                       nt_errstr(status)));
-               psd = pdesc_next;
+       /* Ensure we don't leak psd if we don't choose it.
+        *
+        * We don't allocate it onto frame as it is preferred not to
+        * steal from a talloc pool.
+        */
+       talloc_steal(frame, psd);
+
+       /* determine which type of xattr we got */
+       switch (xattr_version) {
+       case 1:
+       case 2:
+               /* These xattr types are unilatteral, they do not
+                * require confirmation of the hash.  In particular,
+                * the NTVFS file server uses version 1, but
+                * 'samba-tool ntacl' can set these as well */
+               goto out;
+       case 3:
+       case 4:
+               if (ignore_file_system_acl) {
+                       goto out;
+               }
+
+               break;
+       default:
+               DEBUG(10, ("get_nt_acl_internal: ACL blob revision "
+                          "mismatch (%u) for file %s\n",
+                          (unsigned int)hash_type,
+                          name));
+               TALLOC_FREE(psd);
+               psd = NULL;
                goto out;
        }
 
-       status = parse_acl_blob(&blob, &psd,
-                               &hash_type, &hash[0]);
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10, ("parse_acl_blob returned %s\n",
-                               nt_errstr(status)));
-               psd = pdesc_next;
+       /* determine which type of xattr we got */
+       if (hash_type != XATTR_SD_HASH_TYPE_SHA256) {
+               DEBUG(10, ("get_nt_acl_internal: ACL blob hash type "
+                          "(%u) unexpected for file %s\n",
+                          (unsigned int)hash_type,
+                          name));
+               TALLOC_FREE(psd);
+               psd = NULL;
                goto out;
        }
 
-       /* Ensure the hash type is one we know. */
-       switch (hash_type) {
-               case XATTR_SD_HASH_TYPE_NONE:
-                       /* No hash, just return blob sd. */
-                       goto out;
-               case XATTR_SD_HASH_TYPE_SHA256:
-                       break;
-               default:
-                       DEBUG(10, ("get_nt_acl_internal: ACL blob revision "
-                               "mismatch (%u) for file %s\n",
-                               (unsigned int)hash_type,
-                               name));
+       /* determine which type of xattr we got */
+       switch (xattr_version) {
+       case 4:
+       {
+               int ret;
+               char *sys_acl_blob_description;
+               DATA_BLOB sys_acl_blob;
+               if (fsp) {
+                       /* Get the full underlying sd, then hash. */
+                       ret = SMB_VFS_NEXT_SYS_ACL_BLOB_GET_FD(handle,
+                                                              fsp,
+                                                              frame,
+                                                              &sys_acl_blob_description,
+                                                              &sys_acl_blob);
+               } else {
+                       /* Get the full underlying sd, then hash. */
+                       ret = SMB_VFS_NEXT_SYS_ACL_BLOB_GET_FILE(handle,
+                                                                name,
+                                                                frame,
+                                                                &sys_acl_blob_description,
+                                                                &sys_acl_blob);
+               }
+
+               /* If we fail to get the ACL blob (for some reason) then this
+                * is not fatal, we just work based on the NT ACL only */
+               if (ret == 0) {
+                       status = hash_blob_sha256(sys_acl_blob, sys_acl_hash_tmp);
+                       if (!NT_STATUS_IS_OK(status)) {
+                               TALLOC_FREE(frame);
+                               return status;
+                       }
+
+                       if (memcmp(&sys_acl_hash[0], &sys_acl_hash_tmp[0], 
+                                  XATTR_SD_HASH_SIZE) == 0) {
+                               /* Hash matches, return blob sd. */
+                               DEBUG(10, ("get_nt_acl_internal: blob hash "
+                                          "matches for file %s\n",
+                                          name ));
+                               goto out;
+                       }
+               }
+
+               /* Otherwise, fall though and see if the NT ACL hash matches */
+       }
+       case 3:
+               /* Get the full underlying sd for the hash
+                  or to return as backup. */
+               if (fsp) {
+                       status = SMB_VFS_NEXT_FGET_NT_ACL(handle,
+                                                         fsp,
+                                                         HASH_SECURITY_INFO,
+                                                         mem_ctx,
+                                                         &pdesc_next);
+               } else {
+                       status = SMB_VFS_NEXT_GET_NT_ACL(handle,
+                                                        smb_fname,
+                                                        HASH_SECURITY_INFO,
+                                                        mem_ctx,
+                                                        &pdesc_next);
+               }
+
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(10, ("get_nt_acl_internal: get_next_acl for file %s "
+                                  "returned %s\n",
+                                  name,
+                                  nt_errstr(status)));
+                       TALLOC_FREE(frame);
+                       return status;
+               }
+
+               /* Ensure we don't leak psd_next if we don't choose it.
+                *
+                * We don't allocate it onto frame as it is preferred not to
+                * steal from a talloc pool.
+                */
+               talloc_steal(frame, pdesc_next);
+
+               status = hash_sd_sha256(pdesc_next, hash_tmp);
+               if (!NT_STATUS_IS_OK(status)) {
                        TALLOC_FREE(psd);
                        psd = pdesc_next;
                        goto out;
-       }
+               }
 
-       if (ignore_file_system_acl) {
-               goto out;
-       }
+               if (memcmp(&hash[0], &hash_tmp[0], XATTR_SD_HASH_SIZE) == 0) {
+                       /* Hash matches, return blob sd. */
+                       DEBUG(10, ("get_nt_acl_internal: blob hash "
+                                  "matches for file %s\n",
+                                  name ));
+                       goto out;
+               }
+
+               /* Hash doesn't match, return underlying sd. */
+               DEBUG(10, ("get_nt_acl_internal: blob hash "
+                          "does not match for file %s - returning "
+                          "file system SD mapping.\n",
+                          name ));
+
+               if (DEBUGLEVEL >= 10) {
+                       DEBUG(10,("get_nt_acl_internal: acl for blob hash for %s is:\n",
+                                 name ));
+                       NDR_PRINT_DEBUG(security_descriptor, pdesc_next);
+               }
 
-       status = hash_sd_sha256(pdesc_next, hash_tmp);
-       if (!NT_STATUS_IS_OK(status)) {
                TALLOC_FREE(psd);
                psd = pdesc_next;
-               goto out;
        }
+  out:
 
-       if (memcmp(&hash[0], &hash_tmp[0], XATTR_SD_HASH_SIZE) == 0) {
-               /* Hash matches, return blob sd. */
-               DEBUG(10, ("get_nt_acl_internal: blob hash "
-                       "matches for file %s\n",
-                       name ));
-               goto out;
-       }
+       if (psd == NULL) {
+               /* Get the full underlying sd, as we failed to get the
+                * blob for the hash, or the revision/hash type wasn't
+                * known */
+               if (fsp) {
+                       status = SMB_VFS_NEXT_FGET_NT_ACL(handle,
+                                                         fsp,
+                                                         security_info,
+                                                         mem_ctx,
+                                                         &pdesc_next);
+               } else {
+                       status = SMB_VFS_NEXT_GET_NT_ACL(handle,
+                                                        smb_fname,
+                                                        security_info,
+                                                        mem_ctx,
+                                                        &pdesc_next);
+               }
 
-       /* Hash doesn't match, return underlying sd. */
-       TALLOC_FREE(psd);
-       psd = pdesc_next;
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(10, ("get_nt_acl_internal: get_next_acl for file %s "
+                                  "returned %s\n",
+                                  name,
+                                  nt_errstr(status)));
+                       TALLOC_FREE(frame);
+                       return status;
+               }
 
-  out:
+               /* Ensure we don't leak psd_next if we don't choose it.
+                *
+                * We don't allocate it onto frame as it is preferred not to
+                * steal from a talloc pool.
+                */
+               talloc_steal(frame, pdesc_next);
+               psd = pdesc_next;
+       }
 
        if (psd != pdesc_next) {
                /* We're returning the blob, throw
@@ -367,36 +618,61 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
                if (fsp) {
                        status = vfs_stat_fsp(fsp);
                        if (!NT_STATUS_IS_OK(status)) {
+                               TALLOC_FREE(frame);
                                return status;
                        }
                        psbuf = &fsp->fsp_name->st;
                } else {
-                       int ret = vfs_stat_smb_fname(handle->conn,
+                       /*
+                        * https://bugzilla.samba.org/show_bug.cgi?id=11249
+                        *
+                        * We are currently guaranteed that 'name' here is
+                        * a smb_fname->base_name, which *cannot* contain
+                        * a stream name (':'). vfs_stat_smb_fname() splits
+                        * a name into a base name + stream name, which
+                        * when we get here we know we've already done.
+                        * So we have to call the stat or lstat VFS
+                        * calls directly here. Else, a base_name that
+                        * contains a ':' (from a demangled name) will
+                        * get split again.
+                        *
+                        * FIXME.
+                        * This uglyness will go away once smb_fname
+                        * is fully plumbed through the VFS.
+                        */
+                       int ret = vfs_stat_smb_basename(handle->conn,
                                                name,
                                                &sbuf);
                        if (ret == -1) {
+                               TALLOC_FREE(frame);
                                return map_nt_error_from_unix(errno);
                        }
                }
-               is_directory = S_ISDIR(sbuf.st_ex_mode);
+               is_directory = S_ISDIR(psbuf->st_ex_mode);
 
                if (ignore_file_system_acl) {
                        TALLOC_FREE(pdesc_next);
-                       status = make_default_filesystem_acl(talloc_tos(),
+                       status = make_default_filesystem_acl(mem_ctx,
                                                name,
                                                psbuf,
                                                &psd);
                        if (!NT_STATUS_IS_OK(status)) {
+                               TALLOC_FREE(frame);
                                return status;
                        }
                } else {
                        if (is_directory &&
                                !sd_has_inheritable_components(psd,
                                                        true)) {
-                               add_directory_inheritable_components(handle,
+                               status = add_directory_inheritable_components(
+                                                       handle,
                                                        name,
                                                        psbuf,
                                                        psd);
+                               if (!NT_STATUS_IS_OK(status)) {
+                                       TALLOC_FREE(frame);
+                                       return status;
+                               }
                        }
                        /* The underlying POSIX module always sets
                           the ~SEC_DESC_DACL_PROTECTED bit, as ACLs
@@ -413,14 +689,15 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
                psd->group_sid = NULL;
        }
        if (!(security_info & SECINFO_DACL)) {
+               psd->type &= ~SEC_DESC_DACL_PRESENT;
                psd->dacl = NULL;
        }
        if (!(security_info & SECINFO_SACL)) {
+               psd->type &= ~SEC_DESC_SACL_PRESENT;
                psd->sacl = NULL;
        }
 
        TALLOC_FREE(blob.data);
-       *ppdesc = psd;
 
        if (DEBUGLEVEL >= 10) {
                DEBUG(10,("get_nt_acl_internal: returning acl for %s is:\n",
@@ -428,277 +705,25 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
                NDR_PRINT_DEBUG(security_descriptor, psd);
        }
 
-       return NT_STATUS_OK;
-}
-
-/*********************************************************************
- Create a default ACL by inheriting from the parent. If no inheritance
- from the parent available, don't set anything. This will leave the actual
- permissions the new file or directory already got from the filesystem
- as the NT ACL when read.
-*********************************************************************/
-
-static NTSTATUS inherit_new_acl(vfs_handle_struct *handle,
-                                       files_struct *fsp,
-                                       struct security_descriptor *parent_desc,
-                                       bool is_directory)
-{
-       TALLOC_CTX *ctx = talloc_tos();
-       NTSTATUS status = NT_STATUS_OK;
-       struct security_descriptor *psd = NULL;
-       struct dom_sid *owner_sid = NULL;
-       struct dom_sid *group_sid = NULL;
-       bool inherit_owner = lp_inherit_owner(SNUM(handle->conn));
-       size_t size;
-
-       if (!sd_has_inheritable_components(parent_desc, is_directory)) {
-               return NT_STATUS_OK;
-       }
-
-       /* Create an inherited descriptor from the parent. */
-
-       if (DEBUGLEVEL >= 10) {
-               DEBUG(10,("inherit_new_acl: parent acl for %s is:\n",
-                       fsp_str_dbg(fsp) ));
-               NDR_PRINT_DEBUG(security_descriptor, parent_desc);
-       }
-
-       /* Inherit from parent descriptor if "inherit owner" set. */
-       if (inherit_owner) {
-               owner_sid = parent_desc->owner_sid;
-               group_sid = parent_desc->group_sid;
-       }
+       /* The VFS API is that the ACL is expected to be on mem_ctx */
+       *ppdesc = talloc_move(mem_ctx, &psd);
 
-       if (owner_sid == NULL) {
-               owner_sid = &handle->conn->session_info->security_token->sids[PRIMARY_USER_SID_INDEX];
-       }
-       if (group_sid == NULL) {
-               group_sid = &handle->conn->session_info->security_token->sids[PRIMARY_GROUP_SID_INDEX];
-       }
-
-       status = se_create_child_secdesc(ctx,
-                       &psd,
-                       &size,
-                       parent_desc,
-                       owner_sid,
-                       group_sid,
-                       is_directory);
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
-       }
-
-       if (DEBUGLEVEL >= 10) {
-               DEBUG(10,("inherit_new_acl: child acl for %s is:\n",
-                       fsp_str_dbg(fsp) ));
-               NDR_PRINT_DEBUG(security_descriptor, psd);
-       }
-
-       if (inherit_owner) {
-               /* We need to be root to force this. */
-               become_root();
-       }
-       status = SMB_VFS_FSET_NT_ACL(fsp,
-                               (SECINFO_OWNER |
-                                SECINFO_GROUP |
-                                SECINFO_DACL),
-                               psd);
-       if (inherit_owner) {
-               unbecome_root();
-       }
-       return status;
-}
-
-static NTSTATUS get_parent_acl_common(vfs_handle_struct *handle,
-                               const char *path,
-                               struct security_descriptor **pp_parent_desc)
-{
-       char *parent_name = NULL;
-       NTSTATUS status;
-
-       if (!parent_dirname(talloc_tos(), path, &parent_name, NULL)) {
-               return NT_STATUS_NO_MEMORY;
-       }
-
-       status = get_nt_acl_internal(handle,
-                                       NULL,
-                                       parent_name,
-                                       (SECINFO_OWNER |
-                                        SECINFO_GROUP |
-                                        SECINFO_DACL),
-                                       pp_parent_desc);
-
-       if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10,("get_parent_acl_common: get_nt_acl_internal "
-                       "on directory %s for "
-                       "path %s returned %s\n",
-                       parent_name,
-                       path,
-                       nt_errstr(status) ));
-       }
-       return status;
-}
-
-static NTSTATUS check_parent_acl_common(vfs_handle_struct *handle,
-                               const char *path,
-                               uint32_t access_mask,
-                               struct security_descriptor **pp_parent_desc)
-{
-       char *parent_name = NULL;
-       struct security_descriptor *parent_desc = NULL;
-       uint32_t access_granted = 0;
-       NTSTATUS status;
-
-       status = get_parent_acl_common(handle, path, &parent_desc);
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
-       }
-       if (pp_parent_desc) {
-               *pp_parent_desc = parent_desc;
-       }
-       status = smb1_file_se_access_check(handle->conn,
-                                       parent_desc,
-                                       get_current_nttok(handle->conn),
-                                       access_mask,
-                                       &access_granted);
-       if(!NT_STATUS_IS_OK(status)) {
-               DEBUG(10,("check_parent_acl_common: access check "
-                       "on directory %s for "
-                       "path %s for mask 0x%x returned %s\n",
-                       parent_name,
-                       path,
-                       access_mask,
-                       nt_errstr(status) ));
-               return status;
-       }
+       TALLOC_FREE(frame);
        return NT_STATUS_OK;
 }
 
-/*********************************************************************
- Check ACL on open. For new files inherit from parent directory.
-*********************************************************************/
-
-static int open_acl_common(vfs_handle_struct *handle,
-                       struct smb_filename *smb_fname,
-                       files_struct *fsp,
-                       int flags,
-                       mode_t mode)
-{
-       uint32_t access_granted = 0;
-       struct security_descriptor *pdesc = NULL;
-       bool file_existed = true;
-       char *fname = NULL;
-       NTSTATUS status;
-
-       if (fsp->base_fsp) {
-               /* Stream open. Base filename open already did the ACL check. */
-               DEBUG(10,("open_acl_common: stream open on %s\n",
-                       fsp_str_dbg(fsp) ));
-               return SMB_VFS_NEXT_OPEN(handle, smb_fname, fsp, flags, mode);
-       }
-
-       status = get_full_smb_filename(talloc_tos(), smb_fname,
-                                      &fname);
-       if (!NT_STATUS_IS_OK(status)) {
-               goto err;
-       }
-
-       status = get_nt_acl_internal(handle,
-                               NULL,
-                               fname,
-                               (SECINFO_OWNER |
-                                SECINFO_GROUP |
-                                SECINFO_DACL),
-                               &pdesc);
-        if (NT_STATUS_IS_OK(status)) {
-               /* See if we can access it. */
-               status = smb1_file_se_access_check(handle->conn,
-                                       pdesc,
-                                       get_current_nttok(handle->conn),
-                                       fsp->access_mask,
-                                       &access_granted);
-               if (!NT_STATUS_IS_OK(status)) {
-                       DEBUG(10,("open_acl_xattr: %s open "
-                               "refused with error %s\n",
-                               fsp_str_dbg(fsp),
-                               nt_errstr(status) ));
-                       goto err;
-               }
-        } else if (NT_STATUS_EQUAL(status,NT_STATUS_OBJECT_NAME_NOT_FOUND)) {
-               file_existed = false;
-               /*
-                * If O_CREAT is true then we're trying to create a file.
-                * Check the parent directory ACL will allow this.
-                */
-               if (flags & O_CREAT) {
-                       struct security_descriptor *parent_desc = NULL;
-                       struct security_descriptor **pp_psd = NULL;
-
-                       status = check_parent_acl_common(handle, fname,
-                                       SEC_DIR_ADD_FILE, &parent_desc);
-                       if (!NT_STATUS_IS_OK(status)) {
-                               goto err;
-                       }
-
-                       /* Cache the parent security descriptor for
-                        * later use. */
-
-                       pp_psd = VFS_ADD_FSP_EXTENSION(handle,
-                                       fsp,
-                                       struct security_descriptor *,
-                                       NULL);
-                       if (!pp_psd) {
-                               status = NT_STATUS_NO_MEMORY;
-                               goto err;
-                       }
-
-                       *pp_psd = parent_desc;
-                       status = NT_STATUS_OK;
-               }
-       }
-
-       DEBUG(10,("open_acl_xattr: get_nt_acl_attr_internal for "
-               "%s returned %s\n",
-               fsp_str_dbg(fsp),
-               nt_errstr(status) ));
-
-       fsp->fh->fd = SMB_VFS_NEXT_OPEN(handle, smb_fname, fsp, flags, mode);
-       return fsp->fh->fd;
-
-  err:
-
-       errno = map_errno_from_nt_status(status);
-       return -1;
-}
-
-static int mkdir_acl_common(vfs_handle_struct *handle, const char *path, mode_t mode)
-{
-       int ret;
-       NTSTATUS status;
-       SMB_STRUCT_STAT sbuf;
-
-       ret = vfs_stat_smb_fname(handle->conn, path, &sbuf);
-       if (ret == -1 && errno == ENOENT) {
-               /* We're creating a new directory. */
-               status = check_parent_acl_common(handle, path,
-                               SEC_DIR_ADD_SUBDIR, NULL);
-               if (!NT_STATUS_IS_OK(status)) {
-                       errno = map_errno_from_nt_status(status);
-                       return -1;
-               }
-       }
-
-       return SMB_VFS_NEXT_MKDIR(handle, path, mode);
-}
-
 /*********************************************************************
  Fetch a security descriptor given an fsp.
 *********************************************************************/
 
-static NTSTATUS fget_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
-        uint32_t security_info, struct security_descriptor **ppdesc)
+static NTSTATUS fget_nt_acl_common(vfs_handle_struct *handle,
+                                  files_struct *fsp,
+                                  uint32_t security_info,
+                                  TALLOC_CTX *mem_ctx,
+                                  struct security_descriptor **ppdesc)
 {
        return get_nt_acl_internal(handle, fsp,
-                               NULL, security_info, ppdesc);
+                                  NULL, security_info, mem_ctx, ppdesc);
 }
 
 /*********************************************************************
@@ -706,10 +731,17 @@ static NTSTATUS fget_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
 *********************************************************************/
 
 static NTSTATUS get_nt_acl_common(vfs_handle_struct *handle,
-        const char *name, uint32_t security_info, struct security_descriptor **ppdesc)
+                                 const struct smb_filename *smb_fname,
+                                 uint32_t security_info,
+                                 TALLOC_CTX *mem_ctx,
+                                 struct security_descriptor **ppdesc)
 {
-       return get_nt_acl_internal(handle, NULL,
-                               name, security_info, ppdesc);
+       return get_nt_acl_internal(handle,
+                               NULL,
+                               smb_fname,
+                               security_info,
+                               mem_ctx,
+                               ppdesc);
 }
 
 /*********************************************************************
@@ -720,24 +752,31 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
         uint32_t security_info_sent, const struct security_descriptor *orig_psd)
 {
        NTSTATUS status;
-       DATA_BLOB blob;
+       int ret;
+       DATA_BLOB blob, sys_acl_blob;
        struct security_descriptor *pdesc_next = NULL;
        struct security_descriptor *psd = NULL;
        uint8_t hash[XATTR_SD_HASH_SIZE];
+       uint8_t sys_acl_hash[XATTR_SD_HASH_SIZE];
+       bool chown_needed = false;
+       char *sys_acl_description;
+       TALLOC_CTX *frame = talloc_stackframe();
 
        if (DEBUGLEVEL >= 10) {
                DEBUG(10,("fset_nt_acl_xattr: incoming sd for file %s\n",
                          fsp_str_dbg(fsp)));
                NDR_PRINT_DEBUG(security_descriptor,
-                       CONST_DISCARD(struct security_descriptor *,orig_psd));
+                       discard_const_p(struct security_descriptor, orig_psd));
        }
 
        status = get_nt_acl_internal(handle, fsp,
                        NULL,
                        SECINFO_OWNER|SECINFO_GROUP|SECINFO_DACL|SECINFO_SACL,
+                                    frame,
                        &psd);
 
        if (!NT_STATUS_IS_OK(status)) {
+               TALLOC_FREE(frame);
                return status;
        }
 
@@ -746,12 +785,29 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
        psd->type = orig_psd->type | SEC_DESC_SELF_RELATIVE;
 
        if ((security_info_sent & SECINFO_OWNER) && (orig_psd->owner_sid != NULL)) {
+               if (!dom_sid_equal(orig_psd->owner_sid, psd->owner_sid)) {
+                       /* We're changing the owner. */
+                       chown_needed = true;
+               }
                psd->owner_sid = orig_psd->owner_sid;
        }
        if ((security_info_sent & SECINFO_GROUP) && (orig_psd->group_sid != NULL)) {
+               if (!dom_sid_equal(orig_psd->group_sid, psd->group_sid)) {
+                       /* We're changing the group. */
+                       chown_needed = true;
+               }
                psd->group_sid = orig_psd->group_sid;
        }
        if (security_info_sent & SECINFO_DACL) {
+               if (security_descriptor_with_ms_nfs(orig_psd)) {
+                       /*
+                        * If the sd contains a MS NFS SID, do
+                        * nothing, it's a chmod() request from OS X
+                        * with AAPL context.
+                        */
+                       TALLOC_FREE(frame);
+                       return NT_STATUS_OK;
+               }
                psd->dacl = orig_psd->dacl;
                psd->type |= SEC_DESC_DACL_PRESENT;
        }
@@ -762,47 +818,122 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
 
        status = SMB_VFS_NEXT_FSET_NT_ACL(handle, fsp, security_info_sent, psd);
        if (!NT_STATUS_IS_OK(status)) {
-               return status;
+               if (!NT_STATUS_EQUAL(status, NT_STATUS_ACCESS_DENIED)) {
+                       TALLOC_FREE(frame);
+                       return status;
+               }
+               /* We got access denied here. If we're already root,
+                  or we didn't need to do a chown, or the fsp isn't
+                  open with WRITE_OWNER access, just return. */
+               if (get_current_uid(handle->conn) == 0 ||
+                               chown_needed == false ||
+                               !(fsp->access_mask & SEC_STD_WRITE_OWNER)) {
+                       TALLOC_FREE(frame);
+                       return NT_STATUS_ACCESS_DENIED;
+               }
+
+               DEBUG(10,("fset_nt_acl_common: overriding chown on file %s "
+                       "for sid %s\n",
+                       fsp_str_dbg(fsp),
+                       sid_string_tos(psd->owner_sid)
+                       ));
+
+               /* Ok, we failed to chown and we have
+                  SEC_STD_WRITE_OWNER access - override. */
+               become_root();
+               status = SMB_VFS_NEXT_FSET_NT_ACL(handle, fsp,
+                               security_info_sent, psd);
+               unbecome_root();
+               if (!NT_STATUS_IS_OK(status)) {
+                       TALLOC_FREE(frame);
+                       return status;
+               }
        }
 
        /* Get the full underlying sd, then hash. */
        status = SMB_VFS_NEXT_FGET_NT_ACL(handle,
-                               fsp,
-                               HASH_SECURITY_INFO,
-                               &pdesc_next);
+                                         fsp,
+                                         HASH_SECURITY_INFO,
+                                         frame,
+                                         &pdesc_next);
 
        if (!NT_STATUS_IS_OK(status)) {
+               TALLOC_FREE(frame);
                return status;
        }
 
        status = hash_sd_sha256(pdesc_next, hash);
        if (!NT_STATUS_IS_OK(status)) {
+               TALLOC_FREE(frame);
+               return status;
+       }
+
+       /* Get the full underlying sd, then hash. */
+       ret = SMB_VFS_NEXT_SYS_ACL_BLOB_GET_FD(handle,
+                                              fsp,
+                                              frame,
+                                              &sys_acl_description,
+                                              &sys_acl_blob);
+
+       /* If we fail to get the ACL blob (for some reason) then this
+        * is not fatal, we just work based on the NT ACL only */
+       if (ret != 0) {
+               if (DEBUGLEVEL >= 10) {
+                       DEBUG(10,("fset_nt_acl_xattr: storing xattr sd for file %s\n",
+                                 fsp_str_dbg(fsp)));
+                       NDR_PRINT_DEBUG(security_descriptor,
+                                       discard_const_p(struct security_descriptor, psd));
+
+                       DEBUG(10,("fset_nt_acl_xattr: storing has in xattr sd based on \n"));
+                       NDR_PRINT_DEBUG(security_descriptor,
+                                       discard_const_p(struct security_descriptor, pdesc_next));
+               }
+               status = create_acl_blob(psd, &blob, XATTR_SD_HASH_TYPE_SHA256, hash);
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(10, ("fset_nt_acl_xattr: create_acl_blob failed\n"));
+                       TALLOC_FREE(frame);
+                       return status;
+               }
+
+               status = store_acl_blob_fsp(handle, fsp, &blob);
+
+               TALLOC_FREE(frame);
+               return status;
+       }
+
+       status = hash_blob_sha256(sys_acl_blob, sys_acl_hash);
+       if (!NT_STATUS_IS_OK(status)) {
+               TALLOC_FREE(frame);
                return status;
        }
 
        if (DEBUGLEVEL >= 10) {
-               DEBUG(10,("fset_nt_acl_xattr: storing xattr sd for file %s\n",
+               DEBUG(10,("fset_nt_acl_xattr: storing xattr sd for file %s based on system ACL\n",
                          fsp_str_dbg(fsp)));
                NDR_PRINT_DEBUG(security_descriptor,
-                       CONST_DISCARD(struct security_descriptor *,psd));
-       }
-       create_acl_blob(psd, &blob, XATTR_SD_HASH_TYPE_SHA256, hash);
-       store_acl_blob_fsp(handle, fsp, &blob);
-
-       return NT_STATUS_OK;
-}
+                               discard_const_p(struct security_descriptor, psd));
 
-static SMB_STRUCT_DIR *opendir_acl_common(vfs_handle_struct *handle,
-                       const char *fname, const char *mask, uint32 attr)
-{
-       NTSTATUS status = check_parent_acl_common(handle, fname,
-                                       SEC_DIR_LIST, NULL);
+               DEBUG(10,("fset_nt_acl_xattr: storing hash in xattr sd based on system ACL and:\n"));
+               NDR_PRINT_DEBUG(security_descriptor,
+                               discard_const_p(struct security_descriptor, pdesc_next));
+       }
 
+       /* We store hashes of both the sys ACL blob and the NT
+        * security desciptor mapped from that ACL so as to improve
+        * our chances against some inadvertant change breaking the
+        * hash used */
+       status = create_sys_acl_blob(psd, &blob, XATTR_SD_HASH_TYPE_SHA256, hash, 
+                                    sys_acl_description, sys_acl_hash);
        if (!NT_STATUS_IS_OK(status)) {
-               errno = map_errno_from_nt_status(status);
-               return NULL;
+               DEBUG(10, ("fset_nt_acl_xattr: create_sys_acl_blob failed\n"));
+               TALLOC_FREE(frame);
+               return status;
        }
-       return SMB_VFS_NEXT_OPENDIR(handle, fname, mask, attr);
+
+       status = store_acl_blob_fsp(handle, fsp, &blob);
+
+       TALLOC_FREE(frame);
+       return status;
 }
 
 static int acl_common_remove_object(vfs_handle_struct *handle,
@@ -817,6 +948,13 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
        const char *final_component = NULL;
        struct smb_filename local_fname;
        int saved_errno = 0;
+       char *saved_dir = NULL;
+
+       saved_dir = vfs_GetWd(talloc_tos(),conn);
+       if (!saved_dir) {
+               saved_errno = errno;
+               goto out;
+       }
 
        if (!parent_dirname(talloc_tos(), path,
                        &parent_dir, &final_component)) {
@@ -829,14 +967,14 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
                parent_dir, final_component ));
 
        /* cd into the parent dir to pin it. */
-       ret = SMB_VFS_CHDIR(conn, parent_dir);
+       ret = vfs_ChDir(conn, parent_dir);
        if (ret == -1) {
                saved_errno = errno;
                goto out;
        }
 
        ZERO_STRUCT(local_fname);
-       local_fname.base_name = CONST_DISCARD(char *,final_component);
+       local_fname.base_name = discard_const_p(char, final_component);
 
        /* Must use lstat here. */
        ret = SMB_VFS_LSTAT(conn, &local_fname);
@@ -848,7 +986,7 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
        /* Ensure we have this file open with DELETE access. */
        id = vfs_file_id_from_sbuf(conn, &local_fname.st);
        for (fsp = file_find_di_first(conn->sconn, id); fsp;
-            file_find_di_next(fsp)) {
+                    fsp = file_find_di_next(fsp)) {
                if (fsp->access_mask & DELETE_ACCESS &&
                                fsp->delete_on_close) {
                        /* We did open this for delete,
@@ -883,7 +1021,9 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
 
        TALLOC_FREE(parent_dir);
 
-       vfs_ChDir(conn, conn->connectpath);
+       if (saved_dir) {
+               vfs_ChDir(conn, saved_dir);
+       }
        if (saved_errno) {
                errno = saved_errno;
        }
@@ -895,129 +1035,23 @@ static int rmdir_acl_common(struct vfs_handle_struct *handle,
 {
        int ret;
 
+       /* Try the normal rmdir first. */
        ret = SMB_VFS_NEXT_RMDIR(handle, path);
-       if (!(ret == -1 && (errno == EACCES || errno == EPERM))) {
-               DEBUG(10,("rmdir_acl_common: unlink of %s failed %s\n",
-                       path,
-                       strerror(errno) ));
-               return ret;
-       }
-
-       return acl_common_remove_object(handle,
-                                       path,
-                                       true);
-}
-
-static NTSTATUS create_file_acl_common(struct vfs_handle_struct *handle,
-                               struct smb_request *req,
-                               uint16_t root_dir_fid,
-                               struct smb_filename *smb_fname,
-                               uint32_t access_mask,
-                               uint32_t share_access,
-                               uint32_t create_disposition,
-                               uint32_t create_options,
-                               uint32_t file_attributes,
-                               uint32_t oplock_request,
-                               uint64_t allocation_size,
-                               uint32_t private_flags,
-                               struct security_descriptor *sd,
-                               struct ea_list *ea_list,
-                               files_struct **result,
-                               int *pinfo)
-{
-       NTSTATUS status, status1;
-       files_struct *fsp = NULL;
-       int info;
-       struct security_descriptor *parent_sd = NULL;
-       struct security_descriptor **pp_parent_sd = NULL;
-
-       status = SMB_VFS_NEXT_CREATE_FILE(handle,
-                                       req,
-                                       root_dir_fid,
-                                       smb_fname,
-                                       access_mask,
-                                       share_access,
-                                       create_disposition,
-                                       create_options,
-                                       file_attributes,
-                                       oplock_request,
-                                       allocation_size,
-                                       private_flags,
-                                       sd,
-                                       ea_list,
-                                       result,
-                                       &info);
-
-       if (!NT_STATUS_IS_OK(status)) {
-               goto out;
-       }
-
-       if (info != FILE_WAS_CREATED) {
-               /* File/directory was opened, not created. */
-               goto out;
-       }
-
-       fsp = *result;
-
-       if (fsp == NULL) {
-               /* Only handle success. */
-               goto out;
-       }
-
-       if (sd) {
-               /* Security descriptor already set. */
-               goto out;
-       }
-
-       if (fsp->base_fsp) {
-               /* Stream open. */
-               goto out;
-       }
-
-       /* See if we have a cached parent sd, if so, use it. */
-       pp_parent_sd = (struct security_descriptor **)VFS_FETCH_FSP_EXTENSION(handle, fsp);
-       if (!pp_parent_sd) {
-               /* Must be a directory, fetch again (sigh). */
-               status = get_parent_acl_common(handle,
-                               fsp->fsp_name->base_name,
-                               &parent_sd);
-               if (!NT_STATUS_IS_OK(status)) {
-                       goto out;
-               }
-       } else {
-               parent_sd = *pp_parent_sd;
+       if (ret == 0) {
+               return 0;
        }
-
-       if (!parent_sd) {
-               goto err;
-       }
-
-       /* New directory - inherit from parent. */
-       status1 = inherit_new_acl(handle, fsp, parent_sd, fsp->is_directory);
-
-       if (!NT_STATUS_IS_OK(status1)) {
-               DEBUG(1,("create_file_acl_common: error setting "
-                       "sd for %s (%s)\n",
-                       fsp_str_dbg(fsp),
-                       nt_errstr(status1) ));
+       if (errno == EACCES || errno == EPERM) {
+               /* Failed due to access denied,
+                  see if we need to root override. */
+               return acl_common_remove_object(handle,
+                                               path,
+                                               true);
        }
 
-  out:
-
-       if (fsp) {
-               VFS_REMOVE_FSP_EXTENSION(handle, fsp);
-       }
-
-       if (NT_STATUS_IS_OK(status) && pinfo) {
-               *pinfo = info;
-       }
-       return status;
-
-  err:
-
-       smb_panic("create_file_acl_common: logic error.\n");
-       /* NOTREACHED */
-       return status;
+       DEBUG(10,("rmdir_acl_common: unlink of %s failed %s\n",
+               path,
+               strerror(errno) ));
+       return -1;
 }
 
 static int unlink_acl_common(struct vfs_handle_struct *handle,
@@ -1025,21 +1059,28 @@ static int unlink_acl_common(struct vfs_handle_struct *handle,
 {
        int ret;
 
+       /* Try the normal unlink first. */
        ret = SMB_VFS_NEXT_UNLINK(handle, smb_fname);
-       if (!(ret == -1 && (errno == EACCES || errno == EPERM))) {
-               DEBUG(10,("unlink_acl_common: unlink of %s failed %s\n",
-                       smb_fname->base_name,
-                       strerror(errno) ));
-               return ret;
-       }
-       /* Don't do anything fancy for streams. */
-       if (smb_fname->stream_name) {
-               return ret;
+       if (ret == 0) {
+               return 0;
        }
+       if (errno == EACCES || errno == EPERM) {
+               /* Failed due to access denied,
+                  see if we need to root override. */
 
-       return acl_common_remove_object(handle,
+               /* Don't do anything fancy for streams. */
+               if (smb_fname->stream_name) {
+                       return -1;
+               }
+               return acl_common_remove_object(handle,
                                        smb_fname->base_name,
                                        false);
+       }
+
+       DEBUG(10,("unlink_acl_common: unlink of %s failed %s\n",
+               smb_fname->base_name,
+               strerror(errno) ));
+       return -1;
 }
 
 static int chmod_acl_module_common(struct vfs_handle_struct *handle,
@@ -1055,7 +1096,7 @@ static int chmod_acl_module_common(struct vfs_handle_struct *handle,
 static int fchmod_acl_module_common(struct vfs_handle_struct *handle,
                        struct files_struct *fsp, mode_t mode)
 {
-       if (fsp->posix_open) {
+       if (fsp->posix_flags & FSP_POSIX_FLAGS_OPEN) {
                /* Only allow this on POSIX opens. */
                return SMB_VFS_NEXT_FCHMOD(handle, fsp, mode);
        }
@@ -1075,7 +1116,7 @@ static int chmod_acl_acl_module_common(struct vfs_handle_struct *handle,
 static int fchmod_acl_acl_module_common(struct vfs_handle_struct *handle,
                        struct files_struct *fsp, mode_t mode)
 {
-       if (fsp->posix_open) {
+       if (fsp->posix_flags & FSP_POSIX_FLAGS_OPEN) {
                /* Only allow this on POSIX opens. */
                return SMB_VFS_NEXT_FCHMOD_ACL(handle, fsp, mode);
        }