From d1d3d8f37021bc18d30c8c97f99f1e0d28fdd646 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Thu, 8 Aug 2019 15:48:54 -0700 Subject: [PATCH] s3: VFS: vfs_ceph_snapshots: Make chown return errno = EROFS on a shadow copy path. smbd has no business modifying a shadow copy filesystem, it should be read-only. Signed-off-by: Jeremy Allison Reviewed-by: David Disseldorp --- source3/modules/vfs_ceph_snapshots.c | 29 ++++------------------------ 1 file changed, 4 insertions(+), 25 deletions(-) diff --git a/source3/modules/vfs_ceph_snapshots.c b/source3/modules/vfs_ceph_snapshots.c index 3076fa54c3a..059dd500606 100644 --- a/source3/modules/vfs_ceph_snapshots.c +++ b/source3/modules/vfs_ceph_snapshots.c @@ -1001,41 +1001,20 @@ static int ceph_snap_gmt_chown(vfs_handle_struct *handle, gid_t gid) { time_t timestamp = 0; - char stripped[PATH_MAX + 1]; - char conv[PATH_MAX + 1]; int ret; - struct smb_filename *new_fname; - int saved_errno; ret = ceph_snap_gmt_strip_snapshot(handle, csmb_fname->base_name, - ×tamp, stripped, sizeof(stripped)); - if (ret < 0) { - errno = -ret; - return -1; - } - if (timestamp == 0) { - return SMB_VFS_NEXT_CHOWN(handle, csmb_fname, uid, gid); - } - - ret = ceph_snap_gmt_convert(handle, stripped, - timestamp, conv, sizeof(conv)); + ×tamp, NULL, 0); if (ret < 0) { errno = -ret; return -1; } - new_fname = cp_smb_filename(talloc_tos(), csmb_fname); - if (new_fname == NULL) { - errno = ENOMEM; + if (timestamp != 0) { + errno = EROFS; return -1; } - new_fname->base_name = conv; - - ret = SMB_VFS_NEXT_CHOWN(handle, new_fname, uid, gid); - saved_errno = errno; - TALLOC_FREE(new_fname); - errno = saved_errno; - return ret; + return SMB_VFS_NEXT_CHOWN(handle, csmb_fname, uid, gid); } static int ceph_snap_gmt_chdir(vfs_handle_struct *handle, -- 2.47.3