diff options
author | Corinna Vinschen <corinna@vinschen.de> | 2007-07-29 19:57:41 +0400 |
---|---|---|
committer | Corinna Vinschen <corinna@vinschen.de> | 2007-07-29 19:57:41 +0400 |
commit | 9d017bd09cf60644db45802bb2e5fd5ffaed619a (patch) | |
tree | 6ed2f4a2155f7e3c4a0e32901503681f92e23b8c /winsup/cygwin/syscalls.cc | |
parent | ed7ec849f625609f8c8e061386adfcb593fcd54a (diff) |
* fhandler_disk_file.cc (fhandler_disk_file::fchmod): Don't allow
FileAttributes set to 0 when calling NtSetInformationFile since it has
a special meaning.
(fhandler_disk_file::facl): Ditto.
(fhandler_disk_file::link): Only set attributes after copying files.
Use SetFileAttributesW.
* syscalls.cc (unlink_nt): Only care for actual FILE_ATTRIBUTE_READONLY.
Don't allow FileAttributes set to 0 when calling NtSetInformationFile.
After marking for deletion, restore R/O attribute on files to
accommodate hardlinks.
Diffstat (limited to 'winsup/cygwin/syscalls.cc')
-rw-r--r-- | winsup/cygwin/syscalls.cc | 29 |
1 files changed, 17 insertions, 12 deletions
diff --git a/winsup/cygwin/syscalls.cc b/winsup/cygwin/syscalls.cc index 8d1c2bbf2..83b7bbb84 100644 --- a/winsup/cygwin/syscalls.cc +++ b/winsup/cygwin/syscalls.cc @@ -238,12 +238,10 @@ unlink_nt (path_conv &pc) FILE_BASIC_INFORMATION fbi; ACCESS_MASK access = DELETE; - /* If one of the R/O attributes is set, we have to open the file with - FILE_WRITE_ATTRIBUTES to be able to remove these flags before trying + /* If the R/O attribute is set, we have to open the file with + FILE_WRITE_ATTRIBUTES to be able to remove this flags before trying to delete it. */ - if (pc.file_attributes () & (FILE_ATTRIBUTE_READONLY - | FILE_ATTRIBUTE_SYSTEM - | FILE_ATTRIBUTE_HIDDEN)) + if (pc.file_attributes () & FILE_ATTRIBUTE_READONLY) access |= FILE_WRITE_ATTRIBUTES; ULONG flags = FILE_OPEN_FOR_BACKUP_INTENT; @@ -330,16 +328,14 @@ unlink_nt (path_conv &pc) if (move_to_bin && !pc.isremote ()) try_to_bin (pc, fh); - /* Get rid of read-only attributes. */ + /* Get rid of read-only attribute. */ if (access & FILE_WRITE_ATTRIBUTES) { FILE_BASIC_INFORMATION fbi; fbi.CreationTime.QuadPart = fbi.LastAccessTime.QuadPart = fbi.LastWriteTime.QuadPart = fbi.ChangeTime.QuadPart = 0LL; - fbi.FileAttributes = pc.file_attributes () - & ~(FILE_ATTRIBUTE_READONLY - | FILE_ATTRIBUTE_SYSTEM - | FILE_ATTRIBUTE_HIDDEN); + fbi.FileAttributes = (pc.file_attributes () & ~FILE_ATTRIBUTE_READONLY) + ?: FILE_ATTRIBUTE_NORMAL; NtSetInformationFile (fh, &io, &fbi, sizeof fbi, FileBasicInformation); } @@ -349,14 +345,23 @@ unlink_nt (path_conv &pc) if (!NT_SUCCESS (status)) { syscall_printf ("Setting delete disposition failed, status = %p", status); - /* Restore R/O attributes. */ if (access & FILE_WRITE_ATTRIBUTES) - { + { + /* Restore R/O attributes. */ fbi.FileAttributes = pc.file_attributes (); NtSetInformationFile (fh, &io, &fbi, sizeof fbi, FileBasicInformation); } } + else if ((access & FILE_WRITE_ATTRIBUTES) && !pc.isdir ()) + { + /* Restore R/O attribute to accommodate hardlinks. Don't try this + with directories! For some reason the below NtSetInformationFile + changes the disposition for delete back to FALSE, at least on XP. */ + fbi.FileAttributes = pc.file_attributes (); + NtSetInformationFile (fh, &io, &fbi, sizeof fbi, + FileBasicInformation); + } NtClose (fh); return status; |