aboutsummaryrefslogtreecommitdiff
path: root/winsup/cygwin/fhandler_socket.cc
diff options
context:
space:
mode:
authorCorinna Vinschen <corinna@vinschen.de>2011-07-05 09:59:34 +0000
committerCorinna Vinschen <corinna@vinschen.de>2011-07-05 09:59:34 +0000
commitfcacca02dd945f1fce8ccfedae14e08b887d9adb (patch)
treecd6da580e778fae9a0e5a355ce404009fbced839 /winsup/cygwin/fhandler_socket.cc
parent3dce4ce6530b1de78b2c782cccef2d76bf5e6700 (diff)
downloadnewlib-fcacca02dd945f1fce8ccfedae14e08b887d9adb.zip
newlib-fcacca02dd945f1fce8ccfedae14e08b887d9adb.tar.gz
newlib-fcacca02dd945f1fce8ccfedae14e08b887d9adb.tar.bz2
* fhandler.cc (fhandler_base::open): Don't open file with WRITE_DAC
access on remote filesystem. Explain why. * fhandler_disk_file.cc (fhandler_disk_file::mkdir): Ditto for directories. * fhandler_socket.cc (fhandler_socket::bind): Ditto for sockets. * path.cc (symlink_worker): Ditto for symlinks.
Diffstat (limited to 'winsup/cygwin/fhandler_socket.cc')
-rw-r--r--winsup/cygwin/fhandler_socket.cc8
1 files changed, 6 insertions, 2 deletions
diff --git a/winsup/cygwin/fhandler_socket.cc b/winsup/cygwin/fhandler_socket.cc
index 9ea1bc6..62609fe 100644
--- a/winsup/cygwin/fhandler_socket.cc
+++ b/winsup/cygwin/fhandler_socket.cc
@@ -970,8 +970,12 @@ fhandler_socket::bind (const struct sockaddr *name, int namelen)
/* If the filesystem supports ACLs, we will overwrite the DACL after the
call to NtCreateFile. This requires a handle with READ_CONTROL and
WRITE_DAC access, otherwise get_file_sd and set_file_sd both have to
- open the file again. */
- if (pc.has_acls ())
+ open the file again.
+ FIXME: On remote NTFS shares open sometimes fails because even the
+ creator of the file doesn't have the right to change the DACL.
+ I don't know what setting that is or howq to recognize such a share,
+ so for now we don't request WRITE_DAC on remote drives. */
+ if (pc.has_acls () && !pc.isremote ())
access |= READ_CONTROL | WRITE_DAC;
status = NtCreateFile (&fh, access, pc.get_object_attr (attr, sa), &io,