summaryrefslogtreecommitdiff
path: root/source3/ubiqx
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>1997-12-23 13:21:36 +0000
committerJeremy Allison <jra@samba.org>1997-12-23 13:21:36 +0000
commit08bd3b8cc0ed557e8fa9769c053b456ba0fa457e (patch)
tree64363299312fa96e34d78ec05e1533340764eb7c /source3/ubiqx
parentd4979e7727ef94de469e66b74a9bee6f9742d4c0 (diff)
downloadsamba-08bd3b8cc0ed557e8fa9769c053b456ba0fa457e.tar.gz
samba-08bd3b8cc0ed557e8fa9769c053b456ba0fa457e.tar.bz2
samba-08bd3b8cc0ed557e8fa9769c053b456ba0fa457e.zip
Fixed nastly little bug where client was sending open_and_X call
with smb_mode (smb_vwv[3]) of 0x20 == DENY_WRITE + read-only-open. and smb_ofun (smb_vwv[8]) of 0x11 == Create if not exist plus append on a read-only share. This was mapped into the strange unix flags of (O_RDONLY|O_CREAT) - essentially O_CREAT as O_RDONLY == 0. We were checking the unix flags directly against O_RDONLY instead of masking off the open mode flag bits before doing the comparison, so this open was being refused even though it was valid on a read-only share. Also ensured that the O_CREAT bit was masked out of the flags bit if the open was done on a read-only share (as doing a unix open( filename, O_RDONLY|O_CREAT, xxx) will create a zero length file if the user had permission to write into the directory - which should be denied on a read-only share. Thanks to Mark Peek @ Whistle for giving me this test case. Jeremy. (This used to be commit eae921ac632c8297836b85e909903da0602eac0e)
Diffstat (limited to 'source3/ubiqx')
0 files changed, 0 insertions, 0 deletions