Fix the overwriting of errno before use in a DEBUG statement and use the return value...
[ddiss/samba.git] / source3 / modules / vfs_acl_common.c
index 003be02b9c19b5ace5f691ef62edb3845a2c639c..bc9f56b798f33f394d8b8b2d27a68426e540b830 100644 (file)
@@ -19,6 +19,8 @@
  * 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"
 
@@ -250,7 +252,7 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
                                uint32_t security_info,
                                struct security_descriptor **ppdesc)
 {
-       DATA_BLOB blob;
+       DATA_BLOB blob = data_blob_null;
        NTSTATUS status;
        uint16_t hash_type = XATTR_SD_HASH_TYPE_NONE;
        uint8_t hash[XATTR_SD_HASH_SIZE];
@@ -376,7 +378,7 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
                                return map_nt_error_from_unix(errno);
                        }
                }
-               is_directory = S_ISDIR(sbuf.st_ex_mode);
+               is_directory = S_ISDIR(psbuf->st_ex_mode);
 
                if (ignore_file_system_acl) {
                        TALLOC_FREE(pdesc_next);
@@ -411,9 +413,11 @@ static NTSTATUS get_nt_acl_internal(vfs_handle_struct *handle,
                psd->group_sid = NULL;
        }
        if (!(security_info & SECINFO_DACL)) {
+               psd->type &= ~SEC_DESC_DACL_PRESENT;
                psd->dacl = NULL;
        }
        if (!(security_info & SECINFO_SACL)) {
+               psd->type &= ~SEC_DESC_SACL_PRESENT;
                psd->sacl = NULL;
        }
 
@@ -444,9 +448,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;
        }
 
@@ -458,38 +469,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->security_token->sids[PRIMARY_USER_SID_INDEX],
-                       &handle->conn->server_info->security_token->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)) {
@@ -501,16 +539,32 @@ static NTSTATUS check_parent_acl_common(vfs_handle_struct *handle,
                                        parent_name,
                                        (SECINFO_OWNER |
                                         SECINFO_GROUP |
-                                        SECINFO_DACL),
-                                       &parent_desc);
+                                        SECINFO_DACL  |
+                                        SECINFO_SACL),
+                                       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)
+{
+       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) {
@@ -523,9 +577,8 @@ static NTSTATUS check_parent_acl_common(vfs_handle_struct *handle,
                                        &access_granted);
        if(!NT_STATUS_IS_OK(status)) {
                DEBUG(10,("check_parent_acl_common: access check "
-                       "on directory %s for "
+                       "on parent directory of "
                        "path %s for mask 0x%x returned %s\n",
-                       parent_name,
                        path,
                        access_mask,
                        nt_errstr(status) ));
@@ -534,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.
 *********************************************************************/
@@ -551,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;
@@ -574,7 +621,8 @@ static int open_acl_common(vfs_handle_struct *handle,
                                fname,
                                (SECINFO_OWNER |
                                 SECINFO_GROUP |
-                                SECINFO_DACL),
+                                SECINFO_DACL  |
+                                SECINFO_SACL),
                                &pdesc);
         if (NT_STATUS_IS_OK(status)) {
                /* See if we can access it. */
@@ -585,8 +633,11 @@ static int open_acl_common(vfs_handle_struct *handle,
                                        &access_granted);
                if (!NT_STATUS_IS_OK(status)) {
                        DEBUG(10,("open_acl_xattr: %s open "
+                               "for access 0x%x (0x%x) "
                                "refused with error %s\n",
                                fsp_str_dbg(fsp),
+                               (unsigned int)fsp->access_mask,
+                               (unsigned int)access_granted,
                                nt_errstr(status) ));
                        goto err;
                }
@@ -597,29 +648,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;
                }
        }
@@ -646,30 +696,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);
@@ -771,22 +804,56 @@ static NTSTATUS fset_nt_acl_common(vfs_handle_struct *handle, files_struct *fsp,
                NDR_PRINT_DEBUG(security_descriptor,
                        CONST_DISCARD(struct security_descriptor *,psd));
        }
+       /*
+        * Perhaps create_acl_blob should have a status return as well
+        */
        create_acl_blob(psd, &blob, XATTR_SD_HASH_TYPE_SHA256, hash);
-       store_acl_blob_fsp(handle, fsp, &blob);
+       status = store_acl_blob_fsp(handle, fsp, &blob);
 
-       return NT_STATUS_OK;
+       return status;
 }
 
 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);
