Fix bug #8370 - vfs_chown_fsp broken -- returns in the wrong directory
[idra/samba.git] / source3 / modules / vfs_acl_common.c
index 58da9047692434d237383cb0e8568a4e98373466..a4d712f5257c19c2f986bbc369d9cfc66a0d18d4 100644 (file)
  * along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
+#include "smbd/smbd.h"
+#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,
@@ -58,9 +62,9 @@ static NTSTATUS hash_sd_sha256(struct security_descriptor *psd,
                return status;
        }
 
-       SHA256_Init(&tctx);
-       SHA256_Update(&tctx, blob.data, blob.length);
-       SHA256_Final(hash, &tctx);
+       samba_SHA256_Init(&tctx);
+       samba_SHA256_Update(&tctx, blob.data, blob.length);
+       samba_SHA256_Final(hash, &tctx);
 
        return NT_STATUS_OK;
 }
@@ -85,7 +89,7 @@ static NTSTATUS parse_acl_blob(const DATA_BLOB *pblob,
        if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
                DEBUG(5, ("parse_acl_blob: ndr_pull_xattr_NTACL failed: %s\n",
                        ndr_errstr(ndr_err)));
-               return ndr_map_error2ntstatus(ndr_err);;
+               return ndr_map_error2ntstatus(ndr_err);
        }
 
        switch (xacl.version) {
@@ -141,7 +145,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);
 
@@ -152,7 +156,7 @@ static NTSTATUS create_acl_blob(const struct security_descriptor *psd,
        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 ndr_map_error2ntstatus(ndr_err);
        }
 
        return NT_STATUS_OK;
@@ -176,7 +180,7 @@ 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 *new_ace_list = talloc_zero_array(talloc_tos(),
                                                struct security_ace,
                                                num_aces + 3);
 
