s3-auth Rename NT_USER_TOKEN user_sids -> sids
[idra/samba.git] / source3 / modules / vfs_acl_common.c
index a12f1057616e932e37fd9d2eebce6996d3417eb6..e759dc2df3f3b1e82d9b19229fe426c7733377b9 100644 (file)
@@ -19,6 +19,8 @@
  * along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
+#include "../librpc/gen_ndr/ndr_security.h"
+
 static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
                        DATA_BLOB *pblob,
                        uint16_t hash_type,
@@ -34,14 +36,10 @@ static NTSTATUS store_acl_blob_fsp(vfs_handle_struct *handle,
                        files_struct *fsp,
                        DATA_BLOB *pblob);
 
-static NTSTATUS store_acl_blob_pathname(vfs_handle_struct *handle,
-                       const char *fname,
-                       DATA_BLOB *pblob);
-
-#define HASH_SECURITY_INFO (OWNER_SECURITY_INFORMATION | \
-                               GROUP_SECURITY_INFORMATION | \
-                               DACL_SECURITY_INFORMATION | \
-                               SACL_SECURITY_INFORMATION)
+#define HASH_SECURITY_INFO (SECINFO_OWNER | \
+                               SECINFO_GROUP | \
+                               SECINFO_DACL | \
+                               SECINFO_SACL)
 
 /*******************************************************************
  Hash a security descriptor.
@@ -81,7 +79,7 @@ static NTSTATUS parse_acl_blob(const DATA_BLOB *pblob,
        enum ndr_err_code ndr_err;
        size_t sd_size;
 
-       ndr_err = ndr_pull_struct_blob(pblob, ctx, NULL, &xacl,
+       ndr_err = ndr_pull_struct_blob(pblob, ctx, &xacl,
                        (ndr_pull_flags_fn_t)ndr_pull_xattr_NTACL);
 
        if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
@@ -92,7 +90,7 @@ static NTSTATUS parse_acl_blob(const DATA_BLOB *pblob,
 
        switch (xacl.version) {
                case 2:
-                       *ppdesc = make_sec_desc(ctx, SEC_DESC_REVISION,
+                       *ppdesc = make_sec_desc(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,
@@ -104,7 +102,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, SEC_DESC_REVISION,
+                       *ppdesc = make_sec_desc(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,
@@ -148,7 +146,7 @@ static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
        memcpy(&xacl.info.sd_hs3->hash[0], hash, XATTR_SD_HASH_SIZE);
 
        ndr_err = ndr_push_struct_blob(
-                       pblob, ctx, NULL, &xacl,
+                       pblob, ctx, &xacl,
                        (ndr_push_flags_fn_t)ndr_push_xattr_NTACL);
 
        if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
@@ -161,7 +159,88 @@ static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
 }
 
 /*******************************************************************
- Store a DATA_BLOB into an xattr given a pathname.
+ 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,
+                                const char *name,
+                               SMB_STRUCT_STAT *psbuf,
+                               struct security_descriptor *psd)
+{
+       struct connection_struct *conn = handle->conn;
+       int num_aces = (psd->dacl ? psd->dacl->num_aces : 0);
+       struct smb_filename smb_fname;
+       enum security_ace_type acltype;
+       uint32_t access_mask;
+       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);
+
+       if (new_ace_list == NULL) {
+               return;
+       }
+
+       /* Fake a quick smb_filename. */
+       ZERO_STRUCT(smb_fname);
+       smb_fname.st = *psbuf;
+       smb_fname.base_name = CONST_DISCARD(char *, name);
+
+       dir_mode = unix_mode(conn,
+                       FILE_ATTRIBUTE_DIRECTORY, &smb_fname, NULL);
+       file_mode = unix_mode(conn,
+                       FILE_ATTRIBUTE_ARCHIVE, &smb_fname, NULL);
+
+       mode = dir_mode | file_mode;
+
+       DEBUG(10, ("add_directory_inheritable_components: directory %s, "
+               "mode = 0%o\n",
+               name,
+               (unsigned int)mode ));
+
+       if (num_aces) {
+               memcpy(new_ace_list, psd->dacl->aces,
+                       num_aces * sizeof(struct security_ace));
+       }
+       access_mask = map_canon_ace_perms(SNUM(conn), &acltype,
+                               mode & 0700, false);
+
+       init_sec_ace(&new_ace_list[num_aces],
+                       &global_sid_Creator_Owner,
+                       acltype,
+                       access_mask,
+                       SEC_ACE_FLAG_CONTAINER_INHERIT|
+                               SEC_ACE_FLAG_OBJECT_INHERIT|
+                               SEC_ACE_FLAG_INHERIT_ONLY);
+       access_mask = map_canon_ace_perms(SNUM(conn), &acltype,
+                               (mode << 3) & 0700, false);
+       init_sec_ace(&new_ace_list[num_aces+1],
+                       &global_sid_Creator_Group,
+                       acltype,
+                       access_mask,
+                       SEC_ACE_FLAG_CONTAINER_INHERIT|
+                               SEC_ACE_FLAG_OBJECT_INHERIT|
+                               SEC_ACE_FLAG_INHERIT_ONLY);
+       access_mask = map_canon_ace_perms(SNUM(conn), &acltype,
+                               (mode << 6) & 0700, false);
+       init_sec_ace(&new_ace_list[num_aces+2],
+                       &global_sid_World,
+                       acltype,
+                       access_mask,
+                       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;
+}
+
+/*******************************************************************
+ Pull a DATA_BLOB from an xattr given a pathname.
+ If the hash doesn't match, or doesn't exist - return the underlying
+ filesystem sd.
 *******************************************************************/
 
 static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
