diff options
author | Jeremy Allison <jra@samba.org> | 2011-02-24 16:04:09 -0800 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2011-02-25 01:57:04 +0100 |
commit | f92fad101a755cdf0845dd8abf9431312ba82094 (patch) | |
tree | 3e7e259ccd63c192a7ac0d3e8c6bcb57abd0a3a8 /source3/smbd | |
parent | bc1312303dd2de401f3c6bd93a705ab1bfe07b56 (diff) | |
download | samba-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')
-rw-r--r-- | source3/smbd/open.c | 3 |
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. */ |