s4-ntvfs: Rename xattr_tdb.c to posix_eadb.c and make more generally useful

This is now a small library, to be called from ntvfs, python and
vfs_posix_eadb.  The rename makes it clear that this has a different
DB format to that used by vfs_xattr_tdb, and matches the posix:eadb
smb.conf parameter used to configure it.

Andrew Bartlett

diff --git a/source4/ntvfs/posix/xattr_tdb.c b/source4/ntvfs/posix/posix_eadb.c
similarity index 70%
rename from source4/ntvfs/posix/xattr_tdb.c
rename to source4/ntvfs/posix/posix_eadb.c
index 2a320ba7a6d41453aab2223719b4cfd2c7002608..3f5a5b8e309e591068f640568474448e71980112 100644 (file)
--- a/source4/ntvfs/posix/xattr_tdb.c
+++ b/source4/ntvfs/posix/posix_eadb.c
@@ -1,4 +1,4 @@
-/* 
+/*
    Unix SMB/CIFS implementation.
 
    POSIX NTVFS backend - xattr support using a tdb
@@ -9,12 +9,12 @@
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
-   
+
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU General Public License for more details.
-   
+
    You should have received a copy of the GNU General Public License
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
@@ -23,6 +23,7 @@
 #include "lib/tdb_wrap/tdb_wrap.h"
 #include "tdb_compat.h"
 #include "vfs_posix.h"
+#include "posix_eadb.h"
 
 #define XATTR_LIST_ATTR ".xattr_list"
 
@@ -30,7 +31,7 @@
   we need to maintain a list of attributes on each file, so that unlink
   can automatically clean them up
 */