@@ -172,9 +251,10 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
 {
        DATA_BLOB blob;
        NTSTATUS status;
-       uint16_t hash_type;
+       uint16_t hash_type = XATTR_SD_HASH_TYPE_NONE;
        uint8_t hash[XATTR_SD_HASH_SIZE];
        uint8_t hash_tmp[XATTR_SD_HASH_SIZE];
+       struct security_descriptor *psd = NULL;
        struct security_descriptor *pdesc_next = NULL;
 
        if (fsp && name == NULL) {
@@ -183,299 +263,240 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
 
        DEBUG(10, ("get_nt_acl_internal: name=%s\n", name));
 
-       status = get_acl_blob(talloc_tos(), handle, fsp, name, &blob);
+       /* 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);
+       }
+
        if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10, ("get_acl_blob returned %s\n", nt_errstr(status)));
+               DEBUG(10, ("get_nt_acl_internal: get_next_acl for file %s "
+                       "returned %s\n",
+                       name,
+                       nt_errstr(status)));
                return status;
        }
 
-       status = parse_acl_blob(&blob, ppdesc,
+       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;
+               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)));
-               return status;
+               psd = pdesc_next;
+               goto out;
        }
 
        /* Ensure the hash type is one we know. */
        switch (hash_type) {
                case XATTR_SD_HASH_TYPE_NONE:
-                       /* No hash, goto return blob sd. */
+                       /* No hash, just return blob sd. */
                        goto out;
                case XATTR_SD_HASH_TYPE_SHA256:
                        break;
                default:
-                       return NT_STATUS_REVISION_MISMATCH;
-       }
-
-       /* Get the full underlying sd, then hash. */
-       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);
+                       DEBUG(10, ("get_nt_acl_internal: ACL blob revision "
+                               "mismatch (%u) for file %s\n",
+                               (unsigned int)hash_type,
+                               name));
+                       TALLOC_FREE(psd);
+                       psd = pdesc_next;
+                       goto out;
        }
 
