ctdb-scripts: Do not de-duplicate the interfaces list
[samba.git] / source3 / modules / vfs_acl_tdb.c
index 817add71fb9de0adba782e0dfba3f5d5010baa9b..bccb1ab0bec75b23e6059d94c417e68fe3f226f2 100644 (file)
 #include "smbd/smbd.h"
 #include "system/filesys.h"
 #include "librpc/gen_ndr/xattr.h"
-#include "librpc/gen_ndr/ndr_xattr.h"
-#include "../lib/crypto/sha256.h"
 #include "dbwrap/dbwrap.h"
 #include "dbwrap/dbwrap_open.h"
 #include "auth.h"
 #include "util_tdb.h"
+#include "vfs_acl_common.h"
 
 #undef DBGC_CLASS
 #define DBGC_CLASS DBGC_VFS
 
 #define ACL_MODULE_NAME "acl_tdb"
-#include "modules/vfs_acl_common.c"
 
 static unsigned int ref_count;
 static struct db_context *acl_db;
@@ -51,7 +49,7 @@ static bool acl_tdb_init(void)
                return true;
        }
 
-       dbname = state_path("file_ntacls.tdb");
+       dbname = state_path(talloc_tos(), "file_ntacls.tdb");
 
        if (dbname == NULL) {
                errno = ENOSYS;
@@ -91,24 +89,6 @@ static void disconnect_acl_tdb(struct vfs_handle_struct *handle)
        }
 }
 
-/*******************************************************************
- Fetch_lock the tdb acl record for a file
-*******************************************************************/
-
-static struct db_record *acl_tdb_lock(TALLOC_CTX *mem_ctx,
-                                       struct db_context *db,
-                                       const struct file_id *id)
-{
-       uint8_t id_buf[16];
-
-       /* For backwards compatibility only store the dev/inode. */
-       push_file_id_16((char *)id_buf, id);
-       return dbwrap_fetch_locked(db,
-                                  mem_ctx,
-                                  make_tdb_data(id_buf,
-                                                sizeof(id_buf)));
-}
-
 /*******************************************************************
  Delete the tdb acl record for a file
 *******************************************************************/