-static NTSTATUS xattr_tdb_add_list(struct tdb_wrap *ea_tdb, TALLOC_CTX *ctx, const char *attr_name,
+static NTSTATUS posix_eadb_add_list(struct tdb_wrap *ea_tdb, TALLOC_CTX *ctx, const char *attr_name,
                                   const char *fname, int fd)
 {
        DATA_BLOB blob;
@@ -63,12 +64,12 @@ static NTSTATUS xattr_tdb_add_list(struct tdb_wrap *ea_tdb, TALLOC_CTX *ctx, con
        blob.data = talloc_realloc(mem_ctx, blob.data, uint8_t, blob.length + len);
        if (blob.data == NULL) {
                talloc_free(mem_ctx);
-               return NT_STATUS_NO_MEMORY;             
+               return NT_STATUS_NO_MEMORY;
        }
        memcpy(blob.data + blob.length, attr_name, len);
        blob.length += len;
 
-       status = push_xattr_blob_tdb_raw(ea_tdb,ctx, XATTR_LIST_ATTR, fname, fd, &blob);
+       status = push_xattr_blob_tdb_raw(ea_tdb, XATTR_LIST_ATTR, fname, fd, &blob);
        talloc_free(mem_ctx);
 
        return status;
@@ -79,7 +80,7 @@ static NTSTATUS xattr_tdb_add_list(struct tdb_wrap *ea_tdb, TALLOC_CTX *ctx, con
 */
 static NTSTATUS get_ea_tdb_key(TALLOC_CTX *mem_ctx,
                              const char *attr_name,
-                             const char *fname, int fd, 
+                             const char *fname, int fd,
                              TDB_DATA *key)
 {
        struct stat st;
@@ -104,7 +105,7 @@ static NTSTATUS get_ea_tdb_key(TALLOC_CTX *mem_ctx,
        SBVAL(key->dptr, 0, st.st_dev);
        SBVAL(key->dptr, 8, st.st_ino);
        memcpy(key->dptr+16, attr_name, len);
-       
+
        return NT_STATUS_OK;
 }
 
@@ -140,10 +141,10 @@ NTSTATUS pull_xattr_blob_tdb_raw(struct tdb_wrap *ea_tdb,
                return NT_STATUS_NO_MEMORY;
        }
 
-       return NT_STATUS_OK;    
+       return NT_STATUS_OK;
 }
 
-NTSTATUS pull_xattr_blob_tdb(struct pvfs_state *pvfs,
+NTSTATUS pull_xattr_blob_tdb(struct pvfs_state *pvfs_state,
                             TALLOC_CTX *mem_ctx,
                             const char *attr_name,
                             const char *fname,
@@ -151,14 +152,13 @@ NTSTATUS pull_xattr_blob_tdb(struct pvfs_state *pvfs,
                             size_t estimated_size,
                             DATA_BLOB *blob)
 {
-       return pull_xattr_blob_tdb_raw(pvfs->ea_db,mem_ctx,attr_name,fname,fd,estimated_size,blob);
+       return pull_xattr_blob_tdb_raw(pvfs_state->ea_db,mem_ctx,attr_name,fname,fd,estimated_size,blob);
 }
 
 /*
   push a xattr as a blob, using ea_tdb
 */
 NTSTATUS push_xattr_blob_tdb_raw(struct tdb_wrap *ea_tdb,
-                            TALLOC_CTX *mem_ctx,
                             const char *attr_name,
                             const char *fname,
                             int fd,
@@ -166,22 +166,28 @@ NTSTATUS push_xattr_blob_tdb_raw(struct tdb_wrap *ea_tdb,
 {
        TDB_DATA tkey, tdata;
        NTSTATUS status;
+       TALLOC_CTX *mem_ctx = talloc_new(ea_tdb);
+       if (!mem_ctx) {
+               return NT_STATUS_NO_MEMORY;
+       }
 
        status = get_ea_tdb_key(mem_ctx, attr_name, fname, fd, &tkey);
        if (!NT_STATUS_IS_OK(status)) {
+               talloc_free(mem_ctx);
                return status;
        }
-       
+
        tdata.dptr = blob->data;
        tdata.dsize = blob->length;
 
        if (tdb_chainlock(ea_tdb->tdb, tkey) != 0) {
-               talloc_free(tkey.dptr);
+               talloc_free(mem_ctx);
                return NT_STATUS_INTERNAL_DB_CORRUPTION;
        }
 
-       status = xattr_tdb_add_list(ea_tdb,mem_ctx, attr_name, fname, fd);
+       status = posix_eadb_add_list(ea_tdb,mem_ctx, attr_name, fname, fd);
        if (!NT_STATUS_IS_OK(status)) {
+               talloc_free(mem_ctx);
                goto done;
        }
 
@@ -191,23 +197,23 @@ NTSTATUS push_xattr_blob_tdb_raw(struct tdb_wrap *ea_tdb,
 
 done:
        tdb_chainunlock(ea_tdb->tdb, tkey);
-       talloc_free(tkey.dptr);
-       return status;  
+       talloc_free(mem_ctx);
+       return status;
 }
-NTSTATUS push_xattr_blob_tdb(struct pvfs_state *pvfs,
+NTSTATUS push_xattr_blob_tdb(struct pvfs_state *pvfs_state,
                             const char *attr_name,
                             const char *fname,
                             int fd,
                             const DATA_BLOB *blob)
 {
-       return push_xattr_blob_tdb_raw(pvfs->ea_db,pvfs,attr_name,fname,fd,blob);
+       return push_xattr_blob_tdb_raw(pvfs_state->ea_db, attr_name, fname, fd, blob);
 }
 
 
 /*
   delete a xattr
 */
-NTSTATUS delete_xattr_tdb(struct pvfs_state *pvfs, const char *attr_name, 
+NTSTATUS delete_posix_eadb_raw(struct tdb_wrap *ea_tdb, const char *attr_name,
                          const char *fname, int fd)
 {
        TDB_DATA tkey;
@@ -217,40 +223,69 @@ NTSTATUS delete_xattr_tdb(struct pvfs_state *pvfs, const char *attr_name,
        if (!NT_STATUS_IS_OK(status)) {
                return status;
        }
-       
-       if (tdb_delete(pvfs->ea_db->tdb, tkey) != 0) {
+
+       if (tdb_delete(ea_tdb->tdb, tkey) != 0) {
                talloc_free(tkey.dptr);
                return NT_STATUS_INTERNAL_DB_CORRUPTION;
        }
 
        talloc_free(tkey.dptr);
-       return NT_STATUS_OK;    
+       return NT_STATUS_OK;
 }
 
 
+/*
+  delete a xattr
+*/
+NTSTATUS delete_posix_eadb(struct pvfs_state *pvfs_state, const char *attr_name,
+                         const char *fname, int fd)
+{
+       return delete_posix_eadb_raw(pvfs_state->ea_db,
+                                   attr_name, fname, fd);
+}
+
 
 /*
   delete all xattrs for a file
 */
-NTSTATUS unlink_xattr_tdb(struct pvfs_state *pvfs, const char *fname)
+NTSTATUS unlink_posix_eadb_raw(struct tdb_wrap *ea_tdb, const char *fname, int fd)
 {
-       TALLOC_CTX *mem_ctx = talloc_new(pvfs);
+       TALLOC_CTX *mem_ctx = talloc_new(ea_tdb);
        DATA_BLOB blob;
        const char *s;
        NTSTATUS status;
 
-       status = pull_xattr_blob_tdb(pvfs, mem_ctx, XATTR_LIST_ATTR,
-                                    fname, -1, 100, &blob);
+       status = pull_xattr_blob_tdb_raw(ea_tdb, mem_ctx, XATTR_LIST_ATTR,
+                                    fname, fd, 100, &blob);
        if (!NT_STATUS_IS_OK(status)) {
                talloc_free(mem_ctx);
                return NT_STATUS_OK;
        }
 
        for (s=(const char *)blob.data; s < (const char *)(blob.data+blob.length); s += strlen(s) + 1) {
-               delete_xattr_tdb(pvfs, s, fname, -1);
+               delete_posix_eadb_raw(ea_tdb, s, fname, -1);
        }
 
-       status = delete_xattr_tdb(pvfs, XATTR_LIST_ATTR, fname, -1);
+       status = delete_posix_eadb_raw(ea_tdb, XATTR_LIST_ATTR, fname, fd);
        talloc_free(mem_ctx);
        return status;
 }
+
+/*
+  delete all xattrs for a file
+*/
+NTSTATUS unlink_posix_eadb(struct pvfs_state *pvfs_state, const char *fname)
+{
+       return unlink_posix_eadb_raw(pvfs_state->ea_db, fname, -1);
+}
+
+/*
+  list all xattrs for a file
+*/
+NTSTATUS list_posix_eadb_raw(struct tdb_wrap *ea_tdb, TALLOC_CTX *mem_ctx,
+                           const char *fname, int fd,
+                           DATA_BLOB *list)
+{
+       return pull_xattr_blob_tdb_raw(ea_tdb, mem_ctx, XATTR_LIST_ATTR,
+                                    fname, fd, 100, list);
+}

diff --git a/source4/ntvfs/posix/posix_eadb.h b/source4/ntvfs/posix/posix_eadb.h
new file mode 100644 (file)
index 0000000..752d322
--- /dev/null
+++ b/source4/ntvfs/posix/posix_eadb.h
@@ -0,0 +1,20 @@
+/*
+   Unix SMB/CIFS implementation. Xattr manipulation bindings.
+   Copyright (C) Andrew Bartlett 2011
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
+*/
+
+struct pvfs_state *pvfs;
+#include "source4/ntvfs/posix/posix_eadb_proto.h"

diff --git a/source4/ntvfs/posix/pvfs_xattr.c b/source4/ntvfs/posix/pvfs_xattr.c
index 04ad1786ac28112359b034736924b991ad899e66..82ce579676a6f6a7c969ccb8e672029bd0841f8f 100644 (file)
--- a/source4/ntvfs/posix/pvfs_xattr.c
+++ b/source4/ntvfs/posix/pvfs_xattr.c
@@ -24,6 +24,7 @@
 #include "../lib/util/unix_privs.h"
 #include "librpc/gen_ndr/ndr_xattr.h"
 #include "param/param.h"
+#include "ntvfs/posix/posix_eadb_proto.h"
 
 /*
   pull a xattr as a blob
@@ -81,7 +82,7 @@ static NTSTATUS delete_xattr(struct pvfs_state *pvfs, const char *attr_name,
                             const char *fname, int fd)
 {
        if (pvfs->ea_db) {
-               return delete_xattr_tdb(pvfs, attr_name, fname, fd);
+               return delete_posix_eadb(pvfs, attr_name, fname, fd);
        }
        return delete_xattr_system(pvfs, attr_name, fname, fd);
 }
@@ -92,7 +93,7 @@ static NTSTATUS delete_xattr(struct pvfs_state *pvfs, const char *attr_name,
 NTSTATUS pvfs_xattr_unlink_hook(struct pvfs_state *pvfs, const char *fname)
 {
        if (pvfs->ea_db) {
-               return unlink_xattr_tdb(pvfs, fname);
+               return unlink_posix_eadb(pvfs, fname);
        }
        return unlink_xattr_system(pvfs, fname);
 }

diff --git a/source4/ntvfs/posix/python/pyxattr_tdb.c b/source4/ntvfs/posix/python/pyxattr_tdb.c
index 1fdae7606d33d406ee7da9aa2a3ee4cbb813970f..d1be87951498b9a302082b1de4336de2e02a4be7 100644 (file)
--- a/source4/ntvfs/posix/python/pyxattr_tdb.c
+++ b/source4/ntvfs/posix/python/pyxattr_tdb.c
@@ -20,11 +20,12 @@
 
 #include <Python.h>
 #include "includes.h"
+#include "system/filesys.h"
 #include "tdb_compat.h"
 #include "lib/tdb_wrap/tdb_wrap.h"
 #include "librpc/ndr/libndr.h"
 #include "lib/util/wrap_xattr.h"
-#include "ntvfs/posix/vfs_posix.h"
+#include "ntvfs/posix/posix_eadb.h"
 #include "libcli/util/pyerrors.h"
 #include "param/pyparam.h"
 
@@ -59,8 +60,8 @@ static PyObject *py_wrap_setxattr(PyObject *self, PyObject *args)
                talloc_free(mem_ctx);
                return NULL;
        }
-       status = push_xattr_blob_tdb_raw(eadb, mem_ctx, attribute, filename, -1,
-                                                                        &blob);
+       status = push_xattr_blob_tdb_raw(eadb, attribute, filename, -1,
+                                        &blob);
        if (!NT_STATUS_IS_OK(status)) {
                PyErr_SetNTSTATUS(status);
                talloc_free(mem_ctx);

diff --git a/source4/ntvfs/posix/wscript_build b/source4/ntvfs/posix/wscript_build
index acceefadf69cf8037f2ce4ba2999709c24725cc1..8bd7648b822c26e8948307457ce53f254785d89c 100644 (file)
--- a/source4/ntvfs/posix/wscript_build
+++ b/source4/ntvfs/posix/wscript_build
@@ -31,11 +31,11 @@ bld.SAMBA_SUBSYSTEM('pvfs_aio',
 
 
 bld.SAMBA_MODULE('ntvfs_posix',
-       source='vfs_posix.c pvfs_util.c pvfs_search.c pvfs_dirlist.c pvfs_fileinfo.c pvfs_unlink.c pvfs_mkdir.c pvfs_open.c pvfs_read.c pvfs_flush.c pvfs_write.c pvfs_fsinfo.c pvfs_qfileinfo.c pvfs_setfileinfo.c pvfs_rename.c pvfs_resolve.c pvfs_shortname.c pvfs_lock.c pvfs_oplock.c pvfs_wait.c pvfs_seek.c pvfs_ioctl.c pvfs_xattr.c pvfs_streams.c pvfs_notify.c pvfs_sys.c xattr_system.c xattr_tdb.c',
+       source='vfs_posix.c pvfs_util.c pvfs_search.c pvfs_dirlist.c pvfs_fileinfo.c pvfs_unlink.c pvfs_mkdir.c pvfs_open.c pvfs_read.c pvfs_flush.c pvfs_write.c pvfs_fsinfo.c pvfs_qfileinfo.c pvfs_setfileinfo.c pvfs_rename.c pvfs_resolve.c pvfs_shortname.c pvfs_lock.c pvfs_oplock.c pvfs_wait.c pvfs_seek.c pvfs_ioctl.c pvfs_xattr.c pvfs_streams.c pvfs_notify.c pvfs_sys.c xattr_system.c',
        autoproto='vfs_posix_proto.h',
        subsystem='ntvfs',
        init_function='ntvfs_posix_init',
-       deps='NDR_XATTR wrap_xattr ntvfs_common MESSAGING LIBWBCLIENT_OLD pvfs_acl pvfs_aio',
+       deps='NDR_XATTR wrap_xattr ntvfs_common MESSAGING LIBWBCLIENT_OLD pvfs_acl pvfs_aio posix_eadb',
        internal_module=True
        )
 
@@ -46,10 +46,15 @@ bld.SAMBA_PYTHON('python_xattr_native',
        realname='samba/xattr_native.so'
        )
 
+bld.SAMBA_LIBRARY('posix_eadb',
+                  source='posix_eadb.c',
+                  deps='tdb tdb-wrap',
+                  autoproto='posix_eadb_proto.h',
+                private_library=True)
 
 bld.SAMBA_PYTHON('python_xattr_tdb',
-       source='python/pyxattr_tdb.c xattr_tdb.c',
-       deps='ndr ldb pyparam_util share attr',
+       source='python/pyxattr_tdb.c',
+       deps='pyparam_util posix_eadb',
        realname='samba/xattr_tdb.so'
        )