-       if (!NT_STATUS_IS_OK(status)) {
-               goto out;
-       }
 
        status = hash_sd_sha256(pdesc_next, hash_tmp);
        if (!NT_STATUS_IS_OK(status)) {
+               TALLOC_FREE(psd);
+               psd = pdesc_next;
                goto out;
        }
 
        if (memcmp(&hash[0], &hash_tmp[0], XATTR_SD_HASH_SIZE) == 0) {
-               TALLOC_FREE(pdesc_next);
                /* Hash matches, return blob sd. */
                goto out;
        }
 
        /* Hash doesn't match, return underlying sd. */
-
-       if (!(security_info & OWNER_SECURITY_INFORMATION)) {
-               pdesc_next->owner_sid = NULL;
-       }
-       if (!(security_info & GROUP_SECURITY_INFORMATION)) {
-               pdesc_next->group_sid = NULL;
-       }
-       if (!(security_info & DACL_SECURITY_INFORMATION)) {
-               pdesc_next->dacl = NULL;
-       }
-       if (!(security_info & SACL_SECURITY_INFORMATION)) {
-               pdesc_next->sacl = NULL;
-       }
-
-       TALLOC_FREE(*ppdesc);
-       *ppdesc = pdesc_next;
+       TALLOC_FREE(psd);
+       psd = pdesc_next;
 
   out:
 