@@ -119,31 +99,22 @@ static NTSTATUS acl_tdb_delete(vfs_handle_struct *handle,
 {
        NTSTATUS status;
        struct file_id id = vfs_file_id_from_sbuf(handle->conn, psbuf);
-       struct db_record *rec = acl_tdb_lock(talloc_tos(), db, &id);
-
-       /*
-        * If rec == NULL there's not much we can do about it
-        */
+       uint8_t id_buf[16];
 
-       if (rec == NULL) {
-               DEBUG(10,("acl_tdb_delete: rec == NULL\n"));
-               TALLOC_FREE(rec);
-               return NT_STATUS_OK;
-       }
+       /* For backwards compatibility only store the dev/inode. */
+       push_file_id_16((char *)id_buf, &id);
 
-       status = dbwrap_record_delete(rec);
-       TALLOC_FREE(rec);
+       status = dbwrap_delete(db, make_tdb_data(id_buf, sizeof(id_buf)));
        return status;
 }
 
 /*******************************************************************
- Pull a security descriptor into a DATA_BLOB from a tdb store.
+ Pull a security descriptor from an fsp into a DATA_BLOB from a tdb store.
 *******************************************************************/
 
-static NTSTATUS get_acl_blob(TALLOC_CTX *ctx,
+static NTSTATUS fget_acl_blob(TALLOC_CTX *ctx,
                        vfs_handle_struct *handle,
                        files_struct *fsp,
-                       const struct smb_filename *smb_fname,
                        DATA_BLOB *pblob)
 {
        uint8_t id_buf[16];
@@ -151,27 +122,13 @@ static NTSTATUS get_acl_blob(TALLOC_CTX *ctx,
        struct file_id id;
        struct db_context *db = acl_db;
        NTSTATUS status = NT_STATUS_OK;
-       SMB_STRUCT_STAT sbuf;
-
-       ZERO_STRUCT(sbuf);
-
-       if (fsp) {
-               status = vfs_stat_fsp(fsp);
-               sbuf = fsp->fsp_name->st;
-       } else {
-               int ret = vfs_stat_smb_basename(handle->conn,
-                               smb_fname,
-                               &sbuf);
-               if (ret == -1) {
-                       status = map_nt_error_from_unix(errno);
-               }
-       }
 
+       status = vfs_stat_fsp(fsp);
        if (!NT_STATUS_IS_OK(status)) {
                return status;
        }
 
-       id = vfs_file_id_from_sbuf(handle->conn, &sbuf);
+       id = vfs_file_id_from_sbuf(handle->conn, &fsp->fsp_name->st);
 
        /* For backwards compatibility only store the dev/inode. */
        push_file_id_16((char *)id_buf, &id);
@@ -187,8 +144,9 @@ static NTSTATUS get_acl_blob(TALLOC_CTX *ctx,
        pblob->data = data.dptr;
        pblob->length = data.dsize;
 
-       DEBUG(10,("get_acl_blob: returned %u bytes from file %s\n",
-               (unsigned int)data.dsize, smb_fname->base_name ));
+       DBG_DEBUG("returned %u bytes from file %s\n",
+               (unsigned int)data.dsize,
+               fsp_str_dbg(fsp));
 
        if (pblob->length == 0 || pblob->data == NULL) {
                return NT_STATUS_NOT_FOUND;
@@ -206,9 +164,8 @@ static NTSTATUS store_acl_blob_fsp(vfs_handle_struct *handle,
 {
        uint8_t id_buf[16];
        struct file_id id;
-       TDB_DATA data;
+       TDB_DATA data = { .dptr = pblob->data, .dsize = pblob->length };
        struct db_context *db = acl_db;
-       struct db_record *rec;
        NTSTATUS status;
 
        DEBUG(10,("store_acl_blob_fsp: storing blob length %u on file %s\n",
@@ -223,24 +180,20 @@ static NTSTATUS store_acl_blob_fsp(vfs_handle_struct *handle,
 
        /* For backwards compatibility only store the dev/inode. */
        push_file_id_16((char *)id_buf, &id);
-       rec = dbwrap_fetch_locked(db, talloc_tos(),
-                                 make_tdb_data(id_buf,
-                                               sizeof(id_buf)));
-       if (rec == NULL) {
-               DEBUG(0, ("store_acl_blob_fsp_tdb: fetch_lock failed\n"));
-               return NT_STATUS_INTERNAL_DB_CORRUPTION;
-       }
-       data.dptr = pblob->data;
-       data.dsize = pblob->length;
-       return dbwrap_record_store(rec, data, 0);
+
+       status = dbwrap_store(
+               db, make_tdb_data(id_buf, sizeof(id_buf)), data, 0);
+       return status;
 }
 
 /*********************************************************************
- On unlink we need to delete the tdb record (if using tdb).
+ On unlinkat we need to delete the tdb record (if using tdb).
 *********************************************************************/
 
-static int unlink_acl_tdb(vfs_handle_struct *handle,
-                         const struct smb_filename *smb_fname)
+static int unlinkat_acl_tdb(vfs_handle_struct *handle,
+                       struct files_struct *dirfsp,
+                       const struct smb_filename *smb_fname,
+                       int flags)
 {
        struct smb_filename *smb_fname_tmp = NULL;
        struct db_context *db = acl_db;
@@ -252,17 +205,21 @@ static int unlink_acl_tdb(vfs_handle_struct *handle,
                goto out;
        }
 
-       if (smb_fname_tmp->flags & SMB_FILENAME_POSIX_PATH) {
-               ret = SMB_VFS_LSTAT(handle->conn, smb_fname_tmp);
-       } else {
-               ret = SMB_VFS_STAT(handle->conn, smb_fname_tmp);
-       }
-
+       ret = vfs_stat(handle->conn, smb_fname_tmp);
        if (ret == -1) {
                goto out;
        }
 
-       ret = unlink_acl_common(handle, smb_fname_tmp);
+       if (flags & AT_REMOVEDIR) {
+               ret = rmdir_acl_common(handle,
+                               dirfsp,
+                               smb_fname_tmp);
+       } else {
+               ret = unlink_acl_common(handle,
+                               dirfsp,
+                               smb_fname_tmp,
+                               flags);
+       }
 
        if (ret == -1) {
                goto out;
@@ -273,32 +230,6 @@ static int unlink_acl_tdb(vfs_handle_struct *handle,
        return ret;
 }
 
-/*********************************************************************
- On rmdir we need to delete the tdb record (if using tdb).
-*********************************************************************/
-
-static int rmdir_acl_tdb(vfs_handle_struct *handle,
-               const struct smb_filename *smb_fname)
-{
-
-       SMB_STRUCT_STAT sbuf;
-       struct db_context *db = acl_db;
-       int ret = -1;
-
-       ret = vfs_stat_smb_basename(handle->conn, smb_fname, &sbuf);
-       if (ret == -1) {
-               return -1;
-       }
-
-       ret = rmdir_acl_common(handle, smb_fname);
-       if (ret == -1) {
-               return -1;
-       }
-
-       acl_tdb_delete(handle, db, &sbuf);
-       return 0;
-}
-
 /*******************************************************************
  Handle opening the storage tdb if so configured.
 *******************************************************************/
@@ -320,7 +251,7 @@ static int connect_acl_tdb(struct vfs_handle_struct *handle,
                return -1;
        }
 
-       ok = init_acl_common_config(handle);
+       ok = init_acl_common_config(handle, ACL_MODULE_NAME);
        if (!ok) {
                DBG_ERR("init_acl_common_config failed\n");
                return -1;
@@ -382,55 +313,13 @@ static int connect_acl_tdb(struct vfs_handle_struct *handle,
  Remove a Windows ACL - we're setting the underlying POSIX ACL.
 *********************************************************************/
 
-static int sys_acl_set_file_tdb(vfs_handle_struct *handle,
-                       const struct smb_filename *smb_fname_in,
-                       SMB_ACL_TYPE_T type,
-                       SMB_ACL_T theacl)
-{
-       struct db_context *db = acl_db;
-       int ret = -1;
-       int saved_errno = 0;
-       struct smb_filename *smb_fname = NULL;
-
-       smb_fname = cp_smb_filename_nostream(talloc_tos(), smb_fname_in);
-       if (smb_fname == NULL) {
-               return -1;
-       };
-
-       ret = SMB_VFS_STAT(handle->conn, smb_fname);
-       if (ret == -1) {
-               saved_errno = errno;
-               goto fail;
-       }
-
-       ret = SMB_VFS_NEXT_SYS_ACL_SET_FILE(handle,
-                                               smb_fname,
-                                               type,
-                                               theacl);
-       if (ret == -1) {
-               saved_errno = errno;
-               goto fail;
-       }
-
-       acl_tdb_delete(handle, db, &smb_fname->st);
-
-fail:
-       TALLOC_FREE(smb_fname);
-
-       if (saved_errno != 0) {
-               errno = saved_errno;
-       }
-       return ret;
-}
-
-/*********************************************************************
- Remove a Windows ACL - we're setting the underlying POSIX ACL.
-*********************************************************************/
-
 static int sys_acl_set_fd_tdb(vfs_handle_struct *handle,
                             files_struct *fsp,
+                           SMB_ACL_TYPE_T type,
                             SMB_ACL_T theacl)
 {
+       struct acl_common_fsp_ext *ext = (struct acl_common_fsp_ext *)
+               VFS_FETCH_FSP_EXTENSION(handle, fsp);
        struct db_context *db = acl_db;
        NTSTATUS status;
        int ret;
@@ -441,29 +330,52 @@ static int sys_acl_set_fd_tdb(vfs_handle_struct *handle,
        }
 
        ret = SMB_VFS_NEXT_SYS_ACL_SET_FD(handle,
-                                               fsp,
-                                               theacl);
+                                         fsp,
+                                         type,
+                                         theacl);
        if (ret == -1) {
                return -1;
        }
 
+       if (ext != NULL && ext->setting_nt_acl) {
+               return 0;
+       }
+
        acl_tdb_delete(handle, db, &fsp->fsp_name->st);
        return 0;
 }
 
+static NTSTATUS acl_tdb_fget_nt_acl(vfs_handle_struct *handle,
+                                   files_struct *fsp,
+                                   uint32_t security_info,
+                                   TALLOC_CTX *mem_ctx,
+                                   struct security_descriptor **ppdesc)
+{
+       NTSTATUS status;
+       status = fget_nt_acl_common(fget_acl_blob, handle, fsp,
+                                  security_info, mem_ctx, ppdesc);
+       return status;
+}
+
+static NTSTATUS acl_tdb_fset_nt_acl(vfs_handle_struct *handle,
+                                   files_struct *fsp,
+                                   uint32_t security_info_sent,
+                                   const struct security_descriptor *psd)
+{
+       NTSTATUS status;
+       status = fset_nt_acl_common(fget_acl_blob, store_acl_blob_fsp,
+                                   ACL_MODULE_NAME,
+                                   handle, fsp, security_info_sent, psd);
+       return status;
+}
+
 static struct vfs_fn_pointers vfs_acl_tdb_fns = {
        .connect_fn = connect_acl_tdb,
        .disconnect_fn = disconnect_acl_tdb,
-       .rmdir_fn = rmdir_acl_tdb,
-       .unlink_fn = unlink_acl_tdb,
-       .chmod_fn = chmod_acl_module_common,
+       .unlinkat_fn = unlinkat_acl_tdb,
        .fchmod_fn = fchmod_acl_module_common,
-       .fget_nt_acl_fn = fget_nt_acl_common,
-       .get_nt_acl_fn = get_nt_acl_common,
-       .fset_nt_acl_fn = fset_nt_acl_common,
-       .chmod_acl_fn = chmod_acl_acl_module_common,
-       .fchmod_acl_fn = fchmod_acl_acl_module_common,
-       .sys_acl_set_file_fn = sys_acl_set_file_tdb,
+       .fget_nt_acl_fn = acl_tdb_fget_nt_acl,
+       .fset_nt_acl_fn = acl_tdb_fset_nt_acl,
        .sys_acl_set_fd_fn = sys_acl_set_fd_tdb
 };