]> git.ipfire.org Git - thirdparty/rsync.git/blobdiff - xattrs.c
More tweaks for Actions.
[thirdparty/rsync.git] / xattrs.c
index f662cf7a487c7d6007feb92b10a04977317718c5..26e50a6f9ccb48e0496da3126363ac5f61d6494b 100644 (file)
--- a/xattrs.c
+++ b/xattrs.c
@@ -3,11 +3,12 @@
  * Written by Jay Fenlason, vaguely based on the ACLs patch.
  *
  * Copyright (C) 2004 Red Hat, Inc.
- * Copyright (C) 2006, 2007 Wayne Davison
+ * Copyright (C) 2006-2022 Wayne Davison
  *
  * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 3 of the License, or
+ * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License along
- * with this program; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA.
+ * with this program; if not, visit the http://fsf.org website.
  */
 
 #include "rsync.h"
+#include "ifuncs.h"
+#include "inums.h"
 #include "lib/sysxattrs.h"
 
 #ifdef SUPPORT_XATTRS
@@ -30,22 +32,29 @@ extern int am_sender;
 extern int am_generator;
 extern int read_only;
 extern int list_only;
+extern int preserve_xattrs;
+extern int preserve_links;
+extern int preserve_devices;
+extern int preserve_specials;
 extern int checksum_seed;
+extern int saw_xattr_filter;
+
+extern struct name_num_item *xattr_sum_nni;
+extern int xattr_sum_len;
 
 #define RSYNC_XAL_INITIAL 5
 #define RSYNC_XAL_LIST_INITIAL 100
 
+#define MAX_XATTR_DIGEST_LEN MD5_DIGEST_LEN
 #define MAX_FULL_DATUM 32
 
-#define HAS_PREFIX(str, prfx) (*(str) == *(prfx) \
-                           && strncmp(str, prfx, sizeof (prfx) - 1) == 0)
+#define HAS_PREFIX(str, prfx) (*(str) == *(prfx) && strncmp(str, prfx, sizeof (prfx) - 1) == 0)
 
 #define XATTR_ABBREV(x) ((size_t)((x).name - (x).datum) < (x).datum_len)
 
-#define XSTATE_ABBREV  0
-#define XSTATE_DONE    1
-#define XSTATE_TODO    2
-#define XSTATE_LOCAL   3
+#define XSTATE_ABBREV  1
+#define XSTATE_DONE    2
+#define XSTATE_TODO    3
 
 #define USER_PREFIX "user."
 #define UPRE_LEN ((int)sizeof USER_PREFIX - 1)
@@ -53,7 +62,7 @@ extern int checksum_seed;
 #define SPRE_LEN ((int)sizeof SYSTEM_PREFIX - 1)
 
 #ifdef HAVE_LINUX_XATTRS
-#define MIGHT_NEED_RPRE (am_root < 0)
+#define MIGHT_NEED_RPRE (am_root <= 0)
 #define RSYNC_PREFIX USER_PREFIX "rsync."
 #else
 #define MIGHT_NEED_RPRE am_root
@@ -61,19 +70,43 @@ extern int checksum_seed;
 #endif
 #define RPRE_LEN ((int)sizeof RSYNC_PREFIX - 1)
 
-#define XSTAT_ATTR RSYNC_PREFIX "%stat"
-#define XSTAT_LEN ((int)sizeof XSTAT_ATTR - 1)
+#define XSTAT_SUFFIX "stat"
+#define XSTAT_ATTR RSYNC_PREFIX "%" XSTAT_SUFFIX
+#define XACC_ACL_SUFFIX "aacl"
+#define XACC_ACL_ATTR RSYNC_PREFIX "%" XACC_ACL_SUFFIX
+#define XDEF_ACL_SUFFIX "dacl"
+#define XDEF_ACL_ATTR RSYNC_PREFIX "%" XDEF_ACL_SUFFIX
 
 typedef struct {
        char *datum, *name;
        size_t datum_len, name_len;
+       int num;
 } rsync_xa;
 
+struct _rsync_xa_list;
+
+typedef struct _rsync_xa_list_ref {
+       struct _rsync_xa_list_ref *next;
+       int ndx;
+} rsync_xa_list_ref;
+
+typedef struct _rsync_xa_list {
+       int ndx;
+       int64 key;
+       item_list xa_items;
+} rsync_xa_list;
+
 static size_t namebuf_len = 0;
 static char *namebuf = NULL;
 
-static item_list empty_xattr = EMPTY_ITEM_LIST;
+static const rsync_xa_list empty_xa_list = {
+       .xa_items = EMPTY_ITEM_LIST,
+};
+static const item_list empty_xattr = EMPTY_ITEM_LIST;
 static item_list rsync_xal_l = EMPTY_ITEM_LIST;
+static struct hashtable *rsync_xal_h = NULL;
+
+static size_t prior_xattr_count = (size_t)-1;
 
 /* ------------------------------------------------------------------------- */
 
@@ -82,14 +115,17 @@ static void rsync_xal_free(item_list *xalp)
        size_t i;
        rsync_xa *rxas = xalp->items;
 
+       if (!xalp->malloced)
+               return;
+
        for (i = 0; i < xalp->count; i++) {
                free(rxas[i].datum);
                /*free(rxas[i].name);*/
        }
-       xalp->count = 0;
+       free(xalp->items);
 }
 
-void free_xattr(statx *sxp)
+void free_xattr(stat_x *sxp)
 {
        if (!sxp->xattr)
                return;
@@ -108,85 +144,81 @@ static int rsync_xal_compare_names(const void *x1, const void *x2)
 static ssize_t get_xattr_names(const char *fname)
 {
        ssize_t list_len;
+       int64 arg;
 
        if (!namebuf) {
                namebuf_len = 1024;
                namebuf = new_array(char, namebuf_len);
-               if (!namebuf)
-                       out_of_memory("get_xattr_names");
        }
 
-       /* The length returned includes all the '\0' terminators. */
-       list_len = sys_llistxattr(fname, namebuf, namebuf_len);
-       if (list_len > (ssize_t)namebuf_len) {
-               list_len = -1;
-               errno = ERANGE;
-       }
-       if (list_len >= 0)
-               return list_len;
-       if (errno == ENOTSUP)
-               return 0;
-       if (errno == ERANGE) {
+       while (1) {
+               /* The length returned includes all the '\0' terminators. */
+               list_len = sys_llistxattr(fname, namebuf, namebuf_len);
+               if (list_len >= 0) {
+                       if ((size_t)list_len <= namebuf_len)
+                               break;
+               } else if (errno == ENOTSUP)
+                       return 0;
+               else if (errno != ERANGE) {
+                       arg = namebuf_len;
+                 got_error:
+                       rsyserr(FERROR_XFER, errno,
+                               "get_xattr_names: llistxattr(%s,%s) failed",
+                               full_fname(fname), big_num(arg));
+                       return -1;
+               }
                list_len = sys_llistxattr(fname, NULL, 0);
                if (list_len < 0) {
-                       rsyserr(FERROR, errno,
-                               "get_xattr_names: llistxattr(\"%s\",0) failed",
-                               fname);
-                       return -1;
+                       arg = 0;
+                       goto got_error;
                }
                if (namebuf_len)
                        free(namebuf);
                namebuf_len = list_len + 1024;
                namebuf = new_array(char, namebuf_len);
-               if (!namebuf)
-                       out_of_memory("get_xattr_names");
-               list_len = sys_llistxattr(fname, namebuf, namebuf_len);
-               if (list_len >= 0)
-                       return list_len;
        }
 
-       rsyserr(FERROR, errno,
-               "get_xattr_names: llistxattr(\"%s\",%ld) failed",
-               fname, (long)namebuf_len);
-       return -1;
+       return list_len;
 }
 
 /* On entry, the *len_ptr parameter contains the size of the extra space we
  * should allocate when we create a buffer for the data.  On exit, it contains
  * the length of the datum. */
-static char *get_xattr_data(const char *fname, const char *name, size_t *len_ptr,
-                           int no_missing_error)
+static char *get_xattr_data(const char *fname, const char *name, size_t *len_ptr, int no_missing_error)
 {
        size_t datum_len = sys_lgetxattr(fname, name, NULL, 0);
+       size_t extra_len = *len_ptr;
        char *ptr;
 
+       *len_ptr = datum_len;
+
        if (datum_len == (size_t)-1) {
                if (errno == ENOTSUP || no_missing_error)
                        return NULL;
-               rsyserr(FERROR, errno,
-                       "get_xattr_data: lgetxattr(\"%s\",\"%s\",0) failed",
-                       fname, name);
+               rsyserr(FERROR_XFER, errno,
+                       "get_xattr_data: lgetxattr(%s,\"%s\",0) failed",
+                       full_fname(fname), name);
                return NULL;
        }
 
-       if (datum_len + *len_ptr < datum_len /* checks for overflow */
-        || !(ptr = new_array(char, datum_len + *len_ptr)))
-               out_of_memory("get_xattr_data");
-
-       *len_ptr = datum_len;
+       if (!datum_len && !extra_len)
+               extra_len = 1; /* request non-zero amount of memory */
+       if (SIZE_MAX - datum_len < extra_len)
+               overflow_exit("get_xattr_data");
+       ptr = new_array(char, datum_len + extra_len);
 
        if (datum_len) {
                size_t len = sys_lgetxattr(fname, name, ptr, datum_len);
                if (len != datum_len) {
                        if (len == (size_t)-1) {
-                               rsyserr(FERROR, errno,
-                                   "get_xattr_data: lgetxattr(\"%s\",\"%s\",%ld)"
-                                   " failed", fname, name, (long)datum_len);
+                               rsyserr(FERROR_XFER, errno,
+                                       "get_xattr_data: lgetxattr(%s,\"%s\",%ld) failed",
+                                       full_fname(fname), name, (long)datum_len);
                        } else {
-                               rprintf(FERROR,
-                                   "get_xattr_data: lgetxattr(\"%s\",\"%s\",%ld)"
-                                   " returned %ld\n", fname, name,
-                                   (long)datum_len, (long)len);
+                               rprintf(FERROR_XFER,
+                                       "get_xattr_data: lgetxattr(%s,\"%s\",%ld) returned %ld\n",
+                                       full_fname(fname), name,
+                                       (long)datum_len, (long)len);
                        }
                        free(ptr);
                        return NULL;
@@ -204,28 +236,36 @@ static int rsync_xal_get(const char *fname, item_list *xalp)
 #ifdef HAVE_LINUX_XATTRS
        int user_only = am_sender ? 0 : !am_root;
 #endif
+       rsync_xa *rxa;
+       int count;
 
        /* This puts the name list into the "namebuf" buffer. */
        if ((list_len = get_xattr_names(fname)) < 0)
                return -1;
 
        for (name = namebuf; list_len > 0; name += name_len) {
-               rsync_xa *rxas;
-
                name_len = strlen(name) + 1;
                list_len -= name_len;
 
+               if (saw_xattr_filter) {
+                       if (name_is_excluded(name, NAME_IS_XATTR, ALL_FILTERS))
+                               continue;
+               }
 #ifdef HAVE_LINUX_XATTRS
-               /* We always ignore the system namespace, and non-root
-                * ignores everything but the user namespace. */
-               if (user_only ? !HAS_PREFIX(name, USER_PREFIX)
-                             : HAS_PREFIX(name, SYSTEM_PREFIX))
+               /* Choose between ignoring the system namespace or (non-root) ignoring any non-user namespace. */
+               else if (user_only ? !HAS_PREFIX(name, USER_PREFIX) : HAS_PREFIX(name, SYSTEM_PREFIX))
                        continue;
 #endif
 
-               if (am_root < 0 && name_len == XSTAT_LEN + 1
-                && name[RPRE_LEN] == '%' && strcmp(name, XSTAT_ATTR) == 0)
-                       continue;
+               /* No rsync.%FOO attributes are copied w/o 2 -X options. */
+               if (name_len > RPRE_LEN && name[RPRE_LEN] == '%' && HAS_PREFIX(name, RSYNC_PREFIX)) {
+                       if ((am_sender && preserve_xattrs < 2)
+                        || (am_root < 0
+                         && (strcmp(name+RPRE_LEN+1, XSTAT_SUFFIX) == 0
+                          || strcmp(name+RPRE_LEN+1, XACC_ACL_SUFFIX) == 0
+                          || strcmp(name+RPRE_LEN+1, XDEF_ACL_SUFFIX) == 0)))
+                               continue;
+               }
 
                datum_len = name_len; /* Pass extra size to get_xattr_data() */
                if (!(ptr = get_xattr_data(fname, name, &datum_len, 0)))
@@ -233,43 +273,59 @@ static int rsync_xal_get(const char *fname, item_list *xalp)
 
                if (datum_len > MAX_FULL_DATUM) {
                        /* For large datums, we store a flag and a checksum. */
-                       name_offset = 1 + MAX_DIGEST_LEN;
-                       sum_init(checksum_seed);
+                       name_offset = 1 + MAX_XATTR_DIGEST_LEN;
+                       sum_init(xattr_sum_nni, checksum_seed);
                        sum_update(ptr, datum_len);
                        free(ptr);
 
-                       if (!(ptr = new_array(char, name_offset + name_len)))
-                               out_of_memory("rsync_xal_get");
+                       ptr = new_array(char, name_offset + name_len);
                        *ptr = XSTATE_ABBREV;
                        sum_end(ptr + 1);
                } else
                        name_offset = datum_len;
 
-#ifdef HAVE_LINUX_XATTRS
-               if (am_root < 0 && name_len > RPRE_LEN
-                && HAS_PREFIX(name, RSYNC_PREFIX)) {
-                       name += RPRE_LEN;
-                       name_len -= RPRE_LEN;
-               }
-#endif
-
-               rxas = EXPAND_ITEM_LIST(xalp, rsync_xa, RSYNC_XAL_INITIAL);
-               rxas->name = ptr + name_offset;
-               memcpy(rxas->name, name, name_len);
-               rxas->datum = ptr;
-               rxas->name_len = name_len;
-               rxas->datum_len = datum_len;
+               rxa = EXPAND_ITEM_LIST(xalp, rsync_xa, RSYNC_XAL_INITIAL);
+               rxa->name = ptr + name_offset;
+               memcpy(rxa->name, name, name_len);
+               rxa->datum = ptr;
+               rxa->name_len = name_len;
+               rxa->datum_len = datum_len;
        }
-       if (xalp->count > 1)
-               qsort(xalp->items, xalp->count, sizeof (rsync_xa), rsync_xal_compare_names);
+       count = xalp->count;
+       rxa = xalp->items;
+       if (count > 1)
+               qsort(rxa, count, sizeof (rsync_xa), rsync_xal_compare_names);
+       for (rxa += count-1; count; count--, rxa--)
+               rxa->num = count;
        return 0;
 }
 
 /* Read the xattr(s) for this filename. */
-int get_xattr(const char *fname, statx *sxp)
+int get_xattr(const char *fname, stat_x *sxp)
 {
        sxp->xattr = new(item_list);
        *sxp->xattr = empty_xattr;
+
+       if (S_ISREG(sxp->st.st_mode) || S_ISDIR(sxp->st.st_mode)) {
+               /* Everyone supports this. */
+       } else if (S_ISLNK(sxp->st.st_mode)) {
+#ifndef NO_SYMLINK_XATTRS
+               if (!preserve_links)
+#endif
+                       return 0;
+       } else if (IS_SPECIAL(sxp->st.st_mode)) {
+#ifndef NO_SPECIAL_XATTRS
+               if (!preserve_specials)
+#endif
+                       return 0;
+       } else if (IS_DEVICE(sxp->st.st_mode)) {
+#ifndef NO_DEVICE_XATTRS
+               if (!preserve_devices)
+#endif
+                       return 0;
+       } else if (IS_MISSING_FILE(sxp->st))
+               return 0;
+
        if (rsync_xal_get(fname, sxp->xattr) < 0) {
                free_xattr(sxp);
                return -1;
@@ -277,17 +333,96 @@ int get_xattr(const char *fname, statx *sxp)
        return 0;
 }
 
-static int find_matching_xattr(item_list *xalp)
+int copy_xattrs(const char *source, const char *dest)
 {
-       size_t i, j;
-       item_list *lst = rsync_xal_l.items;
+       ssize_t list_len, name_len;
+       size_t datum_len;
+       char *name, *ptr;
+#ifdef HAVE_LINUX_XATTRS
+       int user_only = am_sender ? 0 : am_root <= 0;
+#endif
 
-       for (i = 0; i < rsync_xal_l.count; i++) {
-               rsync_xa *rxas1 = lst[i].items;
-               rsync_xa *rxas2 = xalp->items;
+       /* This puts the name list into the "namebuf" buffer. */
+       if ((list_len = get_xattr_names(source)) < 0)
+               return -1;
+
+       for (name = namebuf; list_len > 0; name += name_len) {
+               name_len = strlen(name) + 1;
+               list_len -= name_len;
+
+               if (saw_xattr_filter) {
+                       if (name_is_excluded(name, NAME_IS_XATTR, ALL_FILTERS))
+                               continue;
+               }
+#ifdef HAVE_LINUX_XATTRS
+               /* Choose between ignoring the system namespace or (non-root) ignoring any non-user namespace. */
+               else if (user_only ? !HAS_PREFIX(name, USER_PREFIX) : HAS_PREFIX(name, SYSTEM_PREFIX))
+                       continue;
+#endif
+
+               datum_len = 0;
+               if (!(ptr = get_xattr_data(source, name, &datum_len, 0)))
+                       return -1;
+               if (sys_lsetxattr(dest, name, ptr, datum_len) < 0) {
+                       int save_errno = errno ? errno : EINVAL;
+                       rsyserr(FERROR_XFER, errno,
+                               "copy_xattrs: lsetxattr(%s,\"%s\") failed",
+                               full_fname(dest), name);
+                       errno = save_errno;
+                       return -1;
+               }
+               free(ptr);
+       }
+
+       return 0;
+}
+
+static int64 xattr_lookup_hash(const item_list *xalp)
+{
+       const rsync_xa *rxas = xalp->items;
+       size_t i;
+       int64 key = hashlittle2(&xalp->count, sizeof xalp->count);
+
+       for (i = 0; i < xalp->count; i++) {
+               key += hashlittle2(rxas[i].name, rxas[i].name_len);
+               if (rxas[i].datum_len > MAX_FULL_DATUM)
+                       key += hashlittle2(rxas[i].datum, xattr_sum_len);
+               else
+                       key += hashlittle2(rxas[i].datum, rxas[i].datum_len);
+       }
+
+       return key;
+}
+
+static int find_matching_xattr(const item_list *xalp)
+{
+       const struct ht_int64_node *node;
+       const rsync_xa_list_ref *ref;
+       int64 key;
+
+       if (rsync_xal_h == NULL)
+               return -1;
+
+       key = xattr_lookup_hash(xalp);
+
+       node = hashtable_find(rsync_xal_h, key, NULL);
+       if (node == NULL)
+               return -1;
+
+       if (node->data == NULL)
+               return -1;
+
+       for (ref = node->data; ref != NULL; ref = ref->next) {
+               const rsync_xa_list *ptr = rsync_xal_l.items;
+               const rsync_xa *rxas1;
+               const rsync_xa *rxas2 = xalp->items;
+               size_t j;
+
+               ptr += ref->ndx;
+               rxas1 = ptr->xa_items.items;
 
                /* Wrong number of elements? */
-               if (lst[i].count != xalp->count)
+               if (ptr->xa_items.count != xalp->count)
                        continue;
                /* any elements different? */
                for (j = 0; j < xalp->count; j++) {
@@ -298,7 +433,7 @@ static int find_matching_xattr(item_list *xalp)
                        if (rxas1[j].datum_len > MAX_FULL_DATUM) {
                                if (memcmp(rxas1[j].datum + 1,
                                           rxas2[j].datum + 1,
-                                          MAX_DIGEST_LEN) != 0)
+                                          xattr_sum_len) != 0)
                                        break;
                        } else {
                                if (memcmp(rxas1[j].datum, rxas2[j].datum,
@@ -308,27 +443,56 @@ static int find_matching_xattr(item_list *xalp)
                }
                /* no differences found.  This is The One! */
                if (j == xalp->count)
-                       return i;
+                       return ref->ndx;
        }
 
        return -1;
 }
 
 /* Store *xalp on the end of rsync_xal_l */
-static void rsync_xal_store(item_list *xalp)
+static int rsync_xal_store(item_list *xalp)
 {
-       item_list *new_lst = EXPAND_ITEM_LIST(&rsync_xal_l, item_list, RSYNC_XAL_LIST_INITIAL);
+       struct ht_int64_node *node;
+       int ndx = rsync_xal_l.count; /* pre-incremented count */
+       rsync_xa_list *new_list = EXPAND_ITEM_LIST(&rsync_xal_l, rsync_xa_list, RSYNC_XAL_LIST_INITIAL);
+       rsync_xa_list_ref *new_ref;
        /* Since the following call starts a new list, we know it will hold the
         * entire initial-count, not just enough space for one new item. */
-       *new_lst = empty_xattr;
-       (void)EXPAND_ITEM_LIST(new_lst, rsync_xa, xalp->count);
-       memcpy(new_lst->items, xalp->items, xalp->count * sizeof (rsync_xa));
-       new_lst->count = xalp->count;
+       *new_list = empty_xa_list;
+       (void)EXPAND_ITEM_LIST(&new_list->xa_items, rsync_xa, xalp->count);
+       memcpy(new_list->xa_items.items, xalp->items, xalp->count * sizeof (rsync_xa));
+       new_list->xa_items.count = xalp->count;
        xalp->count = 0;
+
+       new_list->ndx = ndx;
+       new_list->key = xattr_lookup_hash(&new_list->xa_items);
+
+       if (rsync_xal_h == NULL)
+               rsync_xal_h = hashtable_create(512, HT_KEY64);
+
+       new_ref = new0(rsync_xa_list_ref);
+       new_ref->ndx = ndx;
+
+       node = hashtable_find(rsync_xal_h, new_list->key, new_ref);
+       if (node->data != (void*)new_ref) {
+               rsync_xa_list_ref *ref = node->data;
+
+               while (ref != NULL) {
+                       if (ref->next != NULL) {
+                               ref = ref->next;
+                               continue;
+                       }
+
+                       ref->next = new_ref;
+                       break;
+               }
+       }
+
+       return ndx;
 }
 
 /* Send the make_xattr()-generated xattr list for this flist entry. */
-int send_xattr(statx *sxp, int f)
+int send_xattr(int f, stat_x *sxp)
 {
        int ndx = find_matching_xattr(sxp->xattr);
 
@@ -340,32 +504,38 @@ int send_xattr(statx *sxp, int f)
                int count = sxp->xattr->count;
                write_varint(f, count);
                for (rxa = sxp->xattr->items; count--; rxa++) {
+                       size_t name_len = rxa->name_len;
+                       const char *name = rxa->name;
+                       /* Strip the rsync prefix from disguised namespaces. */
+                       if (name_len > RPRE_LEN
 #ifdef HAVE_LINUX_XATTRS
-                       write_varint(f, rxa->name_len);
+                        && am_root < 0
+#endif
+                        && name[RPRE_LEN] != '%' && HAS_PREFIX(name, RSYNC_PREFIX)) {
+                               name += RPRE_LEN;
+                               name_len -= RPRE_LEN;
+                       }
+#ifndef HAVE_LINUX_XATTRS
+                       else {
+                               /* Put everything else in the user namespace. */
+                               name_len += UPRE_LEN;
+                       }
+#endif
+                       write_varint(f, name_len);
                        write_varint(f, rxa->datum_len);
-                       write_buf(f, rxa->name, rxa->name_len);
-#else
-                       /* We strip the rsync prefix from disguised namespaces
-                        * and put everything else in the user namespace. */
-                       if (HAS_PREFIX(rxa->name, RSYNC_PREFIX)
-                        && rxa->name[RPRE_LEN] != '%') {
-                               write_varint(f, rxa->name_len - RPRE_LEN);
-                               write_varint(f, rxa->datum_len);
-                               write_buf(f, rxa->name + RPRE_LEN, rxa->name_len - RPRE_LEN);
-                       } else {
-                               write_varint(f, rxa->name_len + UPRE_LEN);
-                               write_varint(f, rxa->datum_len);
+#ifndef HAVE_LINUX_XATTRS
+                       if (name_len > rxa->name_len) {
                                write_buf(f, USER_PREFIX, UPRE_LEN);
-                               write_buf(f, rxa->name, rxa->name_len);
+                               name_len -= UPRE_LEN;
                        }
 #endif
+                       write_buf(f, name, name_len);
                        if (rxa->datum_len > MAX_FULL_DATUM)
-                               write_buf(f, rxa->datum + 1, MAX_DIGEST_LEN);
+                               write_buf(f, rxa->datum + 1, xattr_sum_len);
                        else
-                               write_buf(f, rxa->datum, rxa->datum_len);
+                               write_bigbuf(f, rxa->datum, rxa->datum_len);
                }
-               ndx = rsync_xal_l.count; /* pre-incremented count */
-               rsync_xal_store(sxp->xattr); /* adds item to rsync_xal_l */
+               ndx = rsync_xal_store(sxp->xattr); /* adds item to rsync_xal_l */
        }
 
        return ndx;
@@ -374,9 +544,10 @@ int send_xattr(statx *sxp, int f)
 /* Return a flag indicating if we need to change a file's xattrs.  If
  * "find_all" is specified, also mark any abbreviated xattrs that we
  * need so that send_xattr_request() can tell the sender about them. */
-int xattr_diff(struct file_struct *file, statx *sxp, int find_all)
+int xattr_diff(struct file_struct *file, stat_x *sxp, int find_all)
 {
-       item_list *lst = rsync_xal_l.items;
+       const rsync_xa_list *glst = rsync_xal_l.items;
+       const item_list *lst;
        rsync_xa *snd_rxa, *rec_rxa;
        int snd_cnt, rec_cnt;
        int cmp, same, xattrs_equal = 1;
@@ -389,9 +560,10 @@ int xattr_diff(struct file_struct *file, statx *sxp, int find_all)
                rec_cnt = 0;
        }
 
-       if (F_XATTR(file) >= 0)
-               lst += F_XATTR(file);
-       else
+       if (F_XATTR(file) >= 0) {
+               glst += F_XATTR(file);
+               lst = &glst->xa_items;
+       } else
                lst = &empty_xattr;
 
        snd_rxa = lst->items;
@@ -412,7 +584,7 @@ int xattr_diff(struct file_struct *file, statx *sxp, int find_all)
                else if (snd_rxa->datum_len > MAX_FULL_DATUM) {
                        same = cmp == 0 && snd_rxa->datum_len == rec_rxa->datum_len
                            && memcmp(snd_rxa->datum + 1, rec_rxa->datum + 1,
-                                     MAX_DIGEST_LEN) == 0;
+                                     xattr_sum_len) == 0;
                        /* Flag unrequested items that we need. */
                        if (!same && find_all && snd_rxa->datum[0] == XSTATE_ABBREV)
                                snd_rxa->datum[0] = XSTATE_TODO;
@@ -443,26 +615,33 @@ int xattr_diff(struct file_struct *file, statx *sxp, int find_all)
        return !xattrs_equal;
 }
 
-/* When called by the generator with a NULL fname, this tells the sender
- * which abbreviated xattr values we need.  When called by the sender
- * (with a non-NULL fname), we send all the extra xattr data it needs. */
+/* When called by the generator (with a NULL fname), this tells the sender
+ * all the abbreviated xattr values we need.  When called by the sender
+ * (with a non-NULL fname), we send all the extra xattr data it needs.
+ * The generator may also call with f_out < 0 to just change all the
+ * XSTATE_ABBREV states into XSTATE_DONE. */
 void send_xattr_request(const char *fname, struct file_struct *file, int f_out)
 {
-       item_list *lst = rsync_xal_l.items;
-       int j, cnt, prior_req = -1;
+       const rsync_xa_list *glst = rsync_xal_l.items;
+       const item_list *lst;
+       int cnt, prior_req = 0;
        rsync_xa *rxa;
 
-       lst += F_XATTR(file);
-       cnt = lst->count;
-       for (rxa = lst->items, j = 0; j < cnt; rxa++, j++) {
+       glst += F_XATTR(file);
+       lst = &glst->xa_items;
+
+       for (rxa = lst->items, cnt = lst->count; cnt--; rxa++) {
                if (rxa->datum_len <= MAX_FULL_DATUM)
                        continue;
                switch (rxa->datum[0]) {
-               case XSTATE_LOCAL:
-                       /* Items set locally will get cached by receiver. */
-                       rxa->datum[0] = XSTATE_DONE;
+               case XSTATE_ABBREV:
+                       /* Items left abbreviated matched the sender's checksum, so
+                        * the receiver will cache the local data for future use. */
+                       if (am_generator)
+                               rxa->datum[0] = XSTATE_DONE;
                        continue;
                case XSTATE_TODO:
+                       assert(f_out >= 0);
                        break;
                default:
                        continue;
@@ -471,74 +650,90 @@ void send_xattr_request(const char *fname, struct file_struct *file, int f_out)
                /* Flag that we handled this abbreviated item. */
                rxa->datum[0] = XSTATE_DONE;
 
-               write_varint(f_out, j - prior_req);
-               prior_req = j;
+               write_varint(f_out, rxa->num - prior_req);
+               prior_req = rxa->num;
 
                if (fname) {
                        size_t len = 0;
                        char *ptr;
 
                        /* Re-read the long datum. */
-                       if (!(ptr = get_xattr_data(fname, rxa->name, &len, 0)))
+                       if (!(ptr = get_xattr_data(fname, rxa->name, &len, 0))) {
+                               rprintf(FERROR_XFER, "failed to re-read xattr %s for %s\n", rxa->name, fname);
+                               write_varint(f_out, 0);
                                continue;
+                       }
 
                        write_varint(f_out, len); /* length might have changed! */
-                       write_buf(f_out, ptr, len);
+                       write_bigbuf(f_out, ptr, len);
                        free(ptr);
                }
        }
 
-       write_byte(f_out, 0); /* end the list */
-}
-
-/* Any items set locally by the generator that the receiver doesn't
- * get told about get changed back to XSTATE_ABBREV. */
-void xattr_clear_locals(struct file_struct *file)
-{
-       item_list *lst = rsync_xal_l.items;
-       rsync_xa *rxa;
-       int cnt;
-
-       if (F_XATTR(file) < 0)
-               return;
-
-       lst += F_XATTR(file);
-       cnt = lst->count;
-       for (rxa = lst->items; cnt--; rxa++) {
-               if (rxa->datum_len <= MAX_FULL_DATUM)
-                       continue;
-               if (rxa->datum[0] == XSTATE_LOCAL)
-                       rxa->datum[0] = XSTATE_ABBREV;
-       }
+       if (f_out >= 0)
+               write_byte(f_out, 0); /* end the list */
 }
 
 /* When called by the sender, read the request from the generator and mark
  * any needed xattrs with a flag that lets us know they need to be sent to
  * the receiver.  When called by the receiver, reads the sent data and
  * stores it in place of its checksum. */
-void recv_xattr_request(struct file_struct *file, int f_in)
+int recv_xattr_request(struct file_struct *file, int f_in)
 {
-       item_list *lst = rsync_xal_l.items;
+       const rsync_xa_list *glst = rsync_xal_l.items;
+       const item_list *lst;
        char *old_datum, *name;
        rsync_xa *rxa;
-       int rel_pos, cnt;
+       int rel_pos, cnt, num, got_xattr_data = 0;
 
        if (F_XATTR(file) < 0) {
                rprintf(FERROR, "recv_xattr_request: internal data error!\n");
-               exit_cleanup(RERR_STREAMIO);
+               exit_cleanup(RERR_PROTOCOL);
        }
-       lst += F_XATTR(file);
+       glst += F_XATTR(file);
+       lst = &glst->xa_items;
 
        cnt = lst->count;
        rxa = lst->items;
-       rxa -= 1;
+       num = 0;
        while ((rel_pos = read_varint(f_in)) != 0) {
-               rxa += rel_pos;
-               cnt -= rel_pos;
-               if (cnt < 0 || rxa->datum_len <= MAX_FULL_DATUM
-                || rxa->datum[0] != XSTATE_ABBREV) {
-                       rprintf(FERROR, "recv_xattr_request: internal abbrev error!\n");
-                       exit_cleanup(RERR_STREAMIO);
+               num += rel_pos;
+               if (am_sender) {
+                       /* The sender-related num values are only in order on the sender.
+                        * We use that order here to scan forward or backward as needed. */
+                       if (rel_pos < 0) {
+                               while (cnt < (int)lst->count && rxa->num > num) {
+                                       rxa--;
+                                       cnt++;
+                               }
+                       } else {
+                               while (cnt > 1 && rxa->num < num) {
+                                       rxa++;
+                                       cnt--;
+                               }
+                       }
+               } else {
+                       int j;
+                       /* The receiving side has no known num order, so we just scan
+                        * forward (w/wrap) and hope that the next value is near by. */
+                       for (j = lst->count; j > 1 && rxa->num != num; j--) {
+                               if (--cnt)
+                                       rxa++;
+                               else {
+                                       cnt = lst->count;
+                                       rxa = lst->items;
+                               }
+                       }
+               }
+               if (!cnt || rxa->num != num) {
+                       rprintf(FERROR, "[%s] could not find xattr #%d for %s\n",
+                               who_am_i(), num, f_name(file, NULL));
+                       exit_cleanup(RERR_PROTOCOL);
+               }
+               if (!XATTR_ABBREV(*rxa) || rxa->datum[0] != XSTATE_ABBREV) {
+                       rprintf(FERROR, "[%s] internal abbrev error on %s (%s, len=%ld)!\n",
+                               who_am_i(), f_name(file, NULL), rxa->name, (long)rxa->datum_len);
+                       exit_cleanup(RERR_PROTOCOL);
                }
 
                if (am_sender) {
@@ -549,26 +744,32 @@ void recv_xattr_request(struct file_struct *file, int f_in)
                old_datum = rxa->datum;
                rxa->datum_len = read_varint(f_in);
 
-               if (rxa->name_len + rxa->datum_len < rxa->name_len)
-                       out_of_memory("recv_xattr_request"); /* overflow */
+               if (SIZE_MAX - rxa->name_len < rxa->datum_len)
+                       overflow_exit("recv_xattr_request");
                rxa->datum = new_array(char, rxa->datum_len + rxa->name_len);
-               if (!rxa->datum)
-                       out_of_memory("recv_xattr_request");
                name = rxa->datum + rxa->datum_len;
                memcpy(name, rxa->name, rxa->name_len);
                rxa->name = name;
                free(old_datum);
                read_buf(f_in, rxa->datum, rxa->datum_len);
+               got_xattr_data = 1;
        }
+
+       return got_xattr_data;
 }
 
 /* ------------------------------------------------------------------------- */
 
 /* receive and build the rsync_xattr_lists */
-void receive_xattr(struct file_struct *file, int f)
+void receive_xattr(int f, struct file_struct *file)
 {
        static item_list temp_xattr = EMPTY_ITEM_LIST;
-       int count;
+       int count, num;
+#ifdef HAVE_LINUX_XATTRS
+       int need_sort = 0;
+#else
+       int need_sort = 1;
+#endif
        int ndx = read_varint(f);
 
        if (ndx < 0 || (size_t)ndx > rsync_xal_l.count) {
@@ -581,44 +782,52 @@ void receive_xattr(struct file_struct *file, int f)
                F_XATTR(file) = ndx - 1;
                return;
        }
-       
+
        if ((count = read_varint(f)) != 0) {
                (void)EXPAND_ITEM_LIST(&temp_xattr, rsync_xa, count);
                temp_xattr.count = 0;
        }
 
-       while (count--) {
+       for (num = 1; num <= count; num++) {
                char *ptr, *name;
                rsync_xa *rxa;
                size_t name_len = read_varint(f);
                size_t datum_len = read_varint(f);
-               size_t dget_len = datum_len > MAX_FULL_DATUM ? 1 + MAX_DIGEST_LEN : datum_len;
+               size_t dget_len = datum_len > MAX_FULL_DATUM ? 1 + (size_t)xattr_sum_len : datum_len;
                size_t extra_len = MIGHT_NEED_RPRE ? RPRE_LEN : 0;
-               if (dget_len + extra_len < dget_len)
-                       out_of_memory("receive_xattr"); /* overflow */
-               if (dget_len + extra_len + name_len < dget_len)
-                       out_of_memory("receive_xattr"); /* overflow */
+               if (SIZE_MAX - dget_len < extra_len || SIZE_MAX - dget_len - extra_len < name_len)
+                       overflow_exit("receive_xattr");
                ptr = new_array(char, dget_len + extra_len + name_len);
-               if (!ptr)
-                       out_of_memory("receive_xattr");
                name = ptr + dget_len + extra_len;
                read_buf(f, name, name_len);
+               if (name_len < 1 || name[name_len-1] != '\0') {
+                       rprintf(FERROR, "Invalid xattr name received (missing trailing \\0).\n");
+                       exit_cleanup(RERR_FILEIO);
+               }
                if (dget_len == datum_len)
                        read_buf(f, ptr, dget_len);
                else {
                        *ptr = XSTATE_ABBREV;
-                       read_buf(f, ptr + 1, MAX_DIGEST_LEN);
+                       read_buf(f, ptr + 1, xattr_sum_len);
+               }
+
+               if (saw_xattr_filter) {
+                       if (name_is_excluded(name, NAME_IS_XATTR, ALL_FILTERS)) {
+                               free(ptr);
+                               continue;
+                       }
                }
 #ifdef HAVE_LINUX_XATTRS
                /* Non-root can only save the user namespace. */
-                       if (am_root <= 0 && !HAS_PREFIX(name, USER_PREFIX)) {
-                               if (!am_root) {
-                                       free(ptr);
-                                       continue;
-                               }
-                               name -= RPRE_LEN;
-                               name_len += RPRE_LEN;
-                               memcpy(name, RSYNC_PREFIX, RPRE_LEN);
+               if (am_root <= 0 && !HAS_PREFIX(name, USER_PREFIX)) {
+                       if (!am_root && !saw_xattr_filter) {
+                               free(ptr);
+                               continue;
+                       }
+                       name -= RPRE_LEN;
+                       name_len += RPRE_LEN;
+                       memcpy(name, RSYNC_PREFIX, RPRE_LEN);
+                       need_sort = 1;
                }
 #else
                /* This OS only has a user namespace, so we either
@@ -636,33 +845,40 @@ void receive_xattr(struct file_struct *file, int f)
                        continue;
                }
 #endif
-               if (am_root < 0 && name_len == XSTAT_LEN + 1
-                && name[RPRE_LEN] == '%' && strcmp(name, XSTAT_ATTR) == 0) {
+               /* No rsync.%FOO attributes are copied w/o 2 -X options. */
+               if (preserve_xattrs < 2 && name_len > RPRE_LEN
+                && name[RPRE_LEN] == '%' && HAS_PREFIX(name, RSYNC_PREFIX)) {
                        free(ptr);
                        continue;
                }
+
                rxa = EXPAND_ITEM_LIST(&temp_xattr, rsync_xa, 1);
                rxa->name = name;
                rxa->datum = ptr;
                rxa->name_len = name_len;
                rxa->datum_len = datum_len;
+               rxa->num = num;
        }
 
-       ndx = rsync_xal_l.count; /* pre-incremented count */
-       rsync_xal_store(&temp_xattr); /* adds item to rsync_xal_l */
+       if (need_sort && count > 1)
+               qsort(temp_xattr.items, count, sizeof (rsync_xa), rsync_xal_compare_names);
+
+       ndx = rsync_xal_store(&temp_xattr); /* adds item to rsync_xal_l */
 
        F_XATTR(file) = ndx;
 }
 
-/* Turn the xattr data in statx into cached xattr data, setting the index
+/* Turn the xattr data in stat_x into cached xattr data, setting the index
  * values in the file struct. */
-void cache_xattr(struct file_struct *file, statx *sxp)
+void cache_tmp_xattr(struct file_struct *file, stat_x *sxp)
 {
        int ndx;
 
        if (!sxp->xattr)
                return;
 
+       if (prior_xattr_count == (size_t)-1)
+               prior_xattr_count = rsync_xal_l.count;
        ndx = find_matching_xattr(sxp->xattr);
        if (ndx < 0)
                rsync_xal_store(sxp->xattr); /* adds item to rsync_xal_l */
@@ -670,14 +886,65 @@ void cache_xattr(struct file_struct *file, statx *sxp)
        F_XATTR(file) = ndx;
 }
 
+void uncache_tmp_xattrs(void)
+{
+       if (prior_xattr_count != (size_t)-1) {
+               rsync_xa_list *xa_list_item = rsync_xal_l.items;
+               rsync_xa_list *xa_list_start = xa_list_item + prior_xattr_count;
+               xa_list_item += rsync_xal_l.count;
+               rsync_xal_l.count = prior_xattr_count;
+               while (xa_list_item-- > xa_list_start) {
+                       struct ht_int64_node *node;
+                       rsync_xa_list_ref *ref;
+
+                       rsync_xal_free(&xa_list_item->xa_items);
+
+                       if (rsync_xal_h == NULL)
+                               continue;
+
+                       node = hashtable_find(rsync_xal_h, xa_list_item->key, NULL);
+                       if (node == NULL)
+                               continue;
+
+                       if (node->data == NULL)
+                               continue;
+
+                       ref = node->data;
+                       if (xa_list_item->ndx == ref->ndx) {
+                               /* xa_list_item is the first in the list. */
+                               node->data = ref->next;
+                               free(ref);
+                               continue;
+                       }
+
+                       while (1) {
+                               rsync_xa_list_ref *next = ref->next;
+                               if (next == NULL)
+                                       break;
+                               if (xa_list_item->ndx == next->ndx) {
+                                       ref->next = next->next;
+                                       free(next);
+                                       break;
+                               }
+                               ref = next;
+                       }
+               }
+               prior_xattr_count = (size_t)-1;
+       }
+}
+
 static int rsync_xal_set(const char *fname, item_list *xalp,
-                        const char *fnamecmp, statx *sxp)
+                        const char *fnamecmp, stat_x *sxp)
 {
        rsync_xa *rxas = xalp->items;
        ssize_t list_len;
        size_t i, len;
-       char *name, *ptr, sum[MAX_DIGEST_LEN];
-       int name_len, ret = 0;
+       char *name, *ptr, sum[MAX_XATTR_DIGEST_LEN];
+#ifdef HAVE_LINUX_XATTRS
+       int user_only = am_root <= 0;
+#endif
+       size_t name_len;
+       int ret = 0;
 
        /* This puts the current name list into the "namebuf" buffer. */
        if ((list_len = get_xattr_names(fname)) < 0)
@@ -703,10 +970,10 @@ static int rsync_xal_set(const char *fname, item_list *xalp,
                                goto still_abbrev;
                        }
 
-                       sum_init(checksum_seed);
+                       sum_init(xattr_sum_nni, checksum_seed);
                        sum_update(ptr, len);
                        sum_end(sum);
-                       if (memcmp(sum, rxas[i].datum + 1, MAX_DIGEST_LEN) != 0) {
+                       if (memcmp(sum, rxas[i].datum + 1, xattr_sum_len) != 0) {
                                free(ptr);
                                goto still_abbrev;
                        }
@@ -714,16 +981,14 @@ static int rsync_xal_set(const char *fname, item_list *xalp,
                        if (fname == fnamecmp)
                                ; /* Value is already set when identical */
                        else if (sys_lsetxattr(fname, name, ptr, len) < 0) {
-                               rsyserr(FERROR, errno,
-                                       "rsync_xal_set: lsetxattr(\"%s\",\"%s\") failed",
-                                       fname, name);
+                               rsyserr(FERROR_XFER, errno,
+                                       "rsync_xal_set: lsetxattr(%s,\"%s\") failed",
+                                       full_fname(fname), name);
                                ret = -1;
                        } else /* make sure caller sets mtime */
                                sxp->st.st_mtime = (time_t)-1;
 
                        if (am_generator) { /* generator items stay abbreviated */
-                               if (rxas[i].datum[0] == XSTATE_ABBREV)
-                                       rxas[i].datum[0] = XSTATE_LOCAL;
                                free(ptr);
                                continue;
                        }
@@ -737,9 +1002,9 @@ static int rsync_xal_set(const char *fname, item_list *xalp,
                }
 
                if (sys_lsetxattr(fname, name, rxas[i].datum, rxas[i].datum_len) < 0) {
-                       rsyserr(FERROR, errno,
-                               "rsync_xal_set: lsetxattr(\"%s\",\"%s\") failed",
-                               fname, name);
+                       rsyserr(FERROR_XFER, errno,
+                               "rsync_xal_set: lsetxattr(%s,\"%s\") failed",
+                               full_fname(fname), name);
                        ret = -1;
                } else /* make sure caller sets mtime */
                        sxp->st.st_mtime = (time_t)-1;
@@ -750,13 +1015,17 @@ static int rsync_xal_set(const char *fname, item_list *xalp,
                name_len = strlen(name) + 1;
                list_len -= name_len;
 
+               if (saw_xattr_filter) {
+                       if (name_is_excluded(name, NAME_IS_XATTR, ALL_FILTERS))
+                               continue;
+               }
 #ifdef HAVE_LINUX_XATTRS
-               /* We always ignore the system namespace, and non-root
-                * ignores everything but the user namespace. */
-               if (am_root ? HAS_PREFIX(name, SYSTEM_PREFIX)
-                           : !HAS_PREFIX(name, USER_PREFIX))
+               /* Choose between ignoring the system namespace or (non-root) ignoring any non-user namespace. */
+               else if (user_only ? !HAS_PREFIX(name, USER_PREFIX) : HAS_PREFIX(name, SYSTEM_PREFIX))
                        continue;
 #endif
+               if (am_root < 0 && name_len > RPRE_LEN && name[RPRE_LEN] == '%' && strcmp(name, XSTAT_ATTR) == 0)
+                       continue;
 
                for (i = 0; i < xalp->count; i++) {
                        if (strcmp(name, rxas[i].name) == 0)
@@ -764,9 +1033,9 @@ static int rsync_xal_set(const char *fname, item_list *xalp,
                }
                if (i == xalp->count) {
                        if (sys_lremovexattr(fname, name) < 0) {
-                               rsyserr(FERROR, errno,
-                                       "rsync_xal_clear: lremovexattr(\"%s\",\"%s\") failed",
-                                       fname, name);
+                               rsyserr(FERROR_XFER, errno,
+                                       "rsync_xal_set: lremovexattr(%s,\"%s\") failed",
+                                       full_fname(fname), name);
                                ret = -1;
                        } else /* make sure caller sets mtime */
                                sxp->st.st_mtime = (time_t)-1;
@@ -777,11 +1046,11 @@ static int rsync_xal_set(const char *fname, item_list *xalp,
 }
 
 /* Set extended attributes on indicated filename. */
-int set_xattr(const char *fname, const struct file_struct *file,
-             const char *fnamecmp, statx *sxp)
+int set_xattr(const char *fname, const struct file_struct *file, const char *fnamecmp, stat_x *sxp)
 {
-       int ndx;
-       item_list *lst = rsync_xal_l.items;
+       rsync_xa_list *glst = rsync_xal_l.items;
+       item_list *lst;
+       int ndx, added_write_perm = 0;
 
        if (dry_run)
                return 1; /* FIXME: --dry-run needs to compute this value */
@@ -791,13 +1060,74 @@ int set_xattr(const char *fname, const struct file_struct *file,
                return -1;
        }
 
+#ifdef NO_SPECIAL_XATTRS
+       if (IS_SPECIAL(sxp->st.st_mode)) {
+               errno = ENOTSUP;
+               return -1;
+       }
+#endif
+#ifdef NO_DEVICE_XATTRS
+       if (IS_DEVICE(sxp->st.st_mode)) {
+               errno = ENOTSUP;
+               return -1;
+       }
+#endif
+#ifdef NO_SYMLINK_XATTRS
+       if (S_ISLNK(sxp->st.st_mode)) {
+               errno = ENOTSUP;
+               return -1;
+       }
+#endif
+
+       /* If the target file lacks write permission, we try to add it
+        * temporarily so we can change the extended attributes. */
+       if (!am_root
+#ifdef SUPPORT_LINKS
+        && !S_ISLNK(sxp->st.st_mode)
+#endif
+        && access(fname, W_OK) < 0
+        && do_chmod(fname, (sxp->st.st_mode & CHMOD_BITS) | S_IWUSR) == 0)
+               added_write_perm = 1;
+
        ndx = F_XATTR(file);
-       return rsync_xal_set(fname, lst + ndx, fnamecmp, sxp);
+       glst += ndx;
+       lst = &glst->xa_items;
+       int return_value = rsync_xal_set(fname, lst, fnamecmp, sxp);
+       if (added_write_perm) /* remove the temporary write permission */
+               do_chmod(fname, sxp->st.st_mode);
+       return return_value;
 }
 
+#ifdef SUPPORT_ACLS
+char *get_xattr_acl(const char *fname, int is_access_acl, size_t *len_p)
+{
+       const char *name = is_access_acl ? XACC_ACL_ATTR : XDEF_ACL_ATTR;
+       *len_p = 0; /* no extra data alloc needed from get_xattr_data() */
+       return get_xattr_data(fname, name, len_p, 1);
+}
+
+int set_xattr_acl(const char *fname, int is_access_acl, const char *buf, size_t buf_len)
+{
+       const char *name = is_access_acl ? XACC_ACL_ATTR : XDEF_ACL_ATTR;
+       if (sys_lsetxattr(fname, name, buf, buf_len) < 0) {
+               rsyserr(FERROR_XFER, errno,
+                       "set_xattr_acl: lsetxattr(%s,\"%s\") failed",
+                       full_fname(fname), name);
+               return -1;
+       }
+       return 0;
+}
+
+int del_def_xattr_acl(const char *fname)
+{
+       return sys_lremovexattr(fname, XDEF_ACL_ATTR);
+}
+#endif
+
 int get_stat_xattr(const char *fname, int fd, STRUCT_STAT *fst, STRUCT_STAT *xst)
 {
-       int mode, rdev_major, rdev_minor, uid, gid, len;
+       unsigned int mode;
+       int rdev_major, rdev_minor, uid, gid, len;
        char buf[256];
 
        if (am_root >= 0 || IS_DEVICE(fst->st_mode) || IS_SPECIAL(fst->st_mode))
@@ -826,7 +1156,7 @@ int get_stat_xattr(const char *fname, int fd, STRUCT_STAT *fst, STRUCT_STAT *xst
                        xst->st_gid = 0;
                        return 0;
                }
-               rsyserr(FERROR, errno, "failed to read xattr %s for %s",
+               rsyserr(FERROR_XFER, errno, "failed to read xattr %s for %s",
                        XSTAT_ATTR, full_fname(fname));
                return -1;
        }
@@ -847,7 +1177,7 @@ int get_stat_xattr(const char *fname, int fd, STRUCT_STAT *fst, STRUCT_STAT *xst
        return 0;
 }
 
-int set_stat_xattr(const char *fname, struct file_struct *file)
+int set_stat_xattr(const char *fname, struct file_struct *file, mode_t new_mode)
 {
        STRUCT_STAT fst, xst;
        dev_t rdev;
@@ -857,21 +1187,21 @@ int set_stat_xattr(const char *fname, struct file_struct *file)
                return 0;
 
        if (read_only || list_only) {
-               rsyserr(FERROR, EROFS, "failed to write xattr %s for %s",
+               rsyserr(FERROR_XFER, EROFS, "failed to write xattr %s for %s",
                        XSTAT_ATTR, full_fname(fname));
                return -1;
        }
 
        if (x_lstat(fname, &fst, &xst) < 0) {
-               rsyserr(FERROR, errno, "failed to re-stat %s",
+               rsyserr(FERROR_XFER, errno, "failed to re-stat %s",
                        full_fname(fname));
                return -1;
        }
 
        fst.st_mode &= (_S_IFMT | CHMOD_BITS);
-       fmode = file->mode & (_S_IFMT | CHMOD_BITS);
+       fmode = new_mode & (_S_IFMT | CHMOD_BITS);
 
-       if (IS_DEVICE(fmode) || IS_SPECIAL(fmode)) {
+       if (IS_DEVICE(fmode)) {
                uint32 *devp = F_RDEV_P(file);
                rdev = MAKEDEV(DEV_MAJOR(devp), DEV_MINOR(devp));
        } else
@@ -882,14 +1212,14 @@ int set_stat_xattr(const char *fname, struct file_struct *file)
             | (S_ISDIR(fst.st_mode) ? 0700 : 0600);
        if (fst.st_mode != mode)
                do_chmod(fname, mode);
-       if (!IS_DEVICE(fst.st_mode) && !IS_SPECIAL(fst.st_mode))
+       if (!IS_DEVICE(fst.st_mode))
                fst.st_rdev = 0; /* just in case */
 
        if (mode == fmode && fst.st_rdev == rdev
-        && fst.st_uid == F_UID(file) && fst.st_gid == F_GID(file)) {
+        && fst.st_uid == F_OWNER(file) && fst.st_gid == F_GROUP(file)) {
                /* xst.st_mode will be 0 if there's no current stat xattr */
                if (xst.st_mode && sys_lremovexattr(fname, XSTAT_ATTR) < 0) {
-                       rsyserr(FERROR, errno,
+                       rsyserr(FERROR_XFER, errno,
                                "delete of stat xattr failed for %s",
                                full_fname(fname));
                        return -1;
@@ -898,16 +1228,16 @@ int set_stat_xattr(const char *fname, struct file_struct *file)
        }
 
        if (xst.st_mode != fmode || xst.st_rdev != rdev
-        || xst.st_uid != F_UID(file) || xst.st_gid != F_GID(file)) {
+        || xst.st_uid != F_OWNER(file) || xst.st_gid != F_GROUP(file)) {
                char buf[256];
                int len = snprintf(buf, sizeof buf, "%o %u,%u %u:%u",
                        to_wire_mode(fmode),
                        (int)major(rdev), (int)minor(rdev),
-                       (int)F_UID(file), (int)F_GID(file));
+                       F_OWNER(file), F_GROUP(file));
                if (sys_lsetxattr(fname, XSTAT_ATTR, buf, len) < 0) {
                        if (errno == EPERM && S_ISLNK(fst.st_mode))
                                return 0;
-                       rsyserr(FERROR, errno,
+                       rsyserr(FERROR_XFER, errno,
                                "failed to write xattr %s for %s",
                                XSTAT_ATTR, full_fname(fname));
                        return -1;