+       NTSTATUS status;
+       uint32_t access_granted = 0;
+       struct security_descriptor *sd = NULL;
 
+       status = get_nt_acl_internal(handle,
+                               NULL,
+                               fname,
+                               (SECINFO_OWNER |
+                                SECINFO_GROUP |
+                                SECINFO_DACL  |
+                                SECINFO_SACL),
+                               &sd);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(10,("opendir_acl_common: "
+                       "get_nt_acl_internal for dir %s "
+                       "failed with error %s\n",
+                       fname,
+                       nt_errstr(status) ));
+               errno = map_errno_from_nt_status(status);
+               return NULL;
+       }
+
+       /* See if we can access it. */
+       status = smb1_file_se_access_check(handle->conn,
+                               sd,
+                               get_current_nttok(handle->conn),
+                               SEC_DIR_LIST,
+                               &access_granted);
        if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(10,("opendir_acl_common: %s open "
+                       "for access SEC_DIR_LIST "
+                       "refused with error %s\n",
+                       fname,
+                       nt_errstr(status) ));
                errno = map_errno_from_nt_status(status);
                return NULL;
        }
+
        return SMB_VFS_NEXT_OPENDIR(handle, fname, mask, attr);
 }
 
@@ -802,6 +869,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)) {
@@ -814,7 +888,7 @@ 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;
@@ -833,7 +907,7 @@ static int acl_common_remove_object(vfs_handle_struct *handle,
        /* Ensure we have this file open with DELETE access. */
        id = vfs_file_id_from_sbuf(conn, &local_fname.st);
        for (fsp = file_find_di_first(conn->sconn, id); fsp;
-            file_find_di_next(fsp)) {
+                    fsp = file_find_di_next(fsp)) {
                if (fsp->access_mask & DELETE_ACCESS &&
                                fsp->delete_on_close) {
                        /* We did open this for delete,
@@ -868,7 +942,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;
        }
@@ -880,17 +956,23 @@ static int rmdir_acl_common(struct vfs_handle_struct *handle,
 {
        int ret;
 
+       /* Try the normal rmdir first. */
        ret = SMB_VFS_NEXT_RMDIR(handle, path);
-       if (!(ret == -1 && (errno == EACCES || errno == EPERM))) {
-               DEBUG(10,("rmdir_acl_common: unlink of %s failed %s\n",
-                       path,
-                       strerror(errno) ));
-               return ret;
+       if (ret == 0) {
+               return 0;
+       }
+       if (errno == EACCES || errno == EPERM) {
+               /* Failed due to access denied,
+                  see if we need to root override. */
+               return acl_common_remove_object(handle,
+                                               path,
+                                               true);
        }
 
-       return acl_common_remove_object(handle,
-                                       path,
-                                       true);
+       DEBUG(10,("rmdir_acl_common: unlink of %s failed %s\n",
+               path,
+               strerror(errno) ));
+       return -1;
 }
 
 static NTSTATUS create_file_acl_common(struct vfs_handle_struct *handle,
@@ -914,6 +996,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,
@@ -958,13 +1041,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;
@@ -982,8 +1071,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;
@@ -1002,19 +1092,66 @@ static int unlink_acl_common(struct vfs_handle_struct *handle,
 {
        int ret;
 
+       /* Try the normal unlink first. */
        ret = SMB_VFS_NEXT_UNLINK(handle, smb_fname);
-       if (!(ret == -1 && (errno == EACCES || errno == EPERM))) {
-               DEBUG(10,("unlink_acl_common: unlink of %s failed %s\n",
-                       smb_fname->base_name,
-                       strerror(errno) ));
-               return ret;
-       }
-       /* Don't do anything fancy for streams. */
-       if (smb_fname->stream_name) {
-               return ret;
+       if (ret == 0) {
+               return 0;
        }
+       if (errno == EACCES || errno == EPERM) {
+               /* Failed due to access denied,
+                  see if we need to root override. */
 
-       return acl_common_remove_object(handle,
+               /* Don't do anything fancy for streams. */
+               if (smb_fname->stream_name) {
+                       return -1;
+               }
+               return acl_common_remove_object(handle,
                                        smb_fname->base_name,
                                        false);
+       }
+
+       DEBUG(10,("unlink_acl_common: unlink of %s failed %s\n",
+               smb_fname->base_name,
+               strerror(errno) ));
+       return -1;
+}
+
+static int chmod_acl_module_common(struct vfs_handle_struct *handle,
+                       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;
 }