-       if (!(security_info & OWNER_SECURITY_INFORMATION)) {
-               (*ppdesc)->owner_sid = NULL;
+       if (psd != pdesc_next) {
+               /* We're returning the blob, throw
+                * away the filesystem SD. */
+               TALLOC_FREE(pdesc_next);
+       } else {
+               SMB_STRUCT_STAT sbuf;
+               SMB_STRUCT_STAT *psbuf = &sbuf;
+               bool is_directory = false;
+               /*
+                * We're returning the underlying ACL from the
+                * filesystem. If it's a directory, and has no
+                * inheritable ACE entries we have to fake them.
+                */
+               if (fsp) {
+                       is_directory = fsp->is_directory;
+                       psbuf = &fsp->fsp_name->st;
+               } else {
+                       if (vfs_stat_smb_fname(handle->conn,
+                                               name,
+                                               &sbuf) == 0) {
+                               is_directory = S_ISDIR(sbuf.st_ex_mode);
+                       }
+               }
+               if (is_directory &&
+                               !sd_has_inheritable_components(psd,
+                                                       true)) {
+                       add_directory_inheritable_components(handle,
+                                                       name,
+                                                       psbuf,
+                                                       psd);
+               }
        }
-       if (!(security_info & GROUP_SECURITY_INFORMATION)) {
-               (*ppdesc)->group_sid = NULL;
+
+       if (!(security_info & SECINFO_OWNER)) {
+               psd->owner_sid = NULL;
        }
-       if (!(security_info & DACL_SECURITY_INFORMATION)) {
-               (*ppdesc)->dacl = NULL;
+       if (!(security_info & SECINFO_GROUP)) {
+               psd->group_sid = NULL;
        }
-       if (!(security_info & SACL_SECURITY_INFORMATION)) {
-               (*ppdesc)->sacl = NULL;
+       if (!(security_info & SECINFO_DACL)) {
+               psd->dacl = NULL;
        }
-
-       TALLOC_FREE(blob.data);
-       return status;
-}
-
-/*********************************************************************
- Create a default security descriptor for a file in case no inheritance
- exists. All permissions to the owner and SYSTEM.
-*********************************************************************/
-
-static struct security_descriptor *default_file_sd(TALLOC_CTX *mem_ctx,
-                                               SMB_STRUCT_STAT *psbuf,
-                                               bool force_inherit)
-{
-       struct dom_sid owner_sid, group_sid;
-       size_t sd_size;
-       struct security_ace *pace = NULL;
-       struct security_acl *pacl = NULL;
-
-       uid_to_sid(&owner_sid, psbuf->st_ex_uid);
-       gid_to_sid(&group_sid, psbuf->st_ex_gid);
-
-       pace = TALLOC_ARRAY(mem_ctx, struct security_ace, 2);
-       if (!pace) {
-               return NULL;
+       if (!(security_info & SECINFO_SACL)) {
+               psd->sacl = NULL;
        }
 
-       /* If force_inherit is set, this means we are initializing the ACEs for
-        * a container and we want the ACEs for owner_sid and "SYSTEM" to be
-        * inheritable by their children (See Bug #6802).
-        */
-
-       init_sec_ace(&pace[0], &owner_sid, SEC_ACE_TYPE_ACCESS_ALLOWED,
-                       SEC_RIGHTS_FILE_ALL, (force_inherit ?
-                                       (SEC_ACE_FLAG_OBJECT_INHERIT|
-                                       SEC_ACE_FLAG_CONTAINER_INHERIT) :
-                                       0));
-
-       init_sec_ace(&pace[1], &global_sid_System, SEC_ACE_TYPE_ACCESS_ALLOWED,
-                       SEC_RIGHTS_FILE_ALL, (force_inherit ?
-                                       (SEC_ACE_FLAG_OBJECT_INHERIT|
-                                       SEC_ACE_FLAG_CONTAINER_INHERIT) :
-                                       0));
-
-       pacl = make_sec_acl(mem_ctx,
-                               NT4_ACL_REVISION,
-                               2,
-                               pace);
-       if (!pacl) {
-               return NULL;
-       }
-       return make_sec_desc(mem_ctx,
-                       SECURITY_DESCRIPTOR_REVISION_1,
-                       SEC_DESC_SELF_RELATIVE|SEC_DESC_DACL_PRESENT,
-                       &owner_sid,
-                       &group_sid,
-                       NULL,
-                        pacl,
-                       &sd_size);
+       TALLOC_FREE(blob.data);
+       *ppdesc = 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,
-                                       struct smb_filename *smb_fname,
                                        files_struct *fsp,
-                                       bool container)
+                                       struct security_descriptor *parent_desc,
+                                       bool is_directory)
 {
        TALLOC_CTX *ctx = talloc_tos();
-       NTSTATUS status;
-       struct security_descriptor *parent_desc = NULL;
+       NTSTATUS status = NT_STATUS_OK;
        struct security_descriptor *psd = NULL;
-       struct security_descriptor *pdesc_next = NULL;
-       DATA_BLOB blob;
        size_t size;
-       char *parent_name;
-       bool force_inherit = false;
-       uint8_t hash[XATTR_SD_HASH_SIZE];
 
-       if (!parent_dirname(ctx, smb_fname->base_name, &parent_name, NULL)) {
-               return NT_STATUS_NO_MEMORY;
+       if (!sd_has_inheritable_components(parent_desc, is_directory)) {
+               return NT_STATUS_OK;
        }
 
-       DEBUG(10,("inherit_new_acl: check directory %s\n",
-                       parent_name));
-
-       status = get_nt_acl_internal(handle,
-                               NULL,
-                               parent_name,
-                               (OWNER_SECURITY_INFORMATION |
-                                GROUP_SECURITY_INFORMATION |
-                                DACL_SECURITY_INFORMATION),
-                               &parent_desc);
-        if (NT_STATUS_IS_OK(status)) {
-               /* Create an inherited descriptor from the parent. */
-
-               if (DEBUGLEVEL >= 10) {
-                       DEBUG(10,("inherit_new_acl: parent acl is:\n"));
-                       NDR_PRINT_DEBUG(security_descriptor, parent_desc);
-               }
-
-               status = se_create_child_secdesc(ctx,
-                               &psd,
-                               &size,
-                               parent_desc,
-                               &handle->conn->server_info->ptok->user_sids[PRIMARY_USER_SID_INDEX],
-                               &handle->conn->server_info->ptok->user_sids[PRIMARY_GROUP_SID_INDEX],
-                               container);
-               if (!NT_STATUS_IS_OK(status)) {
-                       return status;
-               }
-
-               if (DEBUGLEVEL >= 10) {
-                       DEBUG(10,("inherit_new_acl: child acl is:\n"));
-                       NDR_PRINT_DEBUG(security_descriptor, psd);
-               }
+       /* Create an inherited descriptor from the parent. */
 
-       } else {
-               DEBUG(10,("inherit_new_acl: directory %s failed "
-                       "to get acl %s\n",
-                       parent_name,
-                       nt_errstr(status) ));
+       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);
+       }
+
+       status = se_create_child_secdesc(ctx,
+                       &psd,
+                       &size,
+                       parent_desc,
+                       &handle->conn->server_info->ptok->sids[PRIMARY_USER_SID_INDEX],
+                       &handle->conn->server_info->ptok->sids[PRIMARY_GROUP_SID_INDEX],
+                       is_directory);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
        }
 
-       if (!psd || psd->dacl == NULL) {
-
-               TALLOC_FREE(psd);
-               if (fsp) {
-                       status = vfs_stat_fsp(fsp);
-                       smb_fname->st = fsp->fsp_name->st;
-               } else {
-                       int ret;
-                       if (lp_posix_pathnames()) {
-                               ret = SMB_VFS_LSTAT(handle->conn, smb_fname);
-                       } else {
-                               ret = SMB_VFS_STAT(handle->conn, smb_fname);
-                       }
-                       if (ret == -1) {
-                               status = map_nt_error_from_unix(errno);
-                       }
-               }
-               if (!NT_STATUS_IS_OK(status)) {
-                       return status;
-               }
-
-               /* If we get here, we could have the following possibilities:
-                *      1. No ACLs exist on the parent container.
-                *      2. ACLs exist on the parent container but they were
-                *      not inheritable.
-                *
-                *      Check to see if case #1 occurred.
-                *
-                */
-               if (container &&
-                       (parent_desc == NULL || parent_desc->dacl == NULL)) {
-
-                       /* If no parent descriptor exists, then there were
-                        * no ACLs on the parent and then we must create
-                        * the ACLs on this newly created folder so that they
-                        * will be inherited by their children (See Bug #6802).
-                        */
+       if (DEBUGLEVEL >= 10) {
+               DEBUG(10,("inherit_new_acl: child acl for %s is:\n",
+                       fsp_str_dbg(fsp) ));
+               NDR_PRINT_DEBUG(security_descriptor, parent_desc);
+       }
 
-                       force_inherit = true;
-               }
+       return SMB_VFS_FSET_NT_ACL(fsp,
+                               (SECINFO_OWNER |
+                                SECINFO_GROUP |
+                                SECINFO_DACL),
+                               psd);
+}
 
-               psd = default_file_sd(ctx, &smb_fname->st, force_inherit);
-               if (!psd) {
-                       return NT_STATUS_NO_MEMORY;
-               }
+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;
 
-               if (DEBUGLEVEL >= 10) {
-                       DEBUG(10,("inherit_new_acl: default acl is:\n"));
-                       NDR_PRINT_DEBUG(security_descriptor, psd);
-               }
+       if (!parent_dirname(talloc_tos(), path, &parent_name, NULL)) {
+               return NT_STATUS_NO_MEMORY;
        }
 
-       /* Object exists. Read the current SD to get the hash. */
-       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,
-                               smb_fname->base_name,
-                               HASH_SECURITY_INFO,
-                               &pdesc_next);
-       }
+       status = get_nt_acl_internal(handle,
+                                       NULL,
+                                       parent_name,
+                                       (SECINFO_OWNER |
+                                        SECINFO_GROUP |
+                                        SECINFO_DACL),
+                                       &parent_desc);
 
        if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(10,("check_parent_acl_common: get_nt_acl_internal "
+                       "on directory %s for "
+                       "path %s returned %s\n",
+                       parent_name,
+                       path,
+                       nt_errstr(status) ));
                return status;
        }
