diff options
author | Ralph Boehme <slow@samba.org> | 2021-01-25 11:46:30 +0100 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2021-02-01 07:50:10 +0000 |
commit | 483c1dc818ec748daad85fd8e4f223d6edf22f60 (patch) | |
tree | 94f4219474136ec113f4eb28d25be5c6152810a1 /source3 | |
parent | 2c0987d65646aa41d0bc81f9e1c06f2ad9b5b485 (diff) | |
download | samba-483c1dc818ec748daad85fd8e4f223d6edf22f60.tar.gz |
vfs_error_inject: add unlinkat hook
Note that a failure is only injected if the owner of the parent directory is not
the same as the current user.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=14617
Back-ported from commit c44dad3ac2eb36fc5eb5a9f80a9ef97183be26ef.
Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'source3')
-rw-r--r-- | source3/modules/vfs_error_inject.c | 37 |
1 files changed, 37 insertions, 0 deletions
diff --git a/source3/modules/vfs_error_inject.c b/source3/modules/vfs_error_inject.c index 04880ffd5ab..d8731c29610 100644 --- a/source3/modules/vfs_error_inject.c +++ b/source3/modules/vfs_error_inject.c @@ -30,6 +30,7 @@ struct unix_error_map { { "ESTALE", ESTALE }, { "EBADF", EBADF }, { "EINTR", EINTR }, + { "EACCES", EACCES }, }; static int find_unix_error_from_string(const char *err_str) @@ -122,10 +123,46 @@ static int vfs_error_inject_openat(struct vfs_handle_struct *handle, return SMB_VFS_NEXT_OPENAT(handle, dirfsp, smb_fname, fsp, flags, mode); } +static int vfs_error_inject_unlinkat(struct vfs_handle_struct *handle, + struct files_struct *dirfsp, + const struct smb_filename *smb_fname, + int flags) +{ + struct smb_filename *parent_fname = NULL; + int error = inject_unix_error("unlinkat", handle); + int ret; + bool ok; + + if (error == 0) { + return SMB_VFS_NEXT_UNLINKAT(handle, dirfsp, smb_fname, flags); + } + + ok = parent_smb_fname(talloc_tos(), smb_fname, &parent_fname, NULL); + if (!ok) { + return -1; + } + + ret = SMB_VFS_STAT(handle->conn, parent_fname); + if (ret != 0) { + TALLOC_FREE(parent_fname); + return -1; + } + + if (parent_fname->st.st_ex_uid == get_current_uid(dirfsp->conn)) { + TALLOC_FREE(parent_fname); + return SMB_VFS_NEXT_UNLINKAT(handle, dirfsp, smb_fname, flags); + } + + TALLOC_FREE(parent_fname); + errno = error; + return -1; +} + static struct vfs_fn_pointers vfs_error_inject_fns = { .chdir_fn = vfs_error_inject_chdir, .pwrite_fn = vfs_error_inject_pwrite, .openat_fn = vfs_error_inject_openat, + .unlinkat_fn = vfs_error_inject_unlinkat, }; static_decl_vfs; |