selftest/Samba4: make use of get_cmd_env_vars() to setup all relevant env variables
[samba.git] / source3 / modules / vfs_acl_common.c
index 0fd625bec1de00a3ef4aada50af2cae23f73136f..e04b672cf9ae7ddb2f561ba7f85217d49d9426c9 100644 (file)
 #include "../libcli/security/security.h"
 #include "../librpc/gen_ndr/ndr_security.h"
 #include "../lib/util/bitmap.h"
-#include "lib/crypto/sha256.h"
 #include "passdb/lookup_sid.h"
 
+#include <gnutls/gnutls.h>
+#include <gnutls/crypto.h>
+
 static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
                        DATA_BLOB *pblob,
                        uint16_t hash_type,
@@ -81,13 +83,17 @@ bool init_acl_common_config(vfs_handle_struct *handle,
 static NTSTATUS hash_blob_sha256(DATA_BLOB blob,
                                 uint8_t *hash)
 {
-       SHA256_CTX tctx;
+       int rc;
 
-       memset(hash, '\0', XATTR_SD_HASH_SIZE);
+       ZERO_ARRAY_LEN(hash, XATTR_SD_HASH_SIZE);
 
-       samba_SHA256_Init(&tctx);
-       samba_SHA256_Update(&tctx, blob.data, blob.length);
-       samba_SHA256_Final(hash, &tctx);
+       rc = gnutls_hash_fast(GNUTLS_DIG_SHA256,
+                             blob.data,
+                             blob.length,
+                             hash);
+       if (rc < 0) {
+               return NT_STATUS_INTERNAL_ERROR;
+       }
 
        return NT_STATUS_OK;
 }
@@ -253,9 +259,6 @@ static NTSTATUS create_sys_acl_blob(const struct security_descriptor *psd,
        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);
@@ -266,7 +269,6 @@ static NTSTATUS create_sys_acl_blob(const 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(
@@ -309,7 +311,7 @@ static NTSTATUS add_directory_inheritable_components(vfs_handle_struct *handle,
        } else {
                /*
                 * make_sec_acl() at the bottom of this function
-                * dupliates new_ace_list
+                * duplicates new_ace_list
                 */
                new_ace_list = talloc_zero_array(talloc_tos(),
                                                 struct security_ace,
@@ -398,17 +400,17 @@ static NTSTATUS add_directory_inheritable_components(vfs_handle_struct *handle,
  * and psd_from_fs set to false.
  *
  * Returning the underlying filesystem ACL in case no. 2 is really just an
- * optimisation, because some validations have to fetch the filesytem ACL as
+ * optimisation, because some validations have to fetch the filesystem ACL as
  * part of the validation, so we already have it available and callers might
  * need it as well.
  **/
 static NTSTATUS validate_nt_acl_blob(TALLOC_CTX *mem_ctx,
-                                    vfs_handle_struct *handle,
-                                    files_struct *fsp,
-                                    const struct smb_filename *smb_fname,
-                                    const DATA_BLOB *blob,
-                                    struct security_descriptor **ppsd,
-                                    bool *psd_is_from_fs)
+                               vfs_handle_struct *handle,
+                               struct files_struct *fsp,
+                               const struct smb_filename *smb_fname,
+                               const DATA_BLOB *blob,
+                               struct security_descriptor **ppsd,
+                               bool *psd_is_from_fs)
 {
        NTSTATUS status;
        uint16_t hash_type = XATTR_SD_HASH_TYPE_NONE;
@@ -447,7 +449,7 @@ static NTSTATUS validate_nt_acl_blob(TALLOC_CTX *mem_ctx,
        switch (xattr_version) {
        case 1:
        case 2:
-               /* These xattr types are unilatteral, they do not
+               /* These xattr types are unilateral, 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 */
@@ -481,22 +483,12 @@ static NTSTATUS validate_nt_acl_blob(TALLOC_CTX *mem_ctx,
        case 4:
        {
                int ret;
-               if (fsp) {
-                       /* Get the full underlying sd, then hash. */
-                       ret = SMB_VFS_NEXT_SYS_ACL_BLOB_GET_FD(handle,
-                                                              fsp,
-                                                              mem_ctx,
-                                                              &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,
-                                                smb_fname,
-                                                mem_ctx,
-                                                &sys_acl_blob_description,
-                                                &sys_acl_blob);
-               }
-
+               /* Get the full underlying sd, then hash. */
+               ret = SMB_VFS_NEXT_SYS_ACL_BLOB_GET_FD(handle,
+                                                      fsp,
+                                                      mem_ctx,
+                                                      &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) {
@@ -524,20 +516,11 @@ static NTSTATUS validate_nt_acl_blob(TALLOC_CTX *mem_ctx,
        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,
-                                                         &psd_fs);
-               } else {
-                       status = SMB_VFS_NEXT_GET_NT_ACL(handle,
-                                                        smb_fname,
-                                                        HASH_SECURITY_INFO,
-                                                        mem_ctx,
-                                                        &psd_fs);
-               }
-
+               status = SMB_VFS_NEXT_FGET_NT_ACL(handle,
+                                                 fsp,
+                                                 HASH_SECURITY_INFO,
+                                                 mem_ctx,
+                                                 &psd_fs);
                if (!NT_STATUS_IS_OK(status)) {
                        DBG_DEBUG("get_next_acl for file %s returned %s\n",
                                  smb_fname->base_name, nt_errstr(status));
@@ -587,63 +570,19 @@ fail:
        return status;
 }
 
-static NTSTATUS stat_fsp_or_smb_fname(vfs_handle_struct *handle,
-                                     files_struct *fsp,
-                                     const struct smb_filename *smb_fname,
-                                     SMB_STRUCT_STAT *sbuf,
-                                     SMB_STRUCT_STAT **psbuf)
-{
-       NTSTATUS status;
-       int ret;
-
-       if (fsp) {
-               status = vfs_stat_fsp(fsp);
-               if (!NT_STATUS_IS_OK(status)) {
-                       return status;
-               }
-               *psbuf = &fsp->fsp_name->st;
-       } else {
-               /*
-                * 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.
-                */
-               ret = vfs_stat_smb_basename(handle->conn,
-                                           smb_fname,
-                                           sbuf);
-               if (ret == -1) {
-                       return map_nt_error_from_unix(errno);
-               }
-       }
-
-       return NT_STATUS_OK;
-}
-
 /*******************************************************************
- Pull a DATA_BLOB from an xattr given a pathname.
+ Pull a DATA_BLOB from an xattr given an fsp.
  If the hash doesn't match, or doesn't exist - return the underlying
  filesystem sd.
 *******************************************************************/
 
-NTSTATUS get_nt_acl_common(
-       NTSTATUS (*get_acl_blob_fn)(TALLOC_CTX *ctx,
+NTSTATUS fget_nt_acl_common(
+       NTSTATUS (*fget_acl_blob_fn)(TALLOC_CTX *ctx,
                                    vfs_handle_struct *handle,
                                    files_struct *fsp,
-                                   const struct smb_filename *smb_fname,
                                    DATA_BLOB *pblob),
        vfs_handle_struct *handle,
        files_struct *fsp,
-       const struct smb_filename *smb_fname_in,
        uint32_t security_info,
        TALLOC_CTX *mem_ctx,
        struct security_descriptor **ppdesc)
@@ -651,7 +590,7 @@ NTSTATUS get_nt_acl_common(
        DATA_BLOB blob = data_blob_null;
        NTSTATUS status;
        struct security_descriptor *psd = NULL;
-       const struct smb_filename *smb_fname = NULL;
+       const struct smb_filename *smb_fname = fsp->fsp_name;
        bool psd_is_from_fs = false;
        struct acl_common_config *config = NULL;
 
@@ -659,23 +598,17 @@ NTSTATUS get_nt_acl_common(
                                struct acl_common_config,
                                return NT_STATUS_UNSUCCESSFUL);
 
-       if (fsp && smb_fname_in == NULL) {
-               smb_fname = fsp->fsp_name;
-       } else {
-               smb_fname = smb_fname_in;
-       }
-
        DBG_DEBUG("name=%s\n", smb_fname->base_name);
 
-       status = get_acl_blob_fn(mem_ctx, handle, fsp, smb_fname, &blob);
+       status = fget_acl_blob_fn(mem_ctx, handle, fsp, &blob);
        if (NT_STATUS_IS_OK(status)) {
                status = validate_nt_acl_blob(mem_ctx,
-                                             handle,
-                                             fsp,
-                                             smb_fname,
-                                             &blob,
-                                             &psd,
-                                             &psd_is_from_fs);
+                                       handle,
+                                       fsp,
+                                       smb_fname,
+                                       &blob,
+                                       &psd,
+                                       &psd_is_from_fs);
                TALLOC_FREE(blob.data);
                if (!NT_STATUS_IS_OK(status)) {
                        DBG_DEBUG("ACL validation for [%s] failed\n",
@@ -690,11 +623,7 @@ NTSTATUS get_nt_acl_common(
                 * known */
 
                if (config->ignore_system_acls) {
-                       SMB_STRUCT_STAT sbuf;
-                       SMB_STRUCT_STAT *psbuf = &sbuf;
-
-                       status = stat_fsp_or_smb_fname(handle, fsp, smb_fname,
-                                                      &sbuf, &psbuf);
+                       status = vfs_stat_fsp(fsp);
                        if (!NT_STATUS_IS_OK(status)) {
                                goto fail;
                        }
@@ -703,25 +632,17 @@ NTSTATUS get_nt_acl_common(
                                mem_ctx,
                                config->default_acl_style,
                                smb_fname->base_name,
-                               psbuf,
+                               &fsp->fsp_name->st,
                                &psd);
                        if (!NT_STATUS_IS_OK(status)) {
                                goto fail;
                        }
                } else {
-                       if (fsp) {
-                               status = SMB_VFS_NEXT_FGET_NT_ACL(handle,
-                                                                 fsp,
-                                                                 security_info,
-                                                                 mem_ctx,
-                                                                 &psd);
-                       } else {
-                               status = SMB_VFS_NEXT_GET_NT_ACL(handle,
-                                                                smb_fname,
-                                                                security_info,
-                                                                mem_ctx,
-                                                                &psd);
-                       }
+                       status = SMB_VFS_NEXT_FGET_NT_ACL(handle,
+                                                         fsp,
+                                                         security_info,
+                                                         mem_ctx,
+                                                         &psd);
 
                        if (!NT_STATUS_IS_OK(status)) {
                                DBG_DEBUG("get_next_acl for file %s "
@@ -736,9 +657,10 @@ NTSTATUS get_nt_acl_common(
        }
 
        if (psd_is_from_fs) {
-               SMB_STRUCT_STAT sbuf;
-               SMB_STRUCT_STAT *psbuf = &sbuf;
-               bool is_directory = false;
+               status = vfs_stat_fsp(fsp);
+               if (!NT_STATUS_IS_OK(status)) {
+                       goto fail;
+               }
 
                /*
                 * We're returning the underlying ACL from the
@@ -746,19 +668,12 @@ NTSTATUS get_nt_acl_common(
                 * inheritable ACE entries we have to fake them.
                 */
 
-               status = stat_fsp_or_smb_fname(handle, fsp, smb_fname,
-                                              &sbuf, &psbuf);
-               if (!NT_STATUS_IS_OK(status)) {
-                       goto fail;
-               }
-
-               is_directory = S_ISDIR(psbuf->st_ex_mode);
-
-               if (is_directory && !sd_has_inheritable_components(psd, true)) {
+               if (fsp->fsp_flags.is_directory &&
+                               !sd_has_inheritable_components(psd, true)) {
                        status = add_directory_inheritable_components(
                                handle,
                                smb_fname->base_name,
-                               psbuf,
+                               &fsp->fsp_name->st,
                                psd);
                        if (!NT_STATUS_IS_OK(status)) {
                                goto fail;
@@ -813,6 +728,7 @@ static NTSTATUS set_underlying_acl(vfs_handle_struct *handle, files_struct *fsp,
 {
        NTSTATUS status;
        const struct security_token *token = NULL;
+       struct dom_sid_buf buf;
 
        status = SMB_VFS_NEXT_FSET_NT_ACL(handle, fsp, security_info_sent, psd);
        if (!NT_STATUS_EQUAL(status, NT_STATUS_ACCESS_DENIED)) {
@@ -822,10 +738,13 @@ static NTSTATUS set_underlying_acl(vfs_handle_struct *handle, files_struct *fsp,
        /* 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)) {
+       if (get_current_uid(handle->conn) == 0 || !chown_needed) {
                return NT_STATUS_ACCESS_DENIED;
        }
+       status = check_any_access_fsp(fsp, SEC_STD_WRITE_OWNER);
+       if (!NT_STATUS_IS_OK(status)) {
+               return status;
+       }
 
        /*
         * Only allow take-ownership, not give-ownership. That's the way Windows
@@ -840,7 +759,8 @@ static NTSTATUS set_underlying_acl(vfs_handle_struct *handle, files_struct *fsp,
        }
 
        DBG_DEBUG("overriding chown on file %s for sid %s\n",
-                  fsp_str_dbg(fsp), sid_string_tos(psd->owner_sid));
+                 fsp_str_dbg(fsp),
+                 dom_sid_str_buf(psd->owner_sid, &buf));
 
        /* Ok, we failed to chown and we have
           SEC_STD_WRITE_OWNER access - override. */
@@ -898,10 +818,9 @@ static NTSTATUS store_v3_blob(
 *********************************************************************/
 
 NTSTATUS fset_nt_acl_common(
-       NTSTATUS (*get_acl_blob_fn)(TALLOC_CTX *ctx,
+       NTSTATUS (*fget_acl_blob_fn)(TALLOC_CTX *ctx,
                                    vfs_handle_struct *handle,
                                    files_struct *fsp,
-                                   const struct smb_filename *smb_fname,
                                    DATA_BLOB *pblob),
        NTSTATUS (*store_acl_blob_fsp_fn)(vfs_handle_struct *handle,
                                          files_struct *fsp,
@@ -923,6 +842,7 @@ NTSTATUS fset_nt_acl_common(
        TALLOC_CTX *frame = talloc_stackframe();
        bool ignore_file_system_acl = lp_parm_bool(
            SNUM(handle->conn), module_name, "ignore system acls", false);
+       struct acl_common_fsp_ext *ext = NULL;
 
        if (DEBUGLEVEL >= 10) {
                DBG_DEBUG("incoming sd for file %s\n", fsp_str_dbg(fsp));
@@ -930,8 +850,7 @@ NTSTATUS fset_nt_acl_common(
                        discard_const_p(struct security_descriptor, orig_psd));
        }
 
-       status = get_nt_acl_common(get_acl_blob_fn, handle, fsp,
-                       NULL,
+       status = fget_nt_acl_common(fget_acl_blob_fn, handle, fsp,
                        SECINFO_OWNER|SECINFO_GROUP|SECINFO_DACL|SECINFO_SACL,
                                     frame,
                        &psd);
@@ -980,6 +899,12 @@ NTSTATUS fset_nt_acl_common(
                psd->type |= SEC_DESC_SACL_PRESENT;
        }
 
+       ext = VFS_ADD_FSP_EXTENSION(handle,
+                                   fsp,
+                                   struct acl_common_fsp_ext,
+                                   NULL);
+       ext->setting_nt_acl = true;
+
        if (ignore_file_system_acl) {
                if (chown_needed) {
                        /* send only ownership stuff to lower layer */
@@ -987,23 +912,19 @@ NTSTATUS fset_nt_acl_common(
                        status = set_underlying_acl(handle, fsp, psd,
                                                    security_info_sent, true);
                        if (!NT_STATUS_IS_OK(status)) {
-                               TALLOC_FREE(frame);
-                               return status;
+                               goto done;
                        }
                }
                ZERO_ARRAY(hash);
                status = store_v3_blob(store_acl_blob_fsp_fn, handle, fsp, psd,
                                       NULL, hash);
-
-               TALLOC_FREE(frame);
-               return status;
+               goto done;
        }
 
        status = set_underlying_acl(handle, fsp, psd, security_info_sent,
                                    chown_needed);
        if (!NT_STATUS_IS_OK(status)) {
-               TALLOC_FREE(frame);
-               return status;
+               goto done;
        }
 
        /* Get the full underlying sd, then hash. */
@@ -1014,14 +935,12 @@ NTSTATUS fset_nt_acl_common(
                                          &pdesc_next);
 
        if (!NT_STATUS_IS_OK(status)) {
-               TALLOC_FREE(frame);
-               return status;
+               goto done;
        }
 
        status = hash_sd_sha256(pdesc_next, hash);
        if (!NT_STATUS_IS_OK(status)) {
-               TALLOC_FREE(frame);
-               return status;
+               goto done;
        }
 
        /* Get the full underlying sd, then hash. */
@@ -1037,14 +956,12 @@ NTSTATUS fset_nt_acl_common(
                status = store_v3_blob(store_acl_blob_fsp_fn, handle, fsp, psd,
                                       pdesc_next, hash);
 
-               TALLOC_FREE(frame);
-               return status;
+               goto done;
        }
 
        status = hash_blob_sha256(sys_acl_blob, sys_acl_hash);
        if (!NT_STATUS_IS_OK(status)) {
-               TALLOC_FREE(frame);
-               return status;
+               goto done;
        }
 
        if (DEBUGLEVEL >= 10) {
@@ -1059,24 +976,26 @@ NTSTATUS fset_nt_acl_common(
        }
 
        /* 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
+        * security descriptor mapped from that ACL so as to improve
+        * our chances against some inadvertent 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)) {
                DBG_DEBUG("create_sys_acl_blob failed\n");
-               TALLOC_FREE(frame);
-               return status;
+               goto done;
        }
 
        status = store_acl_blob_fsp_fn(handle, fsp, &blob);
 
+done:
+       VFS_REMOVE_FSP_EXTENSION(handle, fsp);
        TALLOC_FREE(frame);
        return status;
 }
 
 static int acl_common_remove_object(vfs_handle_struct *handle,
+                                       struct files_struct *dirfsp,
                                        const struct smb_filename *smb_fname,
                                        bool is_directory)
 {
@@ -1084,12 +1003,12 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
        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 = {0};
-       struct smb_filename parent_dir_fname = {0};
+       struct smb_filename *full_fname = NULL;
+       struct smb_filename *local_fname = NULL;
+       struct smb_filename *parent_dir_fname = NULL;
        int saved_errno = 0;
        struct smb_filename *saved_dir_fname = NULL;
+       NTSTATUS status;
 
        saved_dir_fname = vfs_GetWd(talloc_tos(),conn);
        if (saved_dir_fname == NULL) {
@@ -1097,39 +1016,47 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
                goto out;
        }
 
-       if (!parent_dirname(talloc_tos(), smb_fname->base_name,
-                       &parent_dir, &final_component)) {
-               saved_errno = ENOMEM;
+       full_fname = full_path_from_dirfsp_atname(talloc_tos(),
+                                                 dirfsp,
+                                                 smb_fname);
+       if (full_fname == NULL) {
                goto out;
        }
 
-       DBG_DEBUG("removing %s %s/%s\n", is_directory ? "directory" : "file",
-                 parent_dir, final_component);
+       status = SMB_VFS_PARENT_PATHNAME(conn,
+                                        talloc_tos(),
+                                        full_fname,
+                                        &parent_dir_fname,
+                                        &local_fname);
+       if (!NT_STATUS_IS_OK(status)) {
+               saved_errno = map_errno_from_nt_status(status);
+               goto out;
+       }
 
-       parent_dir_fname = (struct smb_filename) { .base_name = parent_dir };
+       DBG_DEBUG("removing %s %s\n", is_directory ? "directory" : "file",
+                 smb_fname_str_dbg(full_fname));
 
        /* cd into the parent dir to pin it. */
-       ret = vfs_ChDir(conn, &parent_dir_fname);
+       ret = vfs_ChDir(conn, parent_dir_fname);
        if (ret == -1) {
                saved_errno = errno;
                goto out;
        }
 
-       local_fname.base_name = discard_const_p(char, final_component);
-
        /* Must use lstat here. */
-       ret = SMB_VFS_LSTAT(conn, &local_fname);
+       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(conn->sconn, id); fsp;
-                    fsp = file_find_di_next(fsp)) {
+       id = vfs_file_id_from_sbuf(conn, &local_fname->st);
+       for (fsp = file_find_di_first(conn->sconn, id, true); fsp;
+                    fsp = file_find_di_next(fsp, true)) {
                if (fsp->access_mask & DELETE_ACCESS &&
-                               fsp->delete_on_close) {
+                   fsp->fsp_flags.delete_on_close)
+               {
                        /* We did open this for delete,
                         * allow the delete as root.
                         */
@@ -1138,18 +1065,24 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
        }
 
        if (!fsp) {
-               DBG_DEBUG("%s %s/%s not an open file\n",
+               DBG_DEBUG("%s %s not an open file\n",
                          is_directory ? "directory" : "file",
-                         parent_dir, final_component);
+                         smb_fname_str_dbg(full_fname));
                saved_errno = EACCES;
                goto out;
        }
 
        become_root();
        if (is_directory) {
-               ret = SMB_VFS_NEXT_RMDIR(handle, &local_fname);
+               ret = SMB_VFS_NEXT_UNLINKAT(handle,
+                               dirfsp,
+                               smb_fname,
+                               AT_REMOVEDIR);
        } else {
-               ret = SMB_VFS_NEXT_UNLINK(handle, &local_fname);
+               ret = SMB_VFS_NEXT_UNLINKAT(handle,
+                               dirfsp,
+                               smb_fname,
+                               0);
        }
        unbecome_root();
 
@@ -1159,7 +1092,8 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
 
   out:
 
-       TALLOC_FREE(parent_dir);
+       TALLOC_FREE(parent_dir_fname);
+       TALLOC_FREE(full_fname);
 
        if (saved_dir_fname) {
                vfs_ChDir(conn, saved_dir_fname);
@@ -1172,12 +1106,16 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
 }
 
 int rmdir_acl_common(struct vfs_handle_struct *handle,
-                    const struct smb_filename *smb_fname)
+               struct files_struct *dirfsp,
+               const struct smb_filename *smb_fname)
 {
        int ret;
 
        /* Try the normal rmdir first. */
-       ret = SMB_VFS_NEXT_RMDIR(handle, smb_fname);
+       ret = SMB_VFS_NEXT_UNLINKAT(handle,
+                       dirfsp,
+                       smb_fname,
+                       AT_REMOVEDIR);
        if (ret == 0) {
                return 0;
        }
@@ -1185,6 +1123,7 @@ int rmdir_acl_common(struct vfs_handle_struct *handle,
                /* Failed due to access denied,
                   see if we need to root override. */
                return acl_common_remove_object(handle,
+                                               dirfsp,
                                                smb_fname,
                                                true);
        }
@@ -1196,12 +1135,17 @@ int rmdir_acl_common(struct vfs_handle_struct *handle,
 }
 
 int unlink_acl_common(struct vfs_handle_struct *handle,
-                       const struct smb_filename *smb_fname)
+                       struct files_struct *dirfsp,
+                       const struct smb_filename *smb_fname,
+                       int flags)
 {
        int ret;
 
        /* Try the normal unlink first. */
-       ret = SMB_VFS_NEXT_UNLINK(handle, smb_fname);
+       ret = SMB_VFS_NEXT_UNLINKAT(handle,
+                               dirfsp,
+                               smb_fname,
+                               flags);
        if (ret == 0) {
                return 0;
        }
@@ -1214,6 +1158,7 @@ int unlink_acl_common(struct vfs_handle_struct *handle,
                        return -1;
                }
                return acl_common_remove_object(handle,
+                                       dirfsp,
                                        smb_fname,
                                        false);
        }
@@ -1224,33 +1169,13 @@ int unlink_acl_common(struct vfs_handle_struct *handle,
        return -1;
 }
 
-int chmod_acl_module_common(struct vfs_handle_struct *handle,
-                           const struct smb_filename *smb_fname,
-                           mode_t mode)
-{
-       if (smb_fname->flags & SMB_FILENAME_POSIX_PATH) {
-               /* Only allow this on POSIX pathnames. */
-               return SMB_VFS_NEXT_CHMOD(handle, smb_fname, mode);
-       }
-       return 0;
-}
-
 int fchmod_acl_module_common(struct vfs_handle_struct *handle,
                             struct files_struct *fsp, mode_t mode)
 {
-       if (fsp->posix_flags & FSP_POSIX_FLAGS_OPEN) {
+       if (fsp->posix_flags & FSP_POSIX_FLAGS_PATHNAMES
+           || fsp->fsp_name->flags & SMB_FILENAME_POSIX_PATH) {
                /* Only allow this on POSIX opens. */
                return SMB_VFS_NEXT_FCHMOD(handle, fsp, mode);
        }
        return 0;
 }
-
-int fchmod_acl_acl_module_common(struct vfs_handle_struct *handle,
-                                struct files_struct *fsp, mode_t mode)
-{
-       if (fsp->posix_flags & FSP_POSIX_FLAGS_OPEN) {
-               /* Only allow this on POSIX opens. */
-               return SMB_VFS_NEXT_FCHMOD_ACL(handle, fsp, mode);
-       }
-       return 0;
-}