-
-       status = hash_sd_sha256(pdesc_next, hash);
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
+       if (pp_parent_desc) {
+               *pp_parent_desc = parent_desc;
        }
-       status = create_acl_blob(psd, &blob, XATTR_SD_HASH_TYPE_SHA256, hash);
-       if (!NT_STATUS_IS_OK(status)) {
+       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;
        }
-       if (fsp) {
-               return store_acl_blob_fsp(handle, fsp, &blob);
-       } else {
-               return store_acl_blob_pathname(handle, smb_fname->base_name,
-                                              &blob);
-       }
+       return NT_STATUS_OK;
+}
+
+static void free_sd_common(void **ptr)
+{
+       TALLOC_FREE(*ptr);
 }
 
 /*********************************************************************
@@ -490,6 +511,7 @@ static int open_acl_common(vfs_handle_struct *handle,
 {
        uint32_t access_granted = 0;
        struct security_descriptor *pdesc = NULL;
+       struct security_descriptor *parent_desc = NULL;
        bool file_existed = true;
        char *fname = NULL;
        NTSTATUS status;
@@ -497,83 +519,120 @@ static int open_acl_common(vfs_handle_struct *handle,
        if (fsp->base_fsp) {
                /* Stream open. Base filename open already did the ACL check. */
                DEBUG(10,("open_acl_common: stream open on %s\n",
-                       smb_fname_str_dbg(smb_fname) ));
+                       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)) {
-               errno = map_errno_from_nt_status(status);
-               return -1;
+               goto err;
        }
 
        status = get_nt_acl_internal(handle,
                                NULL,
                                fname,
-                               (OWNER_SECURITY_INFORMATION |
-                                GROUP_SECURITY_INFORMATION |
-                                DACL_SECURITY_INFORMATION),
+                               (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(pdesc,
-                                       handle->conn->server_info->ptok,
+               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: file %s open "
+                       DEBUG(10,("open_acl_xattr: %s open "
                                "refused with error %s\n",
-                               smb_fname_str_dbg(smb_fname),
+                               fsp_str_dbg(fsp),
                                nt_errstr(status) ));
-                       errno = map_errno_from_nt_status(status);
-                       return -1;
+                       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 *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. We do have an fsp here, but to
+                        * keep the code consistent with the directory
+                        * case which doesn't, use the handle. */
+
+                       /* Attach this to the conn, move from talloc_tos(). */
+                       psd = (struct security_descriptor *)talloc_move(handle->conn,
+                               &parent_desc);
+
+                       if (!psd) {
+                               status = NT_STATUS_NO_MEMORY;
+                               goto err;
+                       }
+                       status = NT_STATUS_NO_MEMORY;
+                       SMB_VFS_HANDLE_SET_DATA(handle, psd, free_sd_common,
+                               struct security_descriptor *, goto err);
+                       status = NT_STATUS_OK;
+               }
        }
 
        DEBUG(10,("open_acl_xattr: get_nt_acl_attr_internal for "
-               "file %s returned %s\n",
-               smb_fname_str_dbg(smb_fname),
+               "%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;
 
-       if (!file_existed && fsp->fh->fd != -1) {
-               /* File was created. Inherit from parent directory. */
-               status = fsp_set_smb_fname(fsp, smb_fname);
-               if (!NT_STATUS_IS_OK(status)) {
-                       errno = map_errno_from_nt_status(status);
-                       return -1;
-               }
-               inherit_new_acl(handle, smb_fname, fsp, false);
-       }
+  err:
 
-       return fsp->fh->fd;
+       errno = map_errno_from_nt_status(status);
+       return -1;
 }
 
 static int mkdir_acl_common(vfs_handle_struct *handle, const char *path, mode_t mode)
 {
-       struct smb_filename *smb_fname = NULL;
-       int ret = SMB_VFS_NEXT_MKDIR(handle, path, mode);
+       int ret;
        NTSTATUS status;
+       SMB_STRUCT_STAT sbuf;
 
-       if (ret == -1) {
-               return ret;
-       }
+       ret = vfs_stat_smb_fname(handle->conn, path, &sbuf);
+       if (ret == -1 && errno == ENOENT) {
+               struct security_descriptor *parent_desc = NULL;
+               struct security_descriptor *psd = NULL;
 
-       status = create_synthetic_smb_fname(talloc_tos(), path, NULL, NULL,
-                                           &smb_fname);
-       if (!NT_STATUS_IS_OK(status)) {
-               errno = map_errno_from_nt_status(status);
-               return -1;
+               /* We're creating a new directory. */
+               status = check_parent_acl_common(handle, path,
+                               SEC_DIR_ADD_SUBDIR, &parent_desc);
+               if (!NT_STATUS_IS_OK(status)) {
+                       errno = map_errno_from_nt_status(status);
+                       return -1;
+               }
+
+               /* Cache the parent security descriptor for
+                * later use. We don't have an fsp here so
+                * use the handle. */
+
+               /* Attach this to the conn, move from talloc_tos(). */
+               psd = (struct security_descriptor *)talloc_move(handle->conn,
+                               &parent_desc);
+
+               if (!psd) {
+                       return -1;
+               }
+               SMB_VFS_HANDLE_SET_DATA(handle, psd, free_sd_common,
+                       struct security_descriptor *, return -1);
        }
 
-       /* New directory - inherit from parent. */
-       inherit_new_acl(handle, smb_fname, NULL, true);
-       TALLOC_FREE(smb_fname);
-       return ret;
+       return SMB_VFS_NEXT_MKDIR(handle, path, mode);
 }
 
 /*********************************************************************
@@ -617,25 +676,46 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
                        CONST_DISCARD(struct security_descriptor *,psd));
        }
 
-       /* Ensure owner and group are set. */
-       if (!psd->owner_sid || !psd->group_sid) {
-               DOM_SID owner_sid, group_sid;
-               struct security_descriptor *nc_psd = dup_sec_desc(talloc_tos(), psd);
+        /* Ensure we have OWNER/GROUP/DACL set. */
+
+       if ((security_info_sent & (SECINFO_OWNER|
+                               SECINFO_GROUP|
+                               SECINFO_DACL)) !=
+                               (SECINFO_OWNER|
+                                SECINFO_GROUP|
+                                SECINFO_DACL)) {
+               /* No we don't - read from the existing SD. */
+               struct security_descriptor *nc_psd = NULL;
+
+               status = get_nt_acl_internal(handle, fsp,
+                               NULL,
+                               (SECINFO_OWNER|
+                                SECINFO_GROUP|
+                                SECINFO_DACL),
+                               &nc_psd);
 
-               if (!nc_psd) {
-                       return NT_STATUS_OK;
-               }
-               status = vfs_stat_fsp(fsp);
                if (!NT_STATUS_IS_OK(status)) {
-                       /* Lower level acl set succeeded,
-                        * so still return OK. */
-                       return NT_STATUS_OK;
+                       return status;
                }
-               create_file_sids(&fsp->fsp_name->st, &owner_sid, &group_sid);
+
                /* This is safe as nc_psd is discarded at fn exit. */
-               nc_psd->owner_sid = &owner_sid;
-               nc_psd->group_sid = &group_sid;
-               security_info_sent |= (OWNER_SECURITY_INFORMATION|GROUP_SECURITY_INFORMATION);
+               if (security_info_sent & SECINFO_OWNER) {
+                       nc_psd->owner_sid = psd->owner_sid;
+               }
+               security_info_sent |= SECINFO_OWNER;
+
+               if (security_info_sent & SECINFO_GROUP) {
+                       nc_psd->group_sid = psd->group_sid;
+               }
+               security_info_sent |= SECINFO_GROUP;
+
+               if (security_info_sent & SECINFO_DACL) {
+                       nc_psd->dacl = dup_sec_acl(talloc_tos(), psd->dacl);
+                       if (nc_psd->dacl == NULL) {
+                               return NT_STATUS_NO_MEMORY;
+                       }
+               }
+               security_info_sent |= SECINFO_DACL;
                psd = nc_psd;
        }
 
@@ -670,3 +750,240 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
 
        return NT_STATUS_OK;
 }
+
+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);
+
+       if (!NT_STATUS_IS_OK(status)) {
+               errno = map_errno_from_nt_status(status);
+               return NULL;
+       }
+       return SMB_VFS_NEXT_OPENDIR(handle, fname, mask, attr);
+}
+
+static int acl_common_remove_object(vfs_handle_struct *handle,
+                                       const char *path,
+                                       bool is_directory)
+{
+       connection_struct *conn = handle->conn;
+       struct file_id id;
+       files_struct *fsp = NULL;
+       int ret = 0;
+       char *parent_dir = NULL;
+       const char *final_component = NULL;
+       struct smb_filename local_fname;
+       int saved_errno = 0;
+
+       if (!parent_dirname(talloc_tos(), path,
+                       &parent_dir, &final_component)) {
+               saved_errno = ENOMEM;
+               goto out;
+       }
+
+       DEBUG(10,("acl_common_remove_object: removing %s %s/%s\n",
+               is_directory ? "directory" : "file",
+               parent_dir, final_component ));
+
+       /* cd into the parent dir to pin it. */
+       ret = SMB_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);
+
+       /* Must use lstat here. */
+       ret = SMB_VFS_LSTAT(conn, &local_fname);
+       if (ret == -1) {
+               saved_errno = errno;
+               goto out;
+       }
+
+       /* 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(id); fsp; file_find_di_next(fsp)) {
+               if (fsp->access_mask & DELETE_ACCESS &&
+                               fsp->delete_on_close) {
+                       /* We did open this for delete,
+                        * allow the delete as root.
+                        */
+                       break;
+               }
+       }
+
+       if (!fsp) {
+               DEBUG(10,("acl_common_remove_object: %s %s/%s "
+                       "not an open file\n",
+                       is_directory ? "directory" : "file",
+                       parent_dir, final_component ));
+               saved_errno = EACCES;
+               goto out;
+       }
+
+       become_root();
+       if (is_directory) {
+               ret = SMB_VFS_NEXT_RMDIR(handle, final_component);
+       } else {
+               ret = SMB_VFS_NEXT_UNLINK(handle, &local_fname);
+       }
+       unbecome_root();
+
+       if (ret == -1) {
+               saved_errno = errno;
+       }
+
+  out:
+
+       TALLOC_FREE(parent_dir);
+
+       vfs_ChDir(conn, conn->connectpath);
+       if (saved_errno) {
+               errno = saved_errno;
+       }
+       return ret;
+}
+
+static int rmdir_acl_common(struct vfs_handle_struct *handle,
+                               const char *path)
+{
+       int ret;
+
+       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;
+
+       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 (info != FILE_WAS_CREATED) {
+               /* File/directory was opened, not created. */
+               goto out;
+       }
+
+       fsp = *result;
+
+       if (!NT_STATUS_IS_OK(status) || fsp == NULL) {
+               /* Only handle success. */
+               goto out;
+       }
+
+       if (sd) {
+               /* Security descriptor already set. */
+               goto out;
+       }
+
+       if (fsp->base_fsp) {
+               /* Stream open. */
+               goto out;
+       }
+
+
+       /* We must have a cached parent sd in this case.
+        * attached to the handle. */
+
+       SMB_VFS_HANDLE_GET_DATA(handle, parent_sd,
+               struct security_descriptor,
+               goto err);
+
+       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) ));
+       }
+
+  out:
+
+       /* Ensure we never leave attached data around. */
+       SMB_VFS_HANDLE_FREE_DATA(handle);
+
+       if (NT_STATUS_IS_OK(status) && pinfo) {
+               *pinfo = info;
+       }
+       return status;
+
+  err:
+
+       smb_panic("create_file_acl_common: logic error.\n");
+       /* NOTREACHED */
+       return status;
+}
+
+static int unlink_acl_common(struct vfs_handle_struct *handle,
+                       const struct smb_filename *smb_fname)
+{
+       int ret;
+
+       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;
+       }
+
+       return acl_common_remove_object(handle,
+                                       smb_fname->base_name,
+                                       false);
+}