Minor fix in acl_set_file.

 src/w32.c (acl_set_file): Treat ERROR_ACCESS_DENIED from
 set_file_security as failure due to insufficient privileges.
 Reported by Fabrice Popineau <fabrice.popineau@supelec.fr>.

diff --git a/src/ChangeLog b/src/ChangeLog
index 2954df9d6b83913fe01f96f3228a751def535537..267738787539a5d340c8dd866d79a266ae149dab 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,9 @@
+2013-01-19  Eli Zaretskii  <eliz@gnu.org>
+
+       * w32.c (acl_set_file): Treat ERROR_ACCESS_DENIED from
+       set_file_security as failure due to insufficient privileges.
+       Reported by Fabrice Popineau <fabrice.popineau@supelec.fr>.
+
 2013-01-19  Paul Eggert  <eggert@cs.ucla.edu>
 
        Work around bug in CIFS and vboxsf file systems (Bug#13149).

diff --git a/src/w32.c b/src/w32.c
index 812003e96c011448174a1982f348e59a30bbd234..525b0a8e89b822dcc487c8b66552c6448247731b 100644 (file)
--- a/src/w32.c
+++ b/src/w32.c
@@ -4875,7 +4875,8 @@ acl_set_file (const char *fname, acl_type_t type, acl_t acl)
       retval = 0;
       errno = e;
     }
-  else if (err == ERROR_INVALID_OWNER || err == ERROR_NOT_ALL_ASSIGNED)
+  else if (err == ERROR_INVALID_OWNER || err == ERROR_NOT_ALL_ASSIGNED
+          || err == ERROR_ACCESS_DENIED)
     {
       /* Maybe the requested ACL and the one the file already has are
         identical, in which case we can silently ignore the