cp: fix a probably redundant chmod when setting xattrs

* src/copy.c (copy_reg): Fix initial value of access_changed variable.
This was introduced by Pádraig Brady in commit cca83faf, 2009-09-14,
"cp,mv: preserve extended attributes even for read-only files"

diff --git a/src/copy.c b/src/copy.c
index ad2060b9c433789d02c4296443d992535680d098..b7d113fbeabf768b25539137838b893ec7f83ecc 100644 (file)
--- a/src/copy.c
+++ b/src/copy.c
@@ -839,7 +839,7 @@ copy_reg (char const *src_name, char const *dst_name,
      by xattr_permission() in fs/xattr.c of the GNU/Linux kernel tree.  */
   if (x->preserve_xattr)
     {
-      bool access_changed = true;
+      bool access_changed = false;
 
       if (!(sb.st_mode & S_IWUSR) && geteuid() != 0)
         access_changed = fchmod_or_lchmod (dest_desc, dst_name, 0600) == 0;