@@ -187,7 +191,7 @@ static void add_directory_inheritable_components(vfs_handle_struct *handle,
        /* 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);
@@ -256,6 +260,10 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
        uint8_t hash_tmp[XATTR_SD_HASH_SIZE];
        struct security_descriptor *psd = NULL;
        struct security_descriptor *pdesc_next = NULL;
+       bool ignore_file_system_acl = lp_parm_bool(SNUM(handle->conn),
+                                               ACL_MODULE_NAME,
+                                               "ignore system acls",
+                                               false);
 
        if (fsp && name == NULL) {
                name = fsp->fsp_name->base_name;
@@ -319,6 +327,9 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
                        goto out;
        }
 
+       if (ignore_file_system_acl) {
+               goto out;
+       }
 
        status = hash_sd_sha256(pdesc_next, hash_tmp);
        if (!NT_STATUS_IS_OK(status)) {
@@ -355,28 +366,45 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
                 * inheritable ACE entries we have to fake them.
                 */
                if (fsp) {
-                       is_directory = fsp->is_directory;
+                       status = vfs_stat_fsp(fsp);
+                       if (!NT_STATUS_IS_OK(status)) {
+                               return status;
+                       }
                        psbuf = &fsp->fsp_name->st;
                } else {
-                       if (vfs_stat_smb_fname(handle->conn,
+                       int ret = vfs_stat_smb_fname(handle->conn,
                                                name,
-                                               &sbuf) == 0) {
-                               is_directory = S_ISDIR(sbuf.st_ex_mode);
+                                               &sbuf);
+                       if (ret == -1) {
+                               return map_nt_error_from_unix(errno);
                        }
                }
-               if (is_directory &&
+               is_directory = S_ISDIR(sbuf.st_ex_mode);
+
+               if (ignore_file_system_acl) {
+                       TALLOC_FREE(pdesc_next);
+                       status = make_default_filesystem_acl(talloc_tos(),
+                                               name,
+                                               psbuf,
+                                               &psd);
+                       if (!NT_STATUS_IS_OK(status)) {
+                               return status;
+                       }
+               } else {
+                       if (is_directory &&
                                !sd_has_inheritable_components(psd,
                                                        true)) {
-                       add_directory_inheritable_components(handle,
+                               add_directory_inheritable_components(handle,
                                                        name,
                                                        psbuf,
                                                        psd);
+                       }
+                       /* The underlying POSIX module always sets
+                          the ~SEC_DESC_DACL_PROTECTED bit, as ACLs
+                          can't be inherited in this way under POSIX.
+                          Remove it for Windows-style ACLs. */
+                       psd->type &= ~SEC_DESC_DACL_PROTECTED;
                }
-               /* The underlying POSIX module always sets
-                  the ~SEC_DESC_DACL_PROTECTED bit, as ACLs
-                  can't be inherited in this way under POSIX.
-                  Remove it for Windows-style ACLs. */
-               psd->type &= ~SEC_DESC_DACL_PROTECTED;
        }
 
        if (!(security_info & SECINFO_OWNER)) {
@@ -394,6 +422,13 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
 
        TALLOC_FREE(blob.data);
        *ppdesc = psd;
+
+       if (DEBUGLEVEL >= 10) {
+               DEBUG(10,("get_nt_acl_internal: returning acl for %s is:\n",
+                       name ));
+               NDR_PRINT_DEBUG(security_descriptor, psd);
+       }
+
        return NT_STATUS_OK;
 }
 
@@ -412,9 +447,16 @@ static NTSTATUS inherit_new_acl(vfs_handle_struct *handle,
        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;
+       uint32_t security_info_sent = (SECINFO_OWNER | SECINFO_GROUP | SECINFO_DACL);
+       bool inherit_owner = lp_inherit_owner(SNUM(handle->conn));
+       bool inheritable_components = sd_has_inheritable_components(parent_desc,
+                                       is_directory);
        size_t size;
 
-       if (!sd_has_inheritable_components(parent_desc, is_directory)) {
+       if (!inheritable_components && !inherit_owner) {
+               /* Nothing to inherit and not setting owner. */
                return NT_STATUS_OK;
        }
 
@@ -426,38 +468,65 @@ static NTSTATUS inherit_new_acl(vfs_handle_struct *handle,
                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;
+       }
+
+       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,
-                       &handle->conn->server_info->ptok->sids[PRIMARY_USER_SID_INDEX],
-                       &handle->conn->server_info->ptok->sids[PRIMARY_GROUP_SID_INDEX],
+                       owner_sid,
+                       group_sid,
                        is_directory);
        if (!NT_STATUS_IS_OK(status)) {
                return status;
        }
 
+       /* If inheritable_components == false,
+          se_create_child_secdesc()
+          creates a security desriptor with a NULL dacl
+          entry, but with SEC_DESC_DACL_PRESENT. We need
+          to remove that flag. */
+
+       if (!inheritable_components) {
+               security_info_sent &= ~SECINFO_DACL;
+               psd->type &= ~SEC_DESC_DACL_PRESENT;
+       }
+
        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);
+               NDR_PRINT_DEBUG(security_descriptor, psd);
        }
 
-       return SMB_VFS_FSET_NT_ACL(fsp,
-                               (SECINFO_OWNER |
-                                SECINFO_GROUP |
-                                SECINFO_DACL),
+       if (inherit_owner) {
+               /* We need to be root to force this. */
+               become_root();
+       }
+       status = SMB_VFS_FSET_NT_ACL(fsp,
+                               security_info_sent,
                                psd);
+       if (inherit_owner) {
+               unbecome_root();
+       }
+       return status;
 }
 
-static NTSTATUS check_parent_acl_common(vfs_handle_struct *handle,
+static NTSTATUS get_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 (!parent_dirname(talloc_tos(), path, &parent_name, NULL)) {
@@ -470,15 +539,31 @@ static NTSTATUS check_parent_acl_common(vfs_handle_struct *handle,
                                        (SECINFO_OWNER |
                                         SECINFO_GROUP |
                                         SECINFO_DACL),
-                                       &parent_desc);
+                                       pp_parent_desc);
 
        if (!NT_STATUS_IS_OK(status)) {
-               DEBUG(10,("check_parent_acl_common: get_nt_acl_internal "
+               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) {
@@ -502,11 +587,6 @@ static NTSTATUS check_parent_acl_common(vfs_handle_struct *handle,
        return NT_STATUS_OK;
 }
 
-static void free_sd_common(void **ptr)
-{
-       TALLOC_FREE(*ptr);
-}
-
 /*********************************************************************
  Check ACL on open. For new files inherit from parent directory.
 *********************************************************************/
@@ -519,7 +599,6 @@ 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;
@@ -565,29 +644,28 @@ static int open_acl_common(vfs_handle_struct *handle,
                 * Check the parent directory ACL will allow this.
                 */
                if (flags & O_CREAT) {
-                       struct security_descriptor *psd = NULL;
+                       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. 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);
+                       /* Cache the parent security descriptor for
+                        * later use. */
 
-                       if (!psd) {
+                       pp_psd = VFS_ADD_FSP_EXTENSION(handle,
+                                       fsp,
+                                       struct security_descriptor *,
+                                       NULL);
+                       if (!pp_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);
+
+                       *pp_psd = parent_desc;
                        status = NT_STATUS_OK;
                }
        }
@@ -614,30 +692,13 @@ static int mkdir_acl_common(vfs_handle_struct *handle, const char *path, mode_t
 
        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;
-
                /* We're creating a new directory. */
                status = check_parent_acl_common(handle, path,
-                               SEC_DIR_ADD_SUBDIR, &parent_desc);
+                               SEC_DIR_ADD_SUBDIR, NULL);
                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);
        }
 
        return SMB_VFS_NEXT_MKDIR(handle, path, mode);
@@ -682,7 +743,7 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
                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,
@@ -694,6 +755,10 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
                return status;
        }
 
+       psd->revision = orig_psd->revision;
+       /* All our SD's are self relative. */
+       psd->type = orig_psd->type | SEC_DESC_SELF_RELATIVE;
+
        if ((security_info_sent & SECINFO_OWNER) && (orig_psd->owner_sid != NULL)) {
                psd->owner_sid = orig_psd->owner_sid;
        }
@@ -702,9 +767,11 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
        }
        if (security_info_sent & SECINFO_DACL) {
                psd->dacl = orig_psd->dacl;
+               psd->type |= SEC_DESC_DACL_PRESENT;
        }
        if (security_info_sent & SECINFO_SACL) {
                psd->sacl = orig_psd->sacl;
+               psd->type |= SEC_DESC_SACL_PRESENT;
        }
 
        status = SMB_VFS_NEXT_FSET_NT_ACL(handle, fsp, security_info_sent, psd);
@@ -731,7 +798,7 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
                DEBUG(10,("fset_nt_acl_xattr: storing xattr sd for file %s\n",
                          fsp_str_dbg(fsp)));
                NDR_PRINT_DEBUG(security_descriptor,
-                       CONST_DISCARD(struct security_descriptor *,psd));
+                       discard_const_p(struct security_descriptor, psd));
        }
        create_acl_blob(psd, &blob, XATTR_SD_HASH_TYPE_SHA256, hash);
        store_acl_blob_fsp(handle, fsp, &blob);
@@ -764,6 +831,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)) {
@@ -776,14 +850,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);
@@ -830,7 +904,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;
        }
@@ -876,6 +952,7 @@ static NTSTATUS create_file_acl_common(struct vfs_handle_struct *handle,
        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,
@@ -894,6 +971,10 @@ static NTSTATUS create_file_acl_common(struct vfs_handle_struct *handle,
                                        result,
                                        &info);
 
+       if (!NT_STATUS_IS_OK(status)) {
+               goto out;
+       }
+
        if (info != FILE_WAS_CREATED) {
                /* File/directory was opened, not created. */
                goto out;
@@ -901,7 +982,7 @@ static NTSTATUS create_file_acl_common(struct vfs_handle_struct *handle,
 
        fsp = *result;
 
-       if (!NT_STATUS_IS_OK(status) || fsp == NULL) {
+       if (fsp == NULL) {
                /* Only handle success. */
                goto out;
        }
@@ -916,13 +997,19 @@ static NTSTATUS create_file_acl_common(struct vfs_handle_struct *handle,
                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);
+       /* 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 (!parent_sd) {
                goto err;
@@ -940,8 +1027,9 @@ static NTSTATUS create_file_acl_common(struct vfs_handle_struct *handle,
 
   out:
 
-       /* Ensure we never leave attached data around. */
-       SMB_VFS_HANDLE_FREE_DATA(handle);
+       if (fsp) {
+               VFS_REMOVE_FSP_EXTENSION(handle, fsp);
+       }
 
        if (NT_STATUS_IS_OK(status) && pinfo) {
                *pinfo = info;
@@ -976,3 +1064,43 @@ static int unlink_acl_common(struct vfs_handle_struct *handle,
                                        smb_fname->base_name,
                                        false);
 }
+
+static int chmod_acl_module_common(struct vfs_handle_struct *handle,
+                       const char *path, mode_t mode)
+{
+       if (lp_posix_pathnames()) {
+               /* Only allow this on POSIX pathnames. */
+               return SMB_VFS_NEXT_CHMOD(handle, path, mode);
+       }
+       return 0;
+}
+
+static int fchmod_acl_module_common(struct vfs_handle_struct *handle,
+                       struct files_struct *fsp, mode_t mode)
+{
+       if (fsp->posix_open) {
+               /* Only allow this on POSIX opens. */
+               return SMB_VFS_NEXT_FCHMOD(handle, fsp, mode);
+       }
+       return 0;
+}
+
+static int chmod_acl_acl_module_common(struct vfs_handle_struct *handle,
+                       const char *name, mode_t mode)
+{
+       if (lp_posix_pathnames()) {
+               /* Only allow this on POSIX pathnames. */
+               return SMB_VFS_NEXT_CHMOD_ACL(handle, name, mode);
+       }
+       return 0;
+}
+
+static int fchmod_acl_acl_module_common(struct vfs_handle_struct *handle,
+                       struct files_struct *fsp, mode_t mode)
+{
+       if (fsp->posix_open) {
+               /* Only allow this on POSIX opens. */
+               return SMB_VFS_NEXT_FCHMOD_ACL(handle, fsp, mode);
+       }
+       return 0;
+}