summaryrefslogtreecommitdiff
path: root/source3/smbd/open.c
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2011-02-24 16:04:09 -0800
committerJeremy Allison <jra@samba.org>2011-02-25 01:57:04 +0100
commitf92fad101a755cdf0845dd8abf9431312ba82094 (patch)
tree3e7e259ccd63c192a7ac0d3e8c6bcb57abd0a3a8 /source3/smbd/open.c
parentbc1312303dd2de401f3c6bd93a705ab1bfe07b56 (diff)
downloadsamba-f92fad101a755cdf0845dd8abf9431312ba82094.tar.gz
samba-f92fad101a755cdf0845dd8abf9431312ba82094.tar.bz2
samba-f92fad101a755cdf0845dd8abf9431312ba82094.zip
Ensure we don't return an incorrect access mask.
From the Microsoft test suite @ Connectathon: Test Case: TestSuite_ScenarioNo009GrantedAccessTestS0 File created with access = 0x7 (Read, Write, Delete) Query Info on file returns 0x87 (Read, Write, Delete, Read Attributes) Jeremy.
Diffstat (limited to 'source3/smbd/open.c')
-rw-r--r--source3/smbd/open.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/source3/smbd/open.c b/source3/smbd/open.c
index d66f9bae17..7735301c22 100644
--- a/source3/smbd/open.c
+++ b/source3/smbd/open.c
@@ -2261,8 +2261,9 @@ static NTSTATUS open_file_ntcreate(connection_struct *conn,
/*
* According to Samba4, SEC_FILE_READ_ATTRIBUTE is always granted,
+ * but we don't have to store this - just ignore it on access check.
*/
- fsp->access_mask = access_mask | FILE_READ_ATTRIBUTES;
+ fsp->access_mask = access_mask;
if (file_existed) {
/* stat opens on existing files don't get oplocks. */