#include "dbwrap/dbwrap.h"
#include "dbwrap/dbwrap_open.h"
#include "source3/lib/xattr_tdb.h"
+#include "lib/util/tevent_unix.h"
#undef DBGC_CLASS
#define DBGC_CLASS DBGC_VFS
-static bool xattr_tdb_init(int snum, TALLOC_CTX *mem_ctx, struct db_context **p_db);
+struct xattr_tdb_config {
+ struct db_context *db;
+ bool ignore_user_xattr;
+};
+
+static bool xattr_tdb_init(struct vfs_handle_struct *handle,
+ struct xattr_tdb_config **_config);
+
+static bool is_user_xattr(const char *xattr_name)
+{
+ int match;
+
+ match = strncmp(xattr_name, "user.", strlen("user."));
+ return (match == 0);
+}
static int xattr_tdb_get_file_id(struct vfs_handle_struct *handle,
const char *path, struct file_id *id)
TALLOC_CTX *frame = talloc_stackframe();
struct smb_filename *smb_fname;
- smb_fname = synthetic_smb_fname(frame, path, NULL, NULL, 0);
+ smb_fname = synthetic_smb_fname(frame,
+ path,
+ NULL,
+ NULL,
+ 0,
+ 0);
if (smb_fname == NULL) {
TALLOC_FREE(frame);
errno = ENOMEM;
return 0;
}
-static ssize_t xattr_tdb_getxattr(struct vfs_handle_struct *handle,
- const char *path, const char *name,
- void *value, size_t size)
+struct xattr_tdb_getxattrat_state {
+ struct vfs_aio_state vfs_aio_state;
+ ssize_t xattr_size;
+ uint8_t *xattr_value;
+};
+
+static void xattr_tdb_getxattrat_done(struct tevent_req *subreq);
+
+static struct tevent_req *xattr_tdb_getxattrat_send(
+ TALLOC_CTX *mem_ctx,
+ struct tevent_context *ev,
+ struct vfs_handle_struct *handle,
+ files_struct *dir_fsp,
+ const struct smb_filename *smb_fname,
+ const char *xattr_name,
+ size_t alloc_hint)
{
+ struct xattr_tdb_config *config = NULL;
+ struct tevent_req *req = NULL;
+ struct tevent_req *subreq = NULL;
+ struct xattr_tdb_getxattrat_state *state = NULL;
+ struct smb_filename *cwd = NULL;
struct file_id id;
- struct db_context *db;
- ssize_t xattr_size;
int ret;
- DATA_BLOB blob;
- TALLOC_CTX *frame = talloc_stackframe();
+ int error;
+ int cwd_ret;
+ DATA_BLOB xattr_blob;
+
+ if (!xattr_tdb_init(handle, &config)) {
+ return NULL;
+ }
+
+ req = tevent_req_create(mem_ctx, &state,
+ struct xattr_tdb_getxattrat_state);
+ if (req == NULL) {
+ return NULL;
+ }
+ state->xattr_size = -1;
+
+ if (config->ignore_user_xattr && is_user_xattr(xattr_name)) {
+ subreq = SMB_VFS_NEXT_GETXATTRAT_SEND(state,
+ ev,
+ handle,
+ dir_fsp,
+ smb_fname,
+ xattr_name,
+ alloc_hint);
+ if (tevent_req_nomem(subreq, req)) {
+ return tevent_req_post(req, ev);
+ }
+ tevent_req_set_callback(subreq, xattr_tdb_getxattrat_done, req);
+ return req;
+ }
+
+ cwd = SMB_VFS_GETWD(dir_fsp->conn, state);
+ if (tevent_req_nomem(cwd, req)) {
+ return tevent_req_post(req, ev);
+ }
+
+ ret = SMB_VFS_CHDIR(dir_fsp->conn, dir_fsp->fsp_name);
+ if (ret != 0) {
+ tevent_req_error(req, errno);
+ return tevent_req_post(req, ev);
+ }
+
+ ret = xattr_tdb_get_file_id(handle, smb_fname->base_name, &id);
+ error = errno;
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
+ cwd_ret = SMB_VFS_CHDIR(dir_fsp->conn, cwd);
+ SMB_ASSERT(cwd_ret == 0);
- ret = xattr_tdb_get_file_id(handle, path, &id);
if (ret == -1) {
- TALLOC_FREE(frame);
- return -1;
+ tevent_req_error(req, error);
+ return tevent_req_post(req, ev);
}
- xattr_size = xattr_tdb_getattr(db, frame, &id, name, &blob);
- if (xattr_size < 0) {
- errno = ENOATTR;
- TALLOC_FREE(frame);
- return -1;
+ state->xattr_size = xattr_tdb_getattr(config->db,
+ state,
+ &id,
+ xattr_name,
+ &xattr_blob);
+ if (state->xattr_size == -1) {
+ tevent_req_error(req, errno);
+ return tevent_req_post(req, ev);
}
- if (size == 0) {
- TALLOC_FREE(frame);
- return xattr_size;
+ if (alloc_hint == 0) {
+ /*
+ * The caller only wants to know the size.
+ */
+ tevent_req_done(req);
+ return tevent_req_post(req, ev);
}
- if (blob.length > size) {
- TALLOC_FREE(frame);
- errno = ERANGE;
+ if (state->xattr_size == 0) {
+ /*
+ * There's no data.
+ */
+ tevent_req_done(req);
+ return tevent_req_post(req, ev);
+ }
+
+ if (xattr_blob.length > alloc_hint) {
+ /*
+ * The data doesn't fit.
+ */
+ state->xattr_size = -1;
+ tevent_req_error(req, ERANGE);
+ return tevent_req_post(req, ev);
+ }
+
+ /*
+ * take the whole blob.
+ */
+ state->xattr_value = xattr_blob.data;
+
+ tevent_req_done(req);
+ return tevent_req_post(req, ev);
+}
+
+static void xattr_tdb_getxattrat_done(struct tevent_req *subreq)
+{
+ struct tevent_req *req = tevent_req_callback_data(
+ subreq, struct tevent_req);
+ struct xattr_tdb_getxattrat_state *state = tevent_req_data(
+ req, struct xattr_tdb_getxattrat_state);
+
+ state->xattr_size = SMB_VFS_NEXT_GETXATTRAT_RECV(subreq,
+ &state->vfs_aio_state,
+ state,
+ &state->xattr_value);
+ TALLOC_FREE(subreq);
+ if (state->xattr_size == -1) {
+ tevent_req_error(req, state->vfs_aio_state.error);
+ return;
+ }
+
+ tevent_req_done(req);
+}
+
+
+static ssize_t xattr_tdb_getxattrat_recv(struct tevent_req *req,
+ struct vfs_aio_state *aio_state,
+ TALLOC_CTX *mem_ctx,
+ uint8_t **xattr_value)
+{
+ struct xattr_tdb_getxattrat_state *state = tevent_req_data(
+ req, struct xattr_tdb_getxattrat_state);
+ ssize_t xattr_size;
+
+ if (tevent_req_is_unix_error(req, &aio_state->error)) {
+ tevent_req_received(req);
return -1;
}
- memcpy(value, blob.data, xattr_size);
- TALLOC_FREE(frame);
+
+ *aio_state = state->vfs_aio_state;
+ xattr_size = state->xattr_size;
+ if (xattr_value != NULL) {
+ *xattr_value = talloc_move(mem_ctx, &state->xattr_value);
+ }
+
+ tevent_req_received(req);
return xattr_size;
}
struct files_struct *fsp,
const char *name, void *value, size_t size)
{
+ struct xattr_tdb_config *config = NULL;
SMB_STRUCT_STAT sbuf;
struct file_id id;
- struct db_context *db;
ssize_t xattr_size;
DATA_BLOB blob;
- TALLOC_CTX *frame = talloc_stackframe();
+ TALLOC_CTX *frame = NULL;
+
+ if (!xattr_tdb_init(handle, &config)) {
+ return -1;
+ }
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
+ if (config->ignore_user_xattr && is_user_xattr(name)) {
+ return SMB_VFS_NEXT_FGETXATTR(
+ handle, fsp, name, value, size);
+ }
if (SMB_VFS_NEXT_FSTAT(handle, fsp, &sbuf) == -1) {
- TALLOC_FREE(frame);
return -1;
}
+ frame = talloc_stackframe();
+
id = SMB_VFS_NEXT_FILE_ID_CREATE(handle, &sbuf);
- xattr_size = xattr_tdb_getattr(db, frame, &id, name, &blob);
+ xattr_size = xattr_tdb_getattr(config->db, frame, &id, name, &blob);
if (xattr_size < 0) {
errno = ENOATTR;
TALLOC_FREE(frame);
return xattr_size;
}
-static int xattr_tdb_setxattr(struct vfs_handle_struct *handle,
- const char *path, const char *name,
- const void *value, size_t size, int flags)
-{
- struct file_id id;
- struct db_context *db;
- int ret;
- TALLOC_CTX *frame = talloc_stackframe();
-
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
-
- ret = xattr_tdb_get_file_id(handle, path, &id);
- if (ret == -1) {
- TALLOC_FREE(frame);
- return -1;
- }
-
- ret = xattr_tdb_setattr(db, &id, name, value, size, flags);
- TALLOC_FREE(frame);
- return ret;
-}
-
static int xattr_tdb_fsetxattr(struct vfs_handle_struct *handle,
struct files_struct *fsp,
const char *name, const void *value,
size_t size, int flags)
{
+ struct xattr_tdb_config *config = NULL;
SMB_STRUCT_STAT sbuf;
struct file_id id;
- struct db_context *db;
int ret;
- TALLOC_CTX *frame = talloc_stackframe();
-
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
- if (SMB_VFS_NEXT_FSTAT(handle, fsp, &sbuf) == -1) {
- TALLOC_FREE(frame);
+ if (!xattr_tdb_init(handle, &config)) {
return -1;
}
- id = SMB_VFS_NEXT_FILE_ID_CREATE(handle, &sbuf);
-
- ret = xattr_tdb_setattr(db, &id, name, value, size, flags);
- TALLOC_FREE(frame);
- return ret;
-
-}
-
-static ssize_t xattr_tdb_listxattr(struct vfs_handle_struct *handle,
- const char *path, char *list, size_t size)
-{
- struct file_id id;
- struct db_context *db;
- int ret;
- TALLOC_CTX *frame = talloc_stackframe();
-
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
+ if (config->ignore_user_xattr && is_user_xattr(name)) {
+ return SMB_VFS_NEXT_FSETXATTR(
+ handle, fsp, name, value, size, flags);
+ }
- ret = xattr_tdb_get_file_id(handle, path, &id);
- if (ret == -1) {
- TALLOC_FREE(frame);
+ if (SMB_VFS_NEXT_FSTAT(handle, fsp, &sbuf) == -1) {
return -1;
}
- ret = xattr_tdb_listattr(db, &id, list, size);
- TALLOC_FREE(frame);
+ id = SMB_VFS_NEXT_FILE_ID_CREATE(handle, &sbuf);
+
+ ret = xattr_tdb_setattr(config->db, &id, name, value, size, flags);
return ret;
}
struct files_struct *fsp, char *list,
size_t size)
{
+ struct xattr_tdb_config *config = NULL;
SMB_STRUCT_STAT sbuf;
struct file_id id;
- struct db_context *db;
- int ret;
- TALLOC_CTX *frame = talloc_stackframe();
+ ssize_t backend_size;
+ ssize_t ret;
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
+ if (!xattr_tdb_init(handle, &config)) {
+ return -1;
+ }
if (SMB_VFS_NEXT_FSTAT(handle, fsp, &sbuf) == -1) {
- TALLOC_FREE(frame);
return -1;
}
id = SMB_VFS_NEXT_FILE_ID_CREATE(handle, &sbuf);
- ret = xattr_tdb_listattr(db, &id, list, size);
- TALLOC_FREE(frame);
- return ret;
-}
-
-static int xattr_tdb_removexattr(struct vfs_handle_struct *handle,
- const char *path, const char *name)
-{
- struct file_id id;
- struct db_context *db;
- int ret;
- TALLOC_CTX *frame = talloc_stackframe();
-
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
-
- ret = xattr_tdb_get_file_id(handle, path, &id);
+ ret = xattr_tdb_listattr(config->db, &id, list, size);
if (ret == -1) {
- TALLOC_FREE(frame);
+ return -1;
+ }
+ if (ret == size) {
+ return ret;
+ }
+ if (!config->ignore_user_xattr) {
return ret;
}
+ SMB_ASSERT(ret < size);
-
- ret = xattr_tdb_removeattr(db, &id, name);
- TALLOC_FREE(frame);
- return ret;
+ backend_size = SMB_VFS_NEXT_FLISTXATTR(
+ handle, fsp, list + ret, size - ret);
+ if (backend_size == -1) {
+ return -1;
+ }
+
+ return ret + backend_size;
}
static int xattr_tdb_fremovexattr(struct vfs_handle_struct *handle,
struct files_struct *fsp, const char *name)
{
+ struct xattr_tdb_config *config = NULL;
SMB_STRUCT_STAT sbuf;
struct file_id id;
- struct db_context *db;
- int ret;
- TALLOC_CTX *frame = talloc_stackframe();
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
+ if (!xattr_tdb_init(handle, &config)) {
+ return -1;
+ }
+
+ if (config->ignore_user_xattr && is_user_xattr(name)) {
+ return SMB_VFS_NEXT_FREMOVEXATTR(handle, fsp, name);
+ }
if (SMB_VFS_NEXT_FSTAT(handle, fsp, &sbuf) == -1) {
- TALLOC_FREE(frame);
return -1;
}
id = SMB_VFS_NEXT_FILE_ID_CREATE(handle, &sbuf);
- ret = xattr_tdb_removeattr(db, &id, name);
- TALLOC_FREE(frame);
- return ret;
+ return xattr_tdb_removeattr(config->db, &id, name);
+}
+
+/*
+ * Destructor for the VFS private data
+ */
+
+static void config_destructor(void **data)
+{
+ struct xattr_tdb_config **config = (struct xattr_tdb_config **)data;
+ TALLOC_FREE((*config)->db);
}
/*
* Open the tdb file upon VFS_CONNECT
*/
-static bool xattr_tdb_init(int snum, TALLOC_CTX *mem_ctx, struct db_context **p_db)
+static bool xattr_tdb_init(struct vfs_handle_struct *handle,
+ struct xattr_tdb_config **_config)
{
- struct db_context *db;
+ struct xattr_tdb_config *config = NULL;
const char *dbname;
char *def_dbname;
- def_dbname = state_path("xattr.tdb");
+ if (SMB_VFS_HANDLE_TEST_DATA(handle)) {
+ SMB_VFS_HANDLE_GET_DATA(handle, config, struct xattr_tdb_config,
+ return false);
+ if (_config != NULL) {
+ *_config = config;
+ }
+ return true;
+ }
+
+ config = talloc_zero(handle->conn, struct xattr_tdb_config);
+ if (config == NULL) {
+ errno = ENOMEM;
+ goto error;
+ }
+
+ def_dbname = state_path(talloc_tos(), "xattr.tdb");
if (def_dbname == NULL) {
errno = ENOSYS;
- return false;
+ goto error;
}
- dbname = lp_parm_const_string(snum, "xattr_tdb", "file", def_dbname);
+ dbname = lp_parm_const_string(SNUM(handle->conn),
+ "xattr_tdb",
+ "file",
+ def_dbname);
/* now we know dbname is not NULL */
become_root();
- db = db_open(NULL, dbname, 0, TDB_DEFAULT, O_RDWR|O_CREAT, 0600,
- DBWRAP_LOCK_ORDER_2, DBWRAP_FLAG_NONE);
+ config->db = db_open(handle, dbname, 0, TDB_DEFAULT, O_RDWR|O_CREAT, 0600,
+ DBWRAP_LOCK_ORDER_2, DBWRAP_FLAG_NONE);
unbecome_root();
- if (db == NULL) {
+ if (config->db == NULL) {
#if defined(ENOTSUP)
errno = ENOTSUP;
#else
errno = ENOSYS;
#endif
TALLOC_FREE(def_dbname);
- return false;
+ goto error;
}
-
- *p_db = db;
TALLOC_FREE(def_dbname);
+
+ config->ignore_user_xattr = lp_parm_bool(
+ SNUM(handle->conn), "xattr_tdb", "ignore_user_xattr", false);
+
+ SMB_VFS_HANDLE_SET_DATA(handle, config, config_destructor,
+ struct xattr_tdb_config, return false);
+
+ if (_config != NULL) {
+ *_config = config;
+ }
return true;
+
+error:
+ DBG_WARNING("Failed to initialize config: %s\n", strerror(errno));
+ lp_do_parameter(SNUM(handle->conn), "ea support", "False");
+ return false;
}
-static int xattr_tdb_open(vfs_handle_struct *handle,
- struct smb_filename *smb_fname,
- files_struct *fsp,
- int flags,
- mode_t mode)
+static int xattr_tdb_openat(struct vfs_handle_struct *handle,
+ const struct files_struct *dirfsp,
+ const struct smb_filename *smb_fname,
+ struct files_struct *fsp,
+ const struct vfs_open_how *how)
{
- struct db_context *db = NULL;
- TALLOC_CTX *frame = NULL;
+ struct xattr_tdb_config *config = NULL;
+ SMB_STRUCT_STAT sbuf;
+ int fd;
int ret;
- fsp->fh->fd = SMB_VFS_NEXT_OPEN(handle,
- smb_fname, fsp,
- flags,
- mode);
+ if (!xattr_tdb_init(handle, &config)) {
+ return -1;
+ }
- if (fsp->fh->fd < 0) {
- return fsp->fh->fd;
+ fd = SMB_VFS_NEXT_OPENAT(handle,
+ dirfsp,
+ smb_fname,
+ fsp,
+ how);
+ if (fd == -1) {
+ return -1;
}
- if ((flags & (O_CREAT|O_EXCL)) != (O_CREAT|O_EXCL)) {
- return fsp->fh->fd;
+ if ((how->flags & (O_CREAT|O_EXCL)) != (O_CREAT|O_EXCL)) {
+ return fd;
}
/*
* We must have created the file.
*/
- ret = SMB_VFS_FSTAT(fsp, &smb_fname->st);
+ fsp_set_fd(fsp, fd);
+ ret = SMB_VFS_FSTAT(fsp, &sbuf);
+ fsp_set_fd(fsp, -1);
if (ret == -1) {
/* Can't happen... */
DBG_WARNING("SMB_VFS_FSTAT failed on file %s (%s)\n",
- smb_fname_str_dbg(smb_fname),
- strerror(errno));
+ smb_fname_str_dbg(smb_fname),
+ strerror(errno));
return -1;
}
- fsp->file_id = SMB_VFS_FILE_ID_CREATE(fsp->conn, &smb_fname->st);
- frame = talloc_stackframe();
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
+ fsp->file_id = SMB_VFS_FILE_ID_CREATE(fsp->conn, &sbuf);
- xattr_tdb_remove_all_attrs(db, &fsp->file_id);
- TALLOC_FREE(frame);
- return fsp->fh->fd;
+ xattr_tdb_remove_all_attrs(config->db, &fsp->file_id);
+
+ return fd;
}
-static int xattr_tdb_mkdir(vfs_handle_struct *handle,
+static int xattr_tdb_mkdirat(vfs_handle_struct *handle,
+ struct files_struct *dirfsp,
const struct smb_filename *smb_fname,
mode_t mode)
{
- struct db_context *db = NULL;
- TALLOC_CTX *frame = NULL;
+ struct xattr_tdb_config *config = NULL;
struct file_id fileid;
+ struct stat_ex sbuf = { .st_ex_nlink = 0, };
int ret;
- struct smb_filename *smb_fname_tmp = NULL;
- ret = SMB_VFS_NEXT_MKDIR(handle, smb_fname, mode);
+ if (!xattr_tdb_init(handle, &config)) {
+ return -1;
+ }
+
+ ret = SMB_VFS_NEXT_MKDIRAT(handle,
+ dirfsp,
+ smb_fname,
+ mode);
if (ret < 0) {
return ret;
}
- frame = talloc_stackframe();
- smb_fname_tmp = cp_smb_filename(frame, smb_fname);
- if (smb_fname_tmp == NULL) {
- TALLOC_FREE(frame);
- errno = ENOMEM;
- return -1;
- }
+ ret = SMB_VFS_NEXT_FSTATAT(
+ handle, dirfsp, smb_fname, &sbuf, AT_SYMLINK_NOFOLLOW);
- /* Always use LSTAT here - we just creaded the directory. */
- ret = SMB_VFS_LSTAT(handle->conn, smb_fname_tmp);
if (ret == -1) {
/* Rename race. Let upper level take care of it. */
- TALLOC_FREE(frame);
return -1;
}
- if (!S_ISDIR(smb_fname_tmp->st.st_ex_mode)) {
+ if (!S_ISDIR(sbuf.st_ex_mode)) {
/* Rename race. Let upper level take care of it. */
- TALLOC_FREE(frame);
return -1;
}
- fileid = SMB_VFS_FILE_ID_CREATE(handle->conn, &smb_fname_tmp->st);
-
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
+ fileid = SMB_VFS_FILE_ID_CREATE(handle->conn, &sbuf);
- xattr_tdb_remove_all_attrs(db, &fileid);
- TALLOC_FREE(frame);
+ xattr_tdb_remove_all_attrs(config->db, &fileid);
return 0;
}
/*
* On unlink we need to delete the tdb record
*/
-static int xattr_tdb_unlink(vfs_handle_struct *handle,
- const struct smb_filename *smb_fname)
+static int xattr_tdb_unlinkat(vfs_handle_struct *handle,
+ struct files_struct *dirfsp,
+ const struct smb_filename *smb_fname,
+ int flags)
{
+ struct xattr_tdb_config *config = NULL;
struct smb_filename *smb_fname_tmp = NULL;
+ struct smb_filename *full_fname = NULL;
struct file_id id;
- struct db_context *db;
int ret = -1;
bool remove_record = false;
- TALLOC_CTX *frame = talloc_stackframe();
+ TALLOC_CTX *frame = NULL;
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
+ if (!xattr_tdb_init(handle, &config)) {
+ return -1;
+ }
+
+ frame = talloc_stackframe();
smb_fname_tmp = cp_smb_filename(frame, smb_fname);
if (smb_fname_tmp == NULL) {
return -1;
}
- if (smb_fname_tmp->flags & SMB_FILENAME_POSIX_PATH) {
- ret = SMB_VFS_NEXT_LSTAT(handle, smb_fname_tmp);
+ /*
+ * TODO: use SMB_VFS_STATX() once we have that
+ */
+
+ full_fname = full_path_from_dirfsp_atname(frame,
+ dirfsp,
+ smb_fname);
+ if (full_fname == NULL) {
+ goto out;
+ }
+
+ if (full_fname->flags & SMB_FILENAME_POSIX_PATH) {
+ ret = SMB_VFS_NEXT_LSTAT(handle, full_fname);
} else {
- ret = SMB_VFS_NEXT_STAT(handle, smb_fname_tmp);
+ ret = SMB_VFS_NEXT_STAT(handle, full_fname);
+ if (ret == -1 && (errno == ENOENT || errno == ELOOP)) {
+ if (VALID_STAT(smb_fname->st) &&
+ S_ISLNK(smb_fname->st.st_ex_mode)) {
+ /*
+ * Original name was a link - Could be
+ * trying to remove a dangling symlink.
+ */
+ ret = SMB_VFS_NEXT_LSTAT(handle, full_fname);
+ }
+ }
}
if (ret == -1) {
goto out;
}
+ smb_fname_tmp->st = full_fname->st;
- if (smb_fname_tmp->st.st_ex_nlink == 1) {
- /* Only remove record on last link to file. */
+ if (flags & AT_REMOVEDIR) {
+ /* Always remove record when removing a directory succeeds. */
remove_record = true;
+ } else {
+ if (smb_fname_tmp->st.st_ex_nlink == 1) {
+ /* Only remove record on last link to file. */
+ remove_record = true;
+ }
}
- ret = SMB_VFS_NEXT_UNLINK(handle, smb_fname_tmp);
+ ret = SMB_VFS_NEXT_UNLINKAT(handle,
+ dirfsp,
+ smb_fname_tmp,
+ flags);
if (ret == -1) {
goto out;
id = SMB_VFS_NEXT_FILE_ID_CREATE(handle, &smb_fname_tmp->st);
- xattr_tdb_remove_all_attrs(db, &id);
+ xattr_tdb_remove_all_attrs(config->db, &id);
out:
TALLOC_FREE(frame);
return ret;
}
-/*
- * On rmdir we need to delete the tdb record
- */
-static int xattr_tdb_rmdir(vfs_handle_struct *handle,
- const struct smb_filename *smb_fname)
-{
- SMB_STRUCT_STAT sbuf;
- struct file_id id;
- struct db_context *db;
- int ret;
- TALLOC_CTX *frame = talloc_stackframe();
-
- SMB_VFS_HANDLE_GET_DATA(handle, db, struct db_context,
- if (!xattr_tdb_init(-1, frame, &db))
- {
- TALLOC_FREE(frame); return -1;
- });
-
- if (vfs_stat_smb_basename(handle->conn,
- smb_fname,
- &sbuf) == -1) {
- TALLOC_FREE(frame);
- return -1;
- }
-
- ret = SMB_VFS_NEXT_RMDIR(handle, smb_fname);
-
- if (ret == -1) {
- TALLOC_FREE(frame);
- return -1;
- }
-
- id = SMB_VFS_NEXT_FILE_ID_CREATE(handle, &sbuf);
-
- xattr_tdb_remove_all_attrs(db, &id);
-
- TALLOC_FREE(frame);
- return 0;
-}
-
-/*
- * Destructor for the VFS private data
- */
-
-static void close_xattr_db(void **data)
-{
- struct db_context **p_db = (struct db_context **)data;
- TALLOC_FREE(*p_db);
-}
-
static int xattr_tdb_connect(vfs_handle_struct *handle, const char *service,
const char *user)
{
char *sname = NULL;
int res, snum;
- struct db_context *db;
res = SMB_VFS_NEXT_CONNECT(handle, service, user);
if (res < 0) {
return 0;
}
- if (!xattr_tdb_init(snum, NULL, &db)) {
+ if (!xattr_tdb_init(handle, NULL)) {
DEBUG(5, ("Could not init xattr tdb\n"));
lp_do_parameter(snum, "ea support", "False");
return 0;
lp_do_parameter(snum, "ea support", "True");
- SMB_VFS_HANDLE_SET_DATA(handle, db, close_xattr_db,
- struct db_context, return -1);
-
return 0;
}
static struct vfs_fn_pointers vfs_xattr_tdb_fns = {
- .getxattr_fn = xattr_tdb_getxattr,
+ .getxattrat_send_fn = xattr_tdb_getxattrat_send,
+ .getxattrat_recv_fn = xattr_tdb_getxattrat_recv,
.fgetxattr_fn = xattr_tdb_fgetxattr,
- .setxattr_fn = xattr_tdb_setxattr,
.fsetxattr_fn = xattr_tdb_fsetxattr,
- .listxattr_fn = xattr_tdb_listxattr,
.flistxattr_fn = xattr_tdb_flistxattr,
- .removexattr_fn = xattr_tdb_removexattr,
.fremovexattr_fn = xattr_tdb_fremovexattr,
- .open_fn = xattr_tdb_open,
- .mkdir_fn = xattr_tdb_mkdir,
- .unlink_fn = xattr_tdb_unlink,
- .rmdir_fn = xattr_tdb_rmdir,
+ .openat_fn = xattr_tdb_openat,
+ .mkdirat_fn = xattr_tdb_mkdirat,
+ .unlinkat_fn = xattr_tdb_unlinkat,
.connect_fn = xattr_tdb_connect,
};
-NTSTATUS vfs_xattr_tdb_init(void);
-NTSTATUS vfs_xattr_tdb_init(void)
+static_decl_vfs;
+NTSTATUS vfs_xattr_tdb_init(TALLOC_CTX *ctx)
{
return smb_register_vfs(SMB_VFS_INTERFACE_VERSION, "xattr_tdb",
&vfs_xattr_tdb_fns);