Make get_xattr_names() even safer at fetching the list of attr names.
[rsync/rsync.git] / xattrs.c
index 6ab9698..9eca6bd 100644 (file)
--- a/xattrs.c
+++ b/xattrs.c
@@ -47,7 +47,6 @@ extern int checksum_seed;
 #define XSTATE_ABBREV  0
 #define XSTATE_DONE    1
 #define XSTATE_TODO    2
-#define XSTATE_LOCAL   3
 
 #define USER_PREFIX "user."
 #define UPRE_LEN ((int)sizeof USER_PREFIX - 1)
@@ -70,6 +69,7 @@ extern int checksum_seed;
 typedef struct {
        char *datum, *name;
        size_t datum_len, name_len;
+       int num;
 } rsync_xa;
 
 static size_t namebuf_len = 0;
@@ -111,6 +111,7 @@ 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;
+       double arg;
 
        if (!namebuf) {
                namebuf_len = 1024;
@@ -119,39 +120,36 @@ static ssize_t get_xattr_names(const char *fname)
                        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) {
-               list_len = sys_llistxattr(fname, NULL, 0);
-               if (list_len < 0) {
+       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 = (double)namebuf_len;
+                 got_error:
                        rsyserr(FERROR_XFER, errno,
-                               "get_xattr_names: llistxattr(\"%s\",0) failed",
-                               fname);
+                               "get_xattr_names: llistxattr(\"%s\",%.0f) failed",
+                               fname, arg);
                        return -1;
                }
+               list_len = sys_llistxattr(fname, NULL, 0);
+               if (list_len < 0) {
+                       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_XFER, 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
@@ -210,14 +208,14 @@ 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 *rxa;
-
                name_len = strlen(name) + 1;
                list_len -= name_len;
 
@@ -270,8 +268,12 @@ static int rsync_xal_get(const char *fname, item_list *xalp)
                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;
 }
 
@@ -459,18 +461,19 @@ int xattr_diff(struct file_struct *file, stat_x *sxp, int find_all)
 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;
+       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++) {
+       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:
                        break;
@@ -481,8 +484,8 @@ 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;
@@ -504,27 +507,6 @@ void send_xattr_request(const char *fname, struct file_struct *file, int f_out)
        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;
-       }
-}
-
 /* 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
@@ -534,7 +516,7 @@ int recv_xattr_request(struct file_struct *file, int f_in)
        item_list *lst = rsync_xal_l.items;
        char *old_datum, *name;
        rsync_xa *rxa;
-       int rel_pos, cnt, got_xattr_data = 0;
+       int rel_pos, cnt, num, got_xattr_data = 0;
 
        if (F_XATTR(file) < 0) {
                rprintf(FERROR, "recv_xattr_request: internal data error!\n");
@@ -544,13 +526,20 @@ int recv_xattr_request(struct file_struct *file, int f_in)
 
        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");
+               num += rel_pos;
+               while (cnt && rxa->num < num) {
+                   rxa++;
+                   cnt--;
+               }
+               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_STREAMIO);
+               }
+               if (rxa->datum_len <= MAX_FULL_DATUM || rxa->datum[0] != XSTATE_ABBREV) {
+                       rprintf(FERROR, "[%s] internal abbrev error!\n", who_am_i());
                        exit_cleanup(RERR_STREAMIO);
                }
 
@@ -584,7 +573,7 @@ int recv_xattr_request(struct file_struct *file, int f_in)
 void receive_xattr(struct file_struct *file, int f)
 {
        static item_list temp_xattr = EMPTY_ITEM_LIST;
-       int count;
+       int count, num;
        int ndx = read_varint(f);
 
        if (ndx < 0 || (size_t)ndx > rsync_xal_l.count) {
@@ -603,7 +592,7 @@ void receive_xattr(struct file_struct *file, int f)
                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);
@@ -627,14 +616,14 @@ void receive_xattr(struct file_struct *file, int f)
                }
 #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) {
+                               free(ptr);
+                               continue;
+                       }
+                       name -= RPRE_LEN;
+                       name_len += RPRE_LEN;
+                       memcpy(name, RSYNC_PREFIX, RPRE_LEN);
                }
 #else
                /* This OS only has a user namespace, so we either
@@ -663,6 +652,7 @@ void receive_xattr(struct file_struct *file, int f)
                rxa->datum = ptr;
                rxa->name_len = name_len;
                rxa->datum_len = datum_len;
+               rxa->num = num;
        }
 
        ndx = rsync_xal_l.count; /* pre-incremented count */
@@ -739,8 +729,6 @@ static int rsync_xal_set(const char *fname, item_list *xalp,
                                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;
                        }