diff options
| author | Jeremy Allison <jra@samba.org> | 2012-06-04 17:34:54 -0700 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2012-06-05 04:26:10 +0200 |
| commit | 4accd2396e8a9ac4a7b2191a1d3bc9c84d2e0497 (patch) | |
| tree | d6e112fc4684e809888b0048571d1b4520b87369 | |
| parent | 01106230ddc8da90e2ff2667dd3702e3c110f720 (diff) | |
| download | samba-4accd2396e8a9ac4a7b2191a1d3bc9c84d2e0497.tar.gz samba-4accd2396e8a9ac4a7b2191a1d3bc9c84d2e0497.tar.bz2 samba-4accd2396e8a9ac4a7b2191a1d3bc9c84d2e0497.zip | |
Fix bug #8972 - Directory group write permission bit is set if unix extensions are enabled
We can't manipulate file_attributes if it's a posix call. I'll look
at adding a test for this asap.
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue Jun 5 04:26:11 CEST 2012 on sn-devel-104
| -rw-r--r-- | source3/smbd/open.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source3/smbd/open.c b/source3/smbd/open.c index 17721c6127..a012675f56 100644 --- a/source3/smbd/open.c +++ b/source3/smbd/open.c @@ -2714,8 +2714,10 @@ static NTSTATUS open_directory(connection_struct *conn, return NT_STATUS_NOT_A_DIRECTORY; } - /* Ensure we have a directory attribute. */ - file_attributes |= FILE_ATTRIBUTE_DIRECTORY; + if (!(file_attributes & FILE_FLAG_POSIX_SEMANTICS)) { + /* Ensure we have a directory attribute. */ + file_attributes |= FILE_ATTRIBUTE_DIRECTORY; + } DEBUG(5,("open_directory: opening directory %s, access_mask = 0x%x, " "share_access = 0x%x create_options = 0x%x, " |