This is the mail archive of the
cygwin-cvs@cygwin.com
mailing list for the Cygwin project.
[newlib-cygwin] Only request WRITE_DAC rights when writing an ACL
- From: Corinna Vinschen <corinna at sourceware dot org>
- To: cygwin-cvs at sourceware dot org
- Date: 25 Dec 2015 20:42:04 -0000
- Subject: [newlib-cygwin] Only request WRITE_DAC rights when writing an ACL
https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=e983bd6aa4cf25dffc3428f46f8943d7876f75ee
commit e983bd6aa4cf25dffc3428f46f8943d7876f75ee
Author: Corinna Vinschen <corinna@vinschen.de>
Date: Fri Dec 25 21:40:51 2015 +0100
Only request WRITE_DAC rights when writing an ACL
* fhandler_disk_file.cc (fhandler_disk_file::fchmod): Only request
query_write_dac rather than query_write_control access when writing
an ACL. Fix a comment.
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
Diff:
---
winsup/cygwin/ChangeLog | 6 ++++++
winsup/cygwin/fhandler_disk_file.cc | 4 ++--
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index 3bddc68..546df24 100644
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,9 @@
+2015-12-25 Corinna Vinschen <corinna@vinschen.de>
+
+ * fhandler_disk_file.cc (fhandler_disk_file::fchmod): Only request
+ query_write_dac rather than query_write_control access when writing
+ an ACL. Fix a comment.
+
2015-12-24 Corinna Vinschen <corinna@vinschen.de>
* sec_acl.cc (__aclcalcmask): New function to recalculate ACL masks.
diff --git a/winsup/cygwin/fhandler_disk_file.cc b/winsup/cygwin/fhandler_disk_file.cc
index 1dd1b8c..4ee67e2 100644
--- a/winsup/cygwin/fhandler_disk_file.cc
+++ b/winsup/cygwin/fhandler_disk_file.cc
@@ -742,7 +742,7 @@ fhandler_disk_file::fchmod (mode_t mode)
query_open (query_write_dac);
if (!(oret = open (O_BINARY, 0)))
{
- /* Need WRITE_DAC|WRITE_OWNER to write ACLs. */
+ /* Need WRITE_DAC to write ACLs. */
if (pc.has_acls ())
return -1;
/* Otherwise FILE_WRITE_ATTRIBUTES is sufficient. */
@@ -1021,7 +1021,7 @@ cant_access_acl:
if ((cmd == SETACL && !get_handle ())
|| (cmd != SETACL && !get_stat_handle ()))
{
- query_open (cmd == SETACL ? query_write_control : query_read_control);
+ query_open (cmd == SETACL ? query_write_dac : query_read_control);
if (!(oret = open (O_BINARY, 0)))
{
if (cmd == GETACL || cmd == GETACLCNT)