Fixed failing hunks.
[rsync/rsync-patches.git] / fake-super.diff
index e8a078a..fe98e42 100644 (file)
@@ -163,7 +163,7 @@ above:
  FN_LOCAL_BOOL(lp_ignore_errors, ignore_errors)
  FN_LOCAL_BOOL(lp_ignore_nonreadable, ignore_nonreadable)
  FN_LOCAL_BOOL(lp_list, list)
-@@ -816,7 +820,7 @@ BOOL lp_load(char *pszFname, int globals
+@@ -815,7 +819,7 @@ BOOL lp_load(char *pszFname, int globals
  
        if (pszFname)
            pstrcpy(n2,pszFname);
@@ -258,7 +258,7 @@ above:
                                "failed to set permissions on %s",
 --- old/rsync.h
 +++ new/rsync.h
-@@ -707,6 +707,12 @@ typedef struct {
+@@ -704,6 +704,12 @@ typedef struct {
  
  #include "proto.h"
  
@@ -271,11 +271,10 @@ above:
  /* We have replacement versions of these if they're missing. */
  #ifndef HAVE_ASPRINTF
  int asprintf(char **ptr, const char *format, ...);
-@@ -924,3 +930,23 @@ int inet_pton(int af, const char *src, v
- #ifdef MAINTAINER_MODE
+@@ -922,6 +928,26 @@ int inet_pton(int af, const char *src, v
  const char *get_panic_action(void);
  #endif
-+
 +static inline int to_wire_mode(mode_t mode)
 +{
 +#ifdef SUPPORT_LINKS
@@ -295,6 +294,10 @@ above:
 +#endif
 +      return mode;
 +}
++
+ static inline int
+ isDigit(const char *ptr)
+ {
 --- old/rsync.yo
 +++ new/rsync.yo
 @@ -333,6 +333,7 @@ to the detailed description below for a 
@@ -418,7 +421,7 @@ above:
  
  /* These are to make syscall.o shut up. */
  int dry_run = 0;
-+int am_root = 0; /* TODO: add option to set this to -1. */
++int am_root = 0;
  int read_only = 1;
  int list_only = 0;
  int preserve_perms = 0;
@@ -434,54 +437,97 @@ above:
  int preserve_perms = 0;
 --- old/xattr.c
 +++ new/xattr.c
-@@ -28,11 +28,14 @@
- extern int dry_run;
- extern int read_only;
- extern int list_only;
-+extern int am_root;
- extern unsigned int file_struct_len;
+@@ -43,11 +43,16 @@ extern unsigned int file_struct_len;
+ #define SPRE_LEN ((int)sizeof SYSTEM_PREFIX - 1)
  
- #define RSYNC_XAL_INITIAL 5
- #define RSYNC_XAL_LIST_INITIAL 100
-+#define FAKE_XATTR "user.rsync%stat"
+ #ifdef HAVE_LINUX_XATTRS
+-#define RPRE_LEN 0
++#define MIGHT_NEED_RPRE (am_root < 0)
++#define RSYNC_PREFIX USER_PREFIX "rsync."
+ #else
++#define MIGHT_NEED_RPRE am_root
+ #define RSYNC_PREFIX "rsync."
+-#define RPRE_LEN ((int)sizeof RSYNC_PREFIX - 1)
+ #endif
++#define RPRE_LEN ((int)sizeof RSYNC_PREFIX - 1)
 +
++#define XSTAT_ATTR RSYNC_PREFIX "%stat"
++#define XSTAT_LEN ((int)sizeof XSTAT_ATTR - 1)
  typedef struct {
-       char *name;
-       char *datum;
-@@ -134,9 +137,15 @@ static int rsync_xal_get(const char *fna
-       if (name_size == 0)
-               return 0;
-       for (left = name_size, name = namebuf; left > 0 ; left -= len, name += len) {
--              rsync_xa *rxas = EXPAND_ITEM_LIST(xalp, rsync_xa, RSYNC_XAL_INITIAL);
-+              rsync_xa *rxas;
+       char *datum, *name;
+@@ -148,6 +153,10 @@ static int rsync_xal_get(const char *fna
+                       continue;
+ #endif
  
-               len = strlen(name) + 1;
-+              if (am_root < 0 && len == sizeof FAKE_XATTR
-+               && name[10] == '%' && strcmp(name, FAKE_XATTR) == 0)
++              if (am_root < 0 && name_len == XSTAT_LEN + 1
++               && name[RPRE_LEN] == '%' && strcmp(name, XSTAT_ATTR) == 0)
 +                      continue;
 +
-+              rxas = EXPAND_ITEM_LIST(xalp, rsync_xa, RSYNC_XAL_INITIAL);
-+
-               datum_size = sys_lgetxattr(fname, name, NULL, 0);
-               if (datum_size < 0) {
+               datum_len = sys_lgetxattr(fname, name, NULL, 0);
+               if (datum_len < 0) {
                        if (errno == ENOTSUP)
-@@ -288,6 +297,14 @@ void receive_xattr(struct file_struct *f
-                               out_of_memory("receive_xattr");
-                       read_buf(f, ptr, name_len);
-                       read_buf(f, ptr + name_len, datum_len);
-+
-+                      if (am_root < 0 && name_len == sizeof FAKE_XATTR
-+                       && ptr[10] == '%' && strcmp(ptr, FAKE_XATTR) == 0) {
+@@ -177,6 +186,13 @@ static int rsync_xal_get(const char *fna
+                               return -1;
+                       }
+               }
++#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 + datum_len;
+               rxas->datum = ptr;
+@@ -298,13 +314,9 @@ void receive_xattr(struct file_struct *f
+                       rsync_xa *rxa;
+                       size_t name_len = read_int(f);
+                       size_t datum_len = read_int(f);
+-#ifdef HAVE_LINUX_XATTRS
+-                      size_t extra_len = 0;
+-#else
+-                      size_t extra_len = am_root ? RPRE_LEN : 0;
++                      size_t extra_len = MIGHT_NEED_RPRE ? RPRE_LEN : 0;
+                       if (datum_len + extra_len < datum_len)
+                               out_of_memory("receive_xattr"); /* overflow */
+-#endif
+                       if (name_len + datum_len + extra_len < name_len)
+                               out_of_memory("receive_xattr"); /* overflow */
+                       ptr = new_array(char, name_len + datum_len + extra_len);
+@@ -315,9 +327,14 @@ void receive_xattr(struct file_struct *f
+                       read_buf(f, ptr, datum_len);
+ #ifdef HAVE_LINUX_XATTRS
+                       /* Non-root can only save the user namespace. */
+-                      if (!am_root && !HAS_PREFIX(name, USER_PREFIX)) {
+-                              free(ptr);
+-                              continue;
++                      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
+@@ -335,6 +352,12 @@ void receive_xattr(struct file_struct *f
+                               continue;
+                       }
+ #endif
++                      if (am_root < 0 && name_len == XSTAT_LEN + 1
++                       && name[RPRE_LEN] == '%'
++                       && strcmp(name, XSTAT_ATTR) == 0) {
 +                              free(ptr);
-+                              temp_xattr.count--;
 +                              continue;
 +                      }
-+
-                       rxa->name_len = name_len;
-                       rxa->datum_len = datum_len;
-                       rxa->name = ptr;
-@@ -373,4 +390,146 @@ int set_xattr(const char *fname, const s
+                       rxa = EXPAND_ITEM_LIST(&temp_xattr, rsync_xa, count);
+                       rxa->name = name;
+                       rxa->datum = ptr;
+@@ -414,4 +437,149 @@ int set_xattr(const char *fname, const s
        return rsync_xal_set(fname, lst + ndx); /* TODO:  This needs to return 1 if no xattrs changed! */
  }
  
@@ -499,10 +545,10 @@ above:
 +              xst = fst;
 +      if (fname) {
 +              fd = -1;
-+              len = sys_lgetxattr(fname, FAKE_XATTR, buf, sizeof buf - 1);
++              len = sys_lgetxattr(fname, XSTAT_ATTR, buf, sizeof buf - 1);
 +      } else {
 +              fname = "fd";
-+              len = sys_fgetxattr(fd, FAKE_XATTR, buf, sizeof buf - 1);
++              len = sys_fgetxattr(fd, XSTAT_ATTR, buf, sizeof buf - 1);
 +      }
 +      if (len >= (int)sizeof buf) {
 +              len = -1;
@@ -517,7 +563,7 @@ above:
 +                      return 0;
 +              }
 +              rsyserr(FERROR, errno, "failed to read xattr %s for %s",
-+                      FAKE_XATTR, full_fname(fname));
++                      XSTAT_ATTR, full_fname(fname));
 +              return -1;
 +      }
 +      buf[len] = '\0';
@@ -525,7 +571,7 @@ above:
 +      if (sscanf(buf, "%o %d,%d %d:%d",
 +                 &mode, &rdev_major, &rdev_minor, &uid, &gid) != 5) {
 +              rprintf(FERROR, "Corrupt %s xattr attached to %s: \"%s\"\n",
-+                      FAKE_XATTR, full_fname(fname), buf);
++                      XSTAT_ATTR, full_fname(fname), buf);
 +              exit_cleanup(RERR_FILEIO);
 +      }
 +
@@ -541,14 +587,14 @@ above:
 +{
 +      STRUCT_STAT fst, xst;
 +      dev_t rdev;
-+      mode_t mode;
++      mode_t mode, fmode;
 +
 +      if (dry_run)
 +              return 0;
 +
 +      if (read_only || list_only) {
 +              rsyserr(FERROR, EROFS, "failed to write xattr %s for %s",
-+                      FAKE_XATTR, full_fname(fname));
++                      XSTAT_ATTR, full_fname(fname));
 +              return -1;
 +      }
 +
@@ -558,23 +604,26 @@ above:
 +              return -1;
 +      }
 +
-+      if (IS_DEVICE(file->mode) || IS_SPECIAL(file->mode))
++      fst.st_mode &= (_S_IFMT | CHMOD_BITS);
++      fmode = file->mode & (_S_IFMT | CHMOD_BITS);
++
++      if (IS_DEVICE(fmode) || IS_SPECIAL(fmode))
 +              rdev = file->u.rdev;
 +      else
 +              rdev = 0;
 +
 +      /* Dump the special permissions and enable full owner access. */
-+      mode = (fst.st_mode & ~CHMOD_BITS) | (file->mode & ACCESSPERMS)
++      mode = (fst.st_mode & _S_IFMT) | (fmode & ACCESSPERMS)
 +           | (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))
 +              fst.st_rdev = 0; /* just in case */
 +
-+      if (mode == file->mode && fst.st_rdev == rdev
++      if (mode == fmode && fst.st_rdev == rdev
 +       && fst.st_uid == file->uid && fst.st_gid == file->gid) {
 +              /* xst.st_mode will be 0 if there's no current stat xattr */
-+              if (xst.st_mode && sys_lremovexattr(fname, FAKE_XATTR) < 0) {
++              if (xst.st_mode && sys_lremovexattr(fname, XSTAT_ATTR) < 0) {
 +                      rsyserr(FERROR, errno,
 +                              "delete of stat xattr failed for %s",
 +                              full_fname(fname));
@@ -583,19 +632,19 @@ above:
 +              return 0;
 +      }
 +
-+      if (xst.st_mode != file->mode || xst.st_rdev != rdev
++      if (xst.st_mode != fmode || xst.st_rdev != rdev
 +       || xst.st_uid != file->uid || xst.st_gid != file->gid) {
 +              char buf[256];
 +              int len = snprintf(buf, sizeof buf, "%o %u,%u %u:%u",
-+                      to_wire_mode(file->mode) & (_S_IFMT|CHMOD_BITS),
++                      to_wire_mode(fmode),
 +                      (int)major(rdev), (int)minor(rdev),
 +                      (int)file->uid, (int)file->gid);
-+              if (sys_lsetxattr(fname, FAKE_XATTR, buf, len) < 0) {
++              if (sys_lsetxattr(fname, XSTAT_ATTR, buf, len) < 0) {
 +                      if (errno == EPERM && S_ISLNK(fst.st_mode))
 +                              return 0;
 +                      rsyserr(FERROR, errno,
 +                              "failed to write xattr %s for %s",
-+                              FAKE_XATTR, full_fname(fname));
++                              XSTAT_ATTR, full_fname(fname));
 +                      return -1;
 +              }
 +      }