diff options
author | Jeremy Allison <jra@samba.org> | 2010-11-23 15:59:33 -0800 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2010-11-24 01:47:13 +0100 |
commit | b1dacb8195a52b6cfce6ad60a839f7effd35b261 (patch) | |
tree | ae0abe8f35b6a5b253ca479f6f916eb47af730ba /source3/smbd/nttrans.c | |
parent | 1f478afe22c5dc68d7a69e4127c3a8cd6dca1def (diff) | |
download | samba-b1dacb8195a52b6cfce6ad60a839f7effd35b261.tar.gz samba-b1dacb8195a52b6cfce6ad60a839f7effd35b261.tar.bz2 samba-b1dacb8195a52b6cfce6ad60a839f7effd35b261.zip |
Add in fsp->access_mask checks. Not required (underlying system does this)
but makes logic cleaner. Pointed out by Metze.
Jeremy.
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed Nov 24 01:47:13 CET 2010 on sn-devel-104
Diffstat (limited to 'source3/smbd/nttrans.c')
-rw-r--r-- | source3/smbd/nttrans.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source3/smbd/nttrans.c b/source3/smbd/nttrans.c index 855a49b2d5..ad585337b8 100644 --- a/source3/smbd/nttrans.c +++ b/source3/smbd/nttrans.c @@ -1389,7 +1389,8 @@ static NTSTATUS copy_internals(TALLOC_CTX *ctx, req, /* req */ 0, /* root_dir_fid */ smb_fname_src, /* fname */ - FILE_READ_DATA, /* access_mask */ + FILE_READ_DATA|FILE_READ_ATTRIBUTES| + FILE_READ_EA, /* access_mask */ (FILE_SHARE_READ | FILE_SHARE_WRITE | /* share_access */ FILE_SHARE_DELETE), FILE_OPEN, /* create_disposition*/ @@ -1412,7 +1413,8 @@ static NTSTATUS copy_internals(TALLOC_CTX *ctx, req, /* req */ 0, /* root_dir_fid */ smb_fname_dst, /* fname */ - FILE_WRITE_DATA, /* access_mask */ + FILE_WRITE_DATA|FILE_WRITE_ATTRIBUTES| + FILE_WRITE_EA, /* access_mask */ (FILE_SHARE_READ | FILE_SHARE_WRITE | /* share_access */ FILE_SHARE_DELETE), FILE_CREATE, /* create_disposition*/ |