From 16d6da8a9bc87a43e4d31a7c220969bd0eaefa3e Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Thu, 16 Dec 2010 16:50:31 -0800 Subject: Move checks inside file_set_sparse() to allow it to be called from anywhere. Autobuild-User: Jeremy Allison Autobuild-Date: Fri Dec 17 21:56:38 CET 2010 on sn-devel-104 --- source3/smbd/dosmode.c | 22 +++++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) (limited to 'source3/smbd/dosmode.c') diff --git a/source3/smbd/dosmode.c b/source3/smbd/dosmode.c index 86f7807506..628f887187 100644 --- a/source3/smbd/dosmode.c +++ b/source3/smbd/dosmode.c @@ -20,6 +20,7 @@ #include "includes.h" #include "librpc/gen_ndr/ndr_xattr.h" +#include "../libcli/security/security.h" static uint32_t filter_mode_by_protocol(uint32_t mode) { @@ -855,6 +856,25 @@ NTSTATUS file_set_sparse(connection_struct *conn, uint32_t new_dosmode; NTSTATUS status; + if (!CAN_WRITE(conn)) { + DEBUG(9,("file_set_sparse: fname[%s] set[%u] " + "on readonly share[%s]\n", + smb_fname_str_dbg(fsp->fsp_name), + sparse, + lp_servicename(SNUM(conn)))); + return NT_STATUS_MEDIA_WRITE_PROTECTED; + } + + if (!(fsp->access_mask & FILE_WRITE_DATA) && + !(fsp->access_mask & FILE_WRITE_ATTRIBUTES)) { + DEBUG(9,("file_set_sparse: fname[%s] set[%u] " + "access_mask[0x%08X] - access denied\n", + smb_fname_str_dbg(fsp->fsp_name), + sparse, + fsp->access_mask)); + return NT_STATUS_ACCESS_DENIED; + } + DEBUG(10,("file_set_sparse: setting sparse bit %u on file %s\n", sparse, smb_fname_str_dbg(fsp->fsp_name))); @@ -888,7 +908,7 @@ NTSTATUS file_set_sparse(connection_struct *conn, notify_fname(conn, NOTIFY_ACTION_MODIFIED, FILE_NOTIFY_CHANGE_ATTRIBUTES, - smb_fname->base_name); + fsp->fsp_name->base_name); return NT_STATUS_OK; } -- cgit