X-Git-Url: https://mattmccutchen.net/rsync/rsync-patches.git/blobdiff_plain/493b02f67f89aeb4deaf2c9a19aeaadd414071aa..2dbc45e750bbab50a34f0d49925f1f624c86d6b3:/xattrs.diff diff --git a/xattrs.diff b/xattrs.diff index 6f0c0c3..cf46b67 100644 --- a/xattrs.diff +++ b/xattrs.diff @@ -1,14 +1,34 @@ -Depends-On-Patch: acls.diff +This patch adds basic support for extended attributes. It works, but there +are some things that still needs to be improved (see TODO list below). -After applying this patch, run these commands for a successful build: +To use this patch, run these commands for a successful build: + patch -p1 mode)) -@@ -274,6 +283,10 @@ static int keep_backup(char *fname) - if (preserve_acls) - cleanup_keep_backup_acl(); + if (!(file = make_file(fname, NULL, NULL, 0, NO_FILTERS))) + return 1; /* the file could have disappeared */ +@@ -209,6 +222,12 @@ static int keep_backup(const char *fname + cache_acl(file, &sx); + } #endif +#ifdef SUPPORT_XATTRS -+ if (preserve_xattrs) -+ cleanup_keep_backup_xattr(); ++ if (preserve_xattrs) { ++ get_xattr(fname, &sx); ++ cache_xattr(file, &sx); ++ } +#endif - free(file); - if (verbose > 1) { + /* Check to see if this is a device file, or link */ + if ((am_root && preserve_devices && IS_DEVICE(file->mode)) +--- old/compat.c ++++ new/compat.c +@@ -58,6 +58,8 @@ void setup_protocol(int f_out,int f_in) + preserve_gid = ++file_extra_cnt; + if (preserve_acls && !am_sender) + preserve_acls = ++file_extra_cnt; ++ if (preserve_xattrs && !am_sender) ++ preserve_xattrs = ++file_extra_cnt; + + if (remote_protocol == 0) { + if (!read_batch) --- old/configure.in +++ new/configure.in -@@ -814,6 +814,30 @@ samba_cv_HAVE_ACL_GET_PERM_NP=yes,samba_ +@@ -883,6 +883,40 @@ samba_cv_HAVE_ACL_GET_PERM_NP=yes,samba_ AC_MSG_RESULT(no) ) +AC_CHECK_HEADERS(attr/xattr.h) ++AC_CHECK_HEADERS(sys/xattr.h) ++AC_CHECK_HEADERS(sys/extattr.h) +AC_MSG_CHECKING(whether to support extended attributes) +AC_ARG_ENABLE(xattr-support, +AC_HELP_STRING([--enable-xattr-support], [Include extended attribute support (default=no)]), @@ -84,17 +130,25 @@ After applying this patch, run these commands for a successful build: + AC_MSG_RESULT(Using Linux xattrs) + AC_DEFINE(HAVE_LINUX_XATTRS, 1, [True if you have Linux xattrs]) + ;; ++ darwin*) ++ AC_MSG_RESULT(Using OS X xattrs) ++ AC_DEFINE(HAVE_OSX_XATTRS, 1, [True if you have Mac OS X xattrs]) ++ ;; ++ freebsd*) ++ AC_MSG_RESULT(Using FreeBSD extattrs) ++ AC_DEFINE(HAVE_FREEBSD_XATTRS, 1, [True if you have FreeBSD xattrs]) ++ ;; + *) -+ AC_MSG_RESULT(Xattrs requested but not linux. Good luck) ++ AC_MSG_RESULT(Xattrs requested but not Linux or OS X. Good luck...) + ;; + esac + ;; + *) + AC_MSG_RESULT(no) -+ AC_DEFINE(HAVE_NA_XATTRS, 1, [True if you don't have extended attributes]) ++ AC_DEFINE(HAVE_NO_XATTRS, 1, [True if you don't have extended attributes]) + esac ], + AC_MSG_RESULT(no) -+ AC_DEFINE(HAVE_NO_XATTRL, 1, [True if you don't have extended attributes]) ++ AC_DEFINE(HAVE_NO_XATTRS, 1, [True if you don't have extended attributes]) +) + AC_CONFIG_FILES([Makefile lib/dummy zlib/dummy popt/dummy shconfig]) @@ -102,7 +156,7 @@ After applying this patch, run these commands for a successful build: --- old/flist.c +++ new/flist.c -@@ -45,6 +45,7 @@ extern int one_file_system; +@@ -43,6 +43,7 @@ extern int one_file_system; extern int copy_dirlinks; extern int keep_dirlinks; extern int preserve_acls; @@ -110,83 +164,79 @@ After applying this patch, run these commands for a successful build: extern int preserve_links; extern int preserve_hard_links; extern int preserve_devices; -@@ -971,6 +972,10 @@ static struct file_struct *send_file_nam - if (preserve_acls && make_acl(file, fname) < 0) - return NULL; +@@ -865,6 +866,10 @@ static struct file_struct *recv_file_ent + if (preserve_acls) + receive_acl(file, f); #endif +#ifdef SUPPORT_XATTRS -+ if (preserve_xattrs && make_xattr(file, fname) < 0) -+ return NULL; ++ if (preserve_xattrs) ++ receive_xattr(file, f ); +#endif - if (chmod_modes && !S_ISLNK(file->mode)) - file->mode = tweak_mode(file->mode, chmod_modes); -@@ -986,12 +991,20 @@ static struct file_struct *send_file_nam - if (preserve_acls) - send_acl(file, f); + if (S_ISREG(mode) || S_ISLNK(mode)) + stats.total_size += file_length; +@@ -1137,7 +1142,7 @@ static struct file_struct *send_file_nam + int flags, int filter_flags) + { + struct file_struct *file; +-#ifdef SUPPORT_ACLS ++#if defined SUPPORT_ACLS || defined SUPPORT_XATTRS + statx sx; + #endif + +@@ -1156,6 +1161,13 @@ static struct file_struct *send_file_nam + return NULL; + } #endif +#ifdef SUPPORT_XATTRS -+ if (preserve_xattrs) -+ send_xattr(file, f); ++ if (preserve_xattrs && f >= 0) { ++ sx.xattr = NULL; ++ if (get_xattr(fname, &sx) < 0) ++ return NULL; ++ } +#endif - } else { - #ifdef SUPPORT_ACLS - /* Cleanup unsent ACL(s). */ + + maybe_emit_filelist_progress(flist->count + flist_count_offset); + +@@ -1167,6 +1179,10 @@ static struct file_struct *send_file_nam if (preserve_acls) - send_acl(file, -1); + send_acl(&sx, f); #endif +#ifdef SUPPORT_XATTRS + if (preserve_xattrs) -+ send_xattr(file, -1); ++ send_xattr(&sx, f); +#endif } return file; } -@@ -1384,6 +1397,10 @@ struct file_list *recv_file_list(int f) - if (preserve_acls) - receive_acl(file, f); - #endif -+#ifdef SUPPORT_XATTRS -+ if (preserve_xattrs) -+ receive_xattr(file, f ); -+#endif - - if (S_ISREG(file->mode) || S_ISLNK(file->mode)) - stats.total_size += file->length; -@@ -1411,6 +1428,10 @@ struct file_list *recv_file_list(int f) - if (preserve_acls) - sort_file_acl_index_lists(); - #endif -+#ifdef SUPPORT_XATTRS -+ if (preserve_xattrs) -+ sort_file_xattr_index_lists(); -+#endif - - if (f >= 0) { - recv_uid_list(f, flist); --- old/lib/sysxattr.c +++ new/lib/sysxattr.c -@@ -0,0 +1,41 @@ -+/* Extended attribute support for rsync. */ -+/* This file Copyright (C) 2004 Red Hat, Inc. */ -+/* Written by Jay Fenlason */ -+ -+/* This program is free software; you can redistribute it and/or modify -+ it under the terms of the GNU General Public License as published by -+ the Free Software Foundation; either version 2 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 -+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -+ 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., 675 Mass Ave, Cambridge, MA 02139, USA. -+*/ +@@ -0,0 +1,135 @@ ++/* ++ * Extended attribute support for rsync. ++ * ++ * Copyright (C) 2004 Red Hat, Inc. ++ * Written by Jay Fenlason. ++ * ++ * This program is free software; you can redistribute it and/or modify ++ * it under the terms of the GNU General Public License as published by ++ * the Free Software Foundation; either version 2 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 ++ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the ++ * 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. ++ */ + +#include "rsync.h" ++#include "sysxattr.h" ++ ++#ifdef SUPPORT_XATTRS + +#if defined HAVE_LINUX_XATTRS + @@ -195,9 +245,19 @@ After applying this patch, run these commands for a successful build: + return lgetxattr(path, name, value, size); +} + -+int sys_lsetxattr(const char *path, const char *name, const void *value, size_t size, int flags) ++ssize_t sys_fgetxattr(int filedes, const char *name, void *value, size_t size) ++{ ++ return fgetxattr(filedes, name, value, size); ++} ++ ++int sys_lsetxattr(const char *path, const char *name, const void *value, size_t size) ++{ ++ return lsetxattr(path, name, value, size, 0); ++} ++ ++int sys_lremovexattr(const char *path, const char *name) +{ -+ return lsetxattr(path, name, value, size, flags); ++ return lremovexattr(path, name); +} + +ssize_t sys_llistxattr(const char *path, char *list, size_t size) @@ -205,24 +265,120 @@ After applying this patch, run these commands for a successful build: + return llistxattr(path, list, size); +} + ++#elif HAVE_OSX_XATTRS ++ ++ssize_t sys_lgetxattr(const char *path, const char *name, void *value, size_t size) ++{ ++ return getxattr(path, name, value, size, 0, XATTR_NOFOLLOW); ++} ++ ++ssize_t sys_fgetxattr(int filedes, const char *name, void *value, size_t size) ++{ ++ return fgetxattr(filedes, name, value, size, 0, 0); ++} ++ ++int sys_lsetxattr(const char *path, const char *name, const void *value, size_t size) ++{ ++ return setxattr(path, name, value, size, 0, XATTR_NOFOLLOW); ++} ++ ++int sys_lremovexattr(const char *path, const char *name) ++{ ++ return removexattr(path, name, XATTR_NOFOLLOW); ++} ++ ++ssize_t sys_llistxattr(const char *path, char *list, size_t size) ++{ ++ return listxattr(path, list, size, XATTR_NOFOLLOW); ++} ++ ++#elif HAVE_FREEBSD_XATTRS ++ ++ssize_t sys_lgetxattr(const char *path, const char *name, void *value, size_t size) ++{ ++ return extattr_get_link(path, EXTATTR_NAMESPACE_USER, name, value, size); ++} ++ ++ssize_t sys_fgetxattr(int filedes, const char *name, void *value, size_t size) ++{ ++ return extattr_get_fd(filedes, EXTATTR_NAMESPACE_USER, name, value, size); ++} ++ ++int sys_lsetxattr(const char *path, const char *name, const void *value, size_t size) ++{ ++ return extattr_set_link(path, EXTATTR_NAMESPACE_USER, name, value, size); ++} ++ ++int sys_lremovexattr(const char *path, const char *name) ++{ ++ return extattr_delete_link(path, EXTATTR_NAMESPACE_USER, name); ++} ++ ++ssize_t sys_llistxattr(const char *path, char *list, size_t size) ++{ ++ unsigned char keylen; ++ ssize_t off, len = extattr_list_link(path, EXTATTR_NAMESPACE_USER, list, size); ++ ++ if (len <= 0 || (size_t)len > size) ++ return len; ++ ++ /* FreeBSD puts a single-byte length before each string, with no '\0' ++ * terminator. We need to change this into a series of null-terminted ++ * strings. Since the size is the same, we can simply transform the ++ * output in place. */ ++ for (off = 0; off < len; off += keylen + 1) { ++ keylen = ((unsigned char*)list)[off]; ++ if (off + keylen >= len) { ++ /* Should be impossible, but kernel bugs happen! */ ++ errno = EINVAL; ++ return -1; ++ } ++ memmove(list+off, list+off+1, keylen); ++ list[off+keylen] = '\0'; ++ } ++ ++ return len; ++} ++ +#else + -+#endif /* No xattrs */ ++#error You need to create xattr compatibility functions. ++ ++#endif ++ ++#endif /* SUPPORT_XATTRS */ --- old/lib/sysxattr.h +++ new/lib/sysxattr.h -@@ -0,0 +1,9 @@ -+#if defined HAVE_LINUX_XATTRS +@@ -0,0 +1,26 @@ ++#ifdef SUPPORT_XATTRS ++ ++#if defined HAVE_ATTR_XATTR_H ++#include ++#elif defined HAVE_SYS_XATTR_H ++#include ++#elif defined HAVE_SYS_EXTATTR_H ++#include ++#endif ++ ++/* Linux 2.4 does not define this as a distinct errno value: */ ++#ifndef ENOATTR ++#define ENOATTR ENODATA ++#endif + +ssize_t sys_lgetxattr(const char *path, const char *name, void *value, size_t size); -+int sys_lsetxattr(const char *path, const char *name, const void *value, size_t size, int flags); ++ssize_t sys_fgetxattr(int filedes, const char *name, void *value, size_t size); ++int sys_lsetxattr(const char *path, const char *name, const void *value, size_t size); ++int sys_lremovexattr(const char *path, const char *name); +ssize_t sys_llistxattr(const char *path, char *list, size_t size); + +#else + -+#endif /* No xattrs */ ++/* No xattrs available */ ++ ++#endif --- old/options.c +++ new/options.c -@@ -46,6 +46,7 @@ int copy_links = 0; +@@ -48,6 +48,7 @@ int copy_links = 0; int preserve_links = 0; int preserve_hard_links = 0; int preserve_acls = 0; @@ -230,7 +386,7 @@ After applying this patch, run these commands for a successful build: int preserve_perms = 0; int preserve_executability = 0; int preserve_devices = 0; -@@ -196,6 +197,7 @@ static void print_rsync_version(enum log +@@ -201,6 +202,7 @@ static void print_rsync_version(enum log char const *have_inplace = "no "; char const *hardlinks = "no "; char const *acls = "no "; @@ -238,7 +394,7 @@ After applying this patch, run these commands for a successful build: char const *links = "no "; char const *ipv6 = "no "; STRUCT_STAT *dumstat; -@@ -215,7 +217,9 @@ static void print_rsync_version(enum log +@@ -220,7 +222,9 @@ static void print_rsync_version(enum log #ifdef SUPPORT_ACLS acls = ""; #endif @@ -249,29 +405,37 @@ After applying this patch, run these commands for a successful build: #ifdef SUPPORT_LINKS links = ""; #endif -@@ -229,9 +233,9 @@ static void print_rsync_version(enum log - rprintf(f, "Copyright (C) 1996-2006 by Andrew Tridgell, Wayne Davison, and others.\n"); - rprintf(f, "\n"); - rprintf(f, "Capabilities: %d-bit files, %ssocketpairs, " -- "%shard links, %sACLs, %ssymlinks, batchfiles,\n", -+ "%shard links, %sACLs, %sxattrs, %ssymlinks, batchfiles,\n", - (int) (sizeof (OFF_T) * 8), -- got_socketpair, hardlinks, acls, links); -+ got_socketpair, hardlinks, acls, xattrs, links); +@@ -239,8 +243,8 @@ static void print_rsync_version(enum log + (int)(sizeof (int64) * 8)); + rprintf(f, " %ssocketpairs, %shardlinks, %ssymlinks, %sIPv6, batchfiles, %sinplace,\n", + got_socketpair, hardlinks, links, ipv6, have_inplace); +- rprintf(f, " %sappend, %sACLs\n", +- have_inplace, acls); ++ rprintf(f, " %sappend, %sACLs, %sxattrs\n", ++ have_inplace, acls, xattrs); - /* Note that this field may not have type ino_t. It depends - * on the complicated interaction between largefile feature -@@ -304,6 +308,9 @@ void usage(enum logcode F) + #ifdef MAINTAINER_MODE + rprintf(f, "Panic Action: \"%s\"\n", get_panic_action()); +@@ -286,7 +290,7 @@ void usage(enum logcode F) + rprintf(F," -q, --quiet suppress non-error messages\n"); + rprintf(F," --no-motd suppress daemon-mode MOTD (see manpage caveat)\n"); + rprintf(F," -c, --checksum skip based on checksum, not mod-time & size\n"); +- rprintf(F," -a, --archive archive mode; same as -rlptgoD (no -H, -A)\n"); ++ rprintf(F," -a, --archive archive mode; equals -rlptgoD (no -H,-A,-X)\n"); + rprintf(F," --no-OPTION turn off an implied OPTION (e.g. --no-D)\n"); + rprintf(F," -r, --recursive recurse into directories\n"); + rprintf(F," -R, --relative use relative path names\n"); +@@ -311,6 +315,9 @@ void usage(enum logcode F) #ifdef SUPPORT_ACLS rprintf(F," -A, --acls preserve ACLs (implies --perms)\n"); #endif +#ifdef SUPPORT_XATTRS + rprintf(F," -X, --xattrs preserve extended attributes (implies --perms)\n"); +#endif - rprintf(F," --chmod=CHMOD change destination permissions\n"); rprintf(F," -o, --owner preserve owner (super-user only)\n"); rprintf(F," -g, --group preserve group\n"); -@@ -422,6 +429,9 @@ static struct poptOption long_options[] + rprintf(F," --devices preserve device files (super-user only)\n"); +@@ -434,6 +441,9 @@ static struct poptOption long_options[] {"acls", 'A', POPT_ARG_NONE, 0, 'A', 0, 0 }, {"no-acls", 0, POPT_ARG_VAL, &preserve_acls, 0, 0, 0 }, {"no-A", 0, POPT_ARG_VAL, &preserve_acls, 0, 0, 0 }, @@ -281,7 +445,7 @@ After applying this patch, run these commands for a successful build: {"times", 't', POPT_ARG_VAL, &preserve_times, 1, 0, 0 }, {"no-times", 0, POPT_ARG_VAL, &preserve_times, 0, 0, 0 }, {"no-t", 0, POPT_ARG_VAL, &preserve_times, 0, 0, 0 }, -@@ -1096,6 +1106,17 @@ int parse_arguments(int *argc, const cha +@@ -1122,6 +1132,17 @@ int parse_arguments(int *argc, const cha return 0; #endif @@ -295,11 +459,11 @@ After applying this patch, run these commands for a successful build: + "extended attributes are not supported on this %s\n", + am_server ? "server" : "client"); + return 0; -+#endif /* SUPPORT_XATTRS */ ++#endif default: /* A large opt value means that set_refuse_options() -@@ -1544,6 +1565,10 @@ void server_options(char **args,int *arg +@@ -1590,6 +1611,10 @@ void server_options(char **args,int *arg if (preserve_acls) argstr[x++] = 'A'; #endif @@ -307,60 +471,91 @@ After applying this patch, run these commands for a successful build: + if (preserve_xattrs) + argstr[x++] = 'X'; +#endif - if (preserve_uid) - argstr[x++] = 'o'; - if (preserve_gid) + if (recurse) + argstr[x++] = 'r'; + if (always_checksum) --- old/rsync.c +++ new/rsync.c -@@ -34,6 +34,7 @@ extern int verbose; +@@ -33,6 +33,7 @@ + extern int verbose; extern int dry_run; - extern int daemon_log_format_has_i; extern int preserve_acls; +extern int preserve_xattrs; extern int preserve_perms; extern int preserve_executability; extern int preserve_times; -@@ -211,6 +212,10 @@ int set_file_attrs(char *fname, struct f - if (preserve_acls && set_acl(fname, file, st->st_mode) == 0) - updated = 1; - #endif +@@ -321,6 +322,10 @@ int set_file_attrs(char *fname, struct f + if (daemon_chmod_modes && !S_ISLNK(new_mode)) + new_mode = tweak_mode(new_mode, daemon_chmod_modes); + +#ifdef SUPPORT_XATTRS -+ if (preserve_xattrs && set_xattr(fname, file) == 0) ++ if (preserve_xattrs && set_xattr(fname, file, sxp) == 0) + updated = 1; +#endif - - #ifdef HAVE_CHMOD - if ((st->st_mode & CHMOD_BITS) != (file->mode & CHMOD_BITS)) { + #ifdef SUPPORT_ACLS + /* It's OK to call set_acl() now, even for a dir, as the generator + * will enable owner-writability using chmod, if necessary. --- old/rsync.h +++ new/rsync.h -@@ -672,6 +672,14 @@ struct chmod_mode_struct; +@@ -554,6 +554,10 @@ struct idev_node { + #define ACLS_NEED_MASK 1 #endif - #include "smb_acls.h" -+#ifdef HAVE_LINUX_XATTRS ++#ifndef HAVE_NO_XATTRS +#define SUPPORT_XATTRS 1 +#endif + -+#if defined SUPPORT_XATTRS && defined HAVE_ATTR_XATTR_H -+#include + #define GID_NONE ((gid_t)-1) + + union file_extras { +@@ -574,6 +578,7 @@ extern int file_extra_cnt; + extern int preserve_uid; + extern int preserve_gid; + extern int preserve_acls; ++extern int preserve_xattrs; + + #define FILE_STRUCT_LEN (offsetof(struct file_struct, basename)) + #define EXTRA_LEN (sizeof (union file_extras)) +@@ -607,6 +612,7 @@ extern int preserve_acls; + #define F_OWNER(f) REQ_EXTRA(f, preserve_uid)->unum + #define F_GROUP(f) REQ_EXTRA(f, preserve_gid)->unum + #define F_ACL(f) REQ_EXTRA(f, preserve_acls)->unum ++#define F_XATTR(f) REQ_EXTRA(f, preserve_xattrs)->unum + + /* These items are per-entry optional and mutally exclusive: */ + #define F_HL_GNUM(f) OPT_EXTRA(f, LEN64_BUMP(f))->num +@@ -798,6 +804,9 @@ typedef struct { + struct rsync_acl *acc_acl; /* access ACL */ + struct rsync_acl *def_acl; /* default ACL */ + #endif ++#ifdef SUPPORT_XATTRS ++ item_list *xattr; +#endif -+ - #include "proto.h" + } statx; - /* We have replacement versions of these if they're missing. */ + #define ACL_READY(sx) ((sx).acc_acl != NULL) --- old/rsync.yo +++ new/rsync.yo -@@ -322,6 +322,7 @@ to the detailed description below for a - -p, --perms preserve permissions +@@ -301,7 +301,7 @@ to the detailed description below for a + -q, --quiet suppress non-error messages + --no-motd suppress daemon-mode MOTD (see caveat) + -c, --checksum skip based on checksum, not mod-time & size +- -a, --archive archive mode; same as -rlptgoD (no -H, -A) ++ -a, --archive archive mode; equals -rlptgoD (no -H,-A,-X) + --no-OPTION turn off an implied OPTION (e.g. --no-D) + -r, --recursive recurse into directories + -R, --relative use relative path names +@@ -324,6 +324,7 @@ to the detailed description below for a -E, --executability preserve executability + --chmod=CHMOD affect file and/or directory permissions -A, --acls preserve ACLs (implies -p) [non-standard] + -X, --xattrs preserve extended attrs (implies -p) [n.s.] - --chmod=CHMOD change destination permissions -o, --owner preserve owner (super-user only) -g, --group preserve group -@@ -802,6 +803,11 @@ dit(bf(-A, --acls)) This option causes r - ACLs to be the same as the source ACLs. This nonstandard option only - works if the remote rsync also supports it. bf(--acls) implies bf(--perms). + --devices preserve device files (super-user only) +@@ -835,6 +836,11 @@ The ACL-sending protocol used by this ve + the patch that was shipped with 2.6.8. Sending ACLs to an older version + of the ACL patch is not supported. +dit(bf(-X, --xattrs)) This option causes rsync to update the remote +extended attributes to be the same as the local ones. This will work @@ -372,25 +567,28 @@ After applying this patch, run these commands for a successful build: transfer. The resulting value is treated as though it was the permissions --- old/xattr.c +++ new/xattr.c -@@ -0,0 +1,521 @@ -+/* Extended Attribute support for rsync */ -+/* Copyright (C) 2004 Red Hat, Inc */ -+/* Written by Jay Fenlason, vaguely based on the ACLs patch */ -+ -+/* This program is free software; you can redistribute it and/or modify -+ it under the terms of the GNU General Public License as published by -+ the Free Software Foundation; either version 2 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 -+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -+ 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., 675 Mass Ave, Cambridge, MA 02139, USA. -+*/ +@@ -0,0 +1,413 @@ ++/* ++ * Extended Attribute support for rsync. ++ * Written by Jay Fenlason, vaguely based on the ACLs patch. ++ * ++ * Copyright (C) 2004 Red Hat, Inc. ++ * Copyright (C) 2006 Wayne Davison ++ * ++ * This program is free software; you can redistribute it and/or modify ++ * it under the terms of the GNU General Public License as published by ++ * the Free Software Foundation; either version 2 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 ++ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the ++ * 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. ++ */ + +#include "rsync.h" +#include "lib/sysxattr.h" @@ -398,499 +596,388 @@ After applying this patch, run these commands for a successful build: +#ifdef SUPPORT_XATTRS + +extern int dry_run; ++extern int am_root; ++extern int read_only; ++extern int list_only; + +#define RSYNC_XAL_INITIAL 5 +#define RSYNC_XAL_LIST_INITIAL 100 + -+typedef struct { -+ size_t name_len; -+ char *name; -+ size_t datum_len; -+ char *datum; -+} rsync_xa; ++#define HAS_PREFIX(str, prfx) (*(str) == *(prfx) \ ++ && strncmp(str, prfx, sizeof (prfx) - 1) == 0) + -+typedef struct { -+ size_t count; -+ size_t alloc; -+ rsync_xa *rxas; -+} rsync_xal; ++#define USER_PREFIX "user." ++#define UPRE_LEN ((int)sizeof USER_PREFIX - 1) ++#define SYSTEM_PREFIX "system." ++#define SPRE_LEN ((int)sizeof SYSTEM_PREFIX - 1) ++ ++#ifdef HAVE_LINUX_XATTRS ++#define RPRE_LEN 0 ++#else ++#define RSYNC_PREFIX "rsync." ++#define RPRE_LEN ((int)sizeof RSYNC_PREFIX - 1) ++#endif + +typedef struct { -+ size_t count; -+ size_t alloc; -+ rsync_xal *rxals; -+} rsync_xal_list; ++ char *datum, *name; ++ size_t datum_len, name_len; ++} rsync_xa; + +static size_t namebuf_len = 0; +static char *namebuf = NULL; + -+static size_t datumbuf_len = 0; -+static char *datumbuf = NULL; -+ -+static rsync_xal curr_rsync_xal = { 0, 0, NULL }; -+static rsync_xal_list rsync_xal_l = { 0, 0, NULL }; -+ ++static item_list empty_xattr = EMPTY_ITEM_LIST; ++static item_list rsync_xal_l = EMPTY_ITEM_LIST; + +/* ------------------------------------------------------------------------- */ + -+/* the below stuff is only used by the receiver */ -+ -+/* structure to hold index to rsync_xal_l member corresponding to -+ * flist->files[i] */ -+ -+typedef struct { -+ const struct file_struct *file; -+ int xalidx; -+} file_xal_index; -+ -+typedef struct { -+ size_t count; -+ size_t alloc; -+ file_xal_index *filexalidxs; -+} file_xal_index_list; -+ -+static file_xal_index_list fxil = {0, 0, NULL }; -+ -+/* stuff for redirecting calls to set_acl() from set_perms() -+ * for keep_backup() */ -+static const struct file_struct *backup_orig_file = NULL; -+static const char null_string[] = ""; -+static const char *backup_orig_fname = null_string; -+static const char *backup_dest_fname = null_string; -+static rsync_xal backup_xal; -+ -+/* ------------------------------------------------------------------------- */ -+ -+static void rsync_xal_free(rsync_xal *x) ++static void rsync_xal_free(item_list *xalp) +{ + size_t i; ++ rsync_xa *rxas = xalp->items; + -+ for (i = 0; i < x->count; i++) { -+ free(x->rxas[i].name); -+ /* free(x->rxas[i].value); */ ++ for (i = 0; i < xalp->count; i++) { ++ free(rxas[i].datum); ++ /*free(rxas[i].name);*/ + } -+ x->count = 0; ++ xalp->count = 0; +} + -+static int rsync_xal_compare_names(const void *x1, const void *x2) ++void free_xattr(statx *sxp) +{ -+ const rsync_xa *xa1; -+ const rsync_xa *xa2; ++ rsync_xal_free(sxp->xattr); ++ free(sxp->xattr); ++ sxp->xattr = NULL; ++} + -+ xa1 = x1; -+ xa2 = x2; ++static int rsync_xal_compare_names(const void *x1, const void *x2) ++{ ++ const rsync_xa *xa1 = x1; ++ const rsync_xa *xa2 = x2; + return strcmp(xa1->name, xa2->name); +} + -+static int rsync_xal_get(const char *fname, rsync_xal *x) ++static int rsync_xal_get(const char *fname, item_list *xalp) +{ -+ ssize_t name_size; -+ ssize_t datum_size; -+ ssize_t left; -+ char *name; -+ size_t len; -+ char *ptr; ++ ssize_t list_len, name_len, datum_len; ++ char *name, *ptr; + + if (!namebuf) { -+ namebuf_len = 100; ++ namebuf_len = 1024; + namebuf = new_array(char, namebuf_len); -+ datumbuf_len = 100; -+ datumbuf = new_array(char, datumbuf_len); -+ if (!namebuf || !datumbuf) ++ if (!namebuf) + out_of_memory("rsync_xal_get"); + } + -+ name_size = sys_llistxattr(fname, namebuf, namebuf_len); -+ if (name_size > (ssize_t)namebuf_len) { -+ name_size = -1; ++ /* 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 (name_size < 0) { ++ if (list_len < 0) { + if (errno == ENOTSUP) -+ return -1; ++ return 0; + if (errno == ERANGE) { -+ name_size = sys_llistxattr(fname, NULL, 0); -+ if (name_size < 0) { -+ rprintf(FERROR, "%s: rsync_xal_get: llistxattr: %s\n", -+ fname, strerror(errno)); ++ list_len = sys_llistxattr(fname, NULL, 0); ++ if (list_len < 0) { ++ rsyserr(FERROR, errno, ++ "rsync_xal_get: llistxattr(\"%s\",0) failed", ++ fname); + return -1; + } -+ namebuf = realloc_array(namebuf, char, name_size + 1); ++ namebuf = realloc_array(namebuf, char, list_len + 1024); + if (!namebuf) + out_of_memory("rsync_xal_get"); -+ namebuf_len = name_size; -+ name_size = sys_llistxattr(fname, namebuf, namebuf_len); -+ if (name_size < 0) { -+ rprintf(FERROR, -+ "%s: rsync_xal_get: re-llistxattr failed: %s\n", -+ fname, strerror(errno)); ++ namebuf_len = list_len + 1024; ++ list_len = sys_llistxattr(fname, namebuf, namebuf_len); ++ if (list_len < 0) { ++ rsyserr(FERROR, errno, ++ "rsync_xal_get: llistxattr(\"%s\",%ld) failed", ++ fname, (long)namebuf_len); + return -1; + } + } else { -+ rprintf(FERROR, -+ "%s: rsync_xal_get: llistxattr failed: %s\n", -+ fname, strerror(errno)); ++ rsyserr(FERROR, errno, ++ "rsync_xal_get: llistxattr(\"%s\",%ld) failed", ++ fname, (long)namebuf_len); + return -1; + } + } -+ rsync_xal_free(x); -+ if (name_size == 0) -+ return 0; -+ for (left = name_size, name = namebuf; left > 0 ; left -= len, name += len) { -+ len = strlen(name) + 1; -+ -+ if (x->count >= x->alloc) { -+ size_t new_alloc; -+ rsync_xa *new_rxas; -+ -+ new_alloc = x->alloc < RSYNC_XAL_INITIAL ? RSYNC_XAL_INITIAL : x->alloc * 2; -+ new_rxas = realloc_array(x->rxas, rsync_xa, new_alloc); -+ if (!new_rxas) -+ out_of_memory("rsync_xal_get"); -+ x->alloc = new_alloc; -+ x->rxas = new_rxas; -+ } -+ datum_size = sys_lgetxattr(fname, name, datumbuf, datumbuf_len); -+ if (datum_size > (ssize_t)datumbuf_len) { -+ datum_size = -1; -+ errno = ERANGE; -+ } -+ if (datum_size < 0) { ++ ++ for (name = namebuf; list_len > 0; name += name_len) { ++ rsync_xa *rxas; ++ ++ name_len = strlen(name) + 1; ++ list_len -= name_len; ++ ++#ifdef HAVE_LINUX_XATTRS ++ /* We don't send the system namespace. */ ++ if (HAS_PREFIX(name, SYSTEM_PREFIX)) ++ continue; ++#endif ++ ++ datum_len = sys_lgetxattr(fname, name, NULL, 0); ++ if (datum_len < 0) { + if (errno == ENOTSUP) + return -1; -+ if (errno == ERANGE) { -+ datum_size = sys_lgetxattr(fname, name, NULL, 0); -+ if (datum_size < 0) { -+ rprintf(FERROR, -+ "%s: rsync_xal_get: lgetxattr %s failed: %s\n", -+ fname, name, strerror(errno)); -+ return -1; -+ } -+ datumbuf = realloc_array(datumbuf, char, datum_size + 1); -+ if (!datumbuf) -+ out_of_memory("rsync_xal_get"); -+ datumbuf_len = datum_size; -+ datum_size = sys_lgetxattr(fname, name, datumbuf, datumbuf_len); -+ if (datum_size < 0) { ++ rsyserr(FERROR, errno, ++ "rsync_xal_get: lgetxattr(\"%s\",\"%s\",0) failed", ++ fname, name); ++ return -1; ++ } ++ ptr = new_array(char, name_len + datum_len); ++ if (!ptr) ++ out_of_memory("rsync_xal_get"); ++ if (datum_len) { ++ ssize_t len = sys_lgetxattr(fname, name, ptr, datum_len); ++ if (len != datum_len) { ++ if (len < 0) { ++ rsyserr(FERROR, errno, ++ "rsync_xal_get: lgetxattr(\"%s\",\"%s\",%ld)" ++ " failed", fname, name, (long)datum_len); ++ } else { + rprintf(FERROR, -+ "%s: rsync_xal_get: re-lgetxattr of %s failed: %s\n", -+ name, fname, strerror(errno)); -+ return -1; ++ "rsync_xal_get: lgetxattr(\"%s\",\"%s\",%ld)" ++ " returned %ld\n", fname, name, ++ (long)datum_len, (long)len); + } -+ } else { -+ rprintf(FERROR, -+ "%s: rsync_xal_get: lgetxattr %s failed: %s\n", -+ fname, name, strerror(errno)); ++ free(ptr); + return -1; + } + } -+ ptr = new_array(char, len + datum_size); -+ if (!ptr) -+ out_of_memory("rsync_xal_get"); -+ strcpy(ptr, name); -+ if (datum_size) -+ memcpy(ptr + len, datumbuf, datum_size); -+ x->rxas[curr_rsync_xal.count].name_len = len; -+ x->rxas[curr_rsync_xal.count].name = ptr; -+ x->rxas[curr_rsync_xal.count].datum_len = datum_size; -+ x->rxas[curr_rsync_xal.count].datum = ptr + len; -+ x->count++; -+ } -+ if (x->count > 1) { -+ qsort(x->rxas, x->count, sizeof (rsync_xa), rsync_xal_compare_names); ++ rxas = EXPAND_ITEM_LIST(xalp, rsync_xa, RSYNC_XAL_INITIAL); ++ rxas->name = ptr + datum_len; ++ rxas->datum = ptr; ++ rxas->name_len = name_len; ++ rxas->datum_len = datum_len; ++ memcpy(rxas->name, name, name_len); + } ++ if (xalp->count > 1) ++ qsort(xalp->items, xalp->count, sizeof (rsync_xa), rsync_xal_compare_names); + return 0; +} + -+ -+/* generate the xattr(s) for this flist entry; -+ * xattr(s) are either sent or cleaned-up by send_xattr() below */ -+ -+int make_xattr(UNUSED(const struct file_struct *file), const char *fname) ++/* Read the xattr(s) for this filename. */ ++int get_xattr(const char *fname, statx *sxp) +{ -+ rsync_xal_get(fname, &curr_rsync_xal); -+ return 0; /* TODO: This needs to return 1 if no xattrs changed! */ ++ sxp->xattr = new(item_list); ++ *sxp->xattr = empty_xattr; ++ if (rsync_xal_get(fname, sxp->xattr) < 0) { ++ free_xattr(sxp); ++ return -1; ++ } ++ return 0; +} + -+static ssize_t rsync_xal_find_matching(void) ++static int find_matching_xattr(item_list *xalp) +{ -+ size_t i; -+ size_t j; ++ size_t i, j; ++ item_list *lst = rsync_xal_l.items; + + for (i = 0; i < rsync_xal_l.count; i++) { ++ rsync_xa *rxas1 = lst[i].items; ++ rsync_xa *rxas2 = xalp->items; ++ + /* Wrong number of elements? */ -+ if (rsync_xal_l.rxals[i].count != curr_rsync_xal.count) ++ if (lst[i].count != xalp->count) + continue; + /* any elements different? */ -+ for (j = 0; j < curr_rsync_xal.count; j++) { -+ if (rsync_xal_l.rxals[i].rxas[j].name_len != curr_rsync_xal.rxas[j].name_len -+ || rsync_xal_l.rxals[i].rxas[j].datum_len != curr_rsync_xal.rxas[j].datum_len -+ || strcmp(rsync_xal_l.rxals[i].rxas[j].name, curr_rsync_xal.rxas[j].name) -+ || memcmp(rsync_xal_l.rxals[i].rxas[j].datum, curr_rsync_xal.rxas[j].datum, curr_rsync_xal.rxas[j].datum_len)) ++ for (j = 0; j < xalp->count; j++) { ++ if (rxas1[j].name_len != rxas2[j].name_len ++ || rxas1[j].datum_len != rxas2[j].datum_len ++ || strcmp(rxas1[j].name, rxas2[j].name) ++ || memcmp(rxas1[j].datum, rxas2[j].datum, rxas2[j].datum_len)) + break; + } + /* no differences found. This is The One! */ -+ if (j == curr_rsync_xal.count) -+ break; ++ if (j == xalp->count) ++ return i; + } -+ if (i < rsync_xal_l.count) -+ return i; -+ return (ssize_t)-1; ++ ++ return -1; +} + -+/* Store curr_rsync_xal on the end of rsync_xal_l */ -+static void rsync_xal_store(void) ++/* Store *xalp on the end of rsync_xal_l */ ++static void rsync_xal_store(item_list *xalp) +{ -+ if (rsync_xal_l.count <= rsync_xal_l.alloc) { -+ size_t new_alloc; -+ void *new_xal; -+ -+ new_alloc = rsync_xal_l.count < RSYNC_XAL_LIST_INITIAL ? RSYNC_XAL_LIST_INITIAL : rsync_xal_l.count * 2; -+ new_xal = realloc_array(rsync_xal_l.rxals, rsync_xal, new_alloc); -+ if (!new_xal) -+ out_of_memory("rsync_xal_store"); -+ rsync_xal_l.alloc = new_alloc; -+ rsync_xal_l.rxals = new_xal; -+ } -+ rsync_xal_l.rxals[rsync_xal_l.count] = curr_rsync_xal; -+ rsync_xal_l.count++; -+ curr_rsync_xal.count = 0; -+ curr_rsync_xal.alloc = 0; ++ item_list *new_lst = EXPAND_ITEM_LIST(&rsync_xal_l, item_list, RSYNC_XAL_LIST_INITIAL); ++ /* 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; ++ xalp->count = 0; +} + -+/* send the make_xattr()-generated xattr list for this flist entry, -+ * or clean up after an flist entry that's not being sent (f == -1) */ -+ -+void send_xattr(UNUSED(const struct file_struct *file), int f) ++/* Send the make_xattr()-generated xattr list for this flist entry. */ ++void send_xattr(statx *sxp, int f) +{ -+ ssize_t index; -+ -+ if (f == -1) { -+ rsync_xal_free(&curr_rsync_xal); -+ return; -+ } -+ index = rsync_xal_find_matching(); -+ if (index != -1) { ++ int ndx = find_matching_xattr(sxp->xattr); ++ if (ndx != -1) { + write_byte(f, 'x'); -+ write_int(f, index); -+ rsync_xal_free(&curr_rsync_xal); ++ write_int(f, ndx); ++ rsync_xal_free(sxp->xattr); + } else { + rsync_xa *rxa; -+ size_t count; -+ -+ count = curr_rsync_xal.count; ++ int count = sxp->xattr->count; + write_byte(f, 'X'); + write_int(f, count); -+ for (rxa = curr_rsync_xal.rxas; count--; rxa++) { ++ for (rxa = sxp->xattr->items; count--; rxa++) { ++#ifdef HAVE_LINUX_XATTRS + write_int(f, rxa->name_len); + write_int(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_int(f, rxa->name_len - RPRE_LEN); ++ write_int(f, rxa->datum_len); ++ write_buf(f, rxa->name + RPRE_LEN, rxa->name_len - RPRE_LEN); ++ } else { ++ write_int(f, rxa->name_len + UPRE_LEN); ++ write_int(f, rxa->datum_len); ++ write_buf(f, USER_PREFIX, UPRE_LEN); ++ write_buf(f, rxa->name, rxa->name_len); ++ } ++#endif + write_buf(f, rxa->datum, rxa->datum_len); + } -+ rsync_xal_store(); ++ rsync_xal_store(sxp->xattr); /* adds item to rsync_xal_l */ + } ++ free_xattr(sxp); +} + -+ +/* ------------------------------------------------------------------------- */ -+/* receive and build the rsync_xattr_lists */ + ++/* receive and build the rsync_xattr_lists */ +void receive_xattr(struct file_struct *file, int f) +{ -+ char *fname; -+ int tag; -+ -+ fname = f_name(file, NULL); -+ tag = read_byte(f); -+ if (tag != 'X' && tag != 'x') { -+ rprintf(FERROR, -+ "%s: receive_xattr: unknown extended attribute type tag: %c\n", -+ fname, tag); -+ exit_cleanup(RERR_STREAMIO); -+ } -+ -+ if (fxil.alloc <= fxil.count) { -+ void *new_ptr; -+ size_t new_alloc; -+ -+ if (fxil.count < RSYNC_XAL_LIST_INITIAL) -+ new_alloc = fxil.alloc + RSYNC_XAL_LIST_INITIAL; -+ else -+ new_alloc = fxil.alloc * 2; -+ new_ptr = realloc_array(fxil.filexalidxs, file_xal_index, new_alloc); -+ if (!new_ptr) -+ out_of_memory("receive_xattr"); -+ if (verbose >= 3) { -+ rprintf(FINFO, "receive_xattr to %lu bytes, %s move\n", -+ (unsigned long)(new_alloc * sizeof (file_xal_index)), -+ fxil.filexalidxs == new_ptr ? "did not" : "did"); -+ } -+ -+ fxil.filexalidxs = new_ptr; -+ fxil.alloc = new_alloc; -+ } ++ static item_list temp_xattr = EMPTY_ITEM_LIST; ++ int ndx, tag = read_byte(f); + -+ fxil.filexalidxs[fxil.count].file = file; + if (tag == 'X') { -+ size_t count; -+ size_t i; -+ -+ fxil.filexalidxs[fxil.count].xalidx = rsync_xal_l.count; -+ -+ count = read_int(f); -+ curr_rsync_xal.count = count; -+ curr_rsync_xal.alloc = count; -+ curr_rsync_xal.rxas = new_array(rsync_xa, count); -+ if (!curr_rsync_xal.rxas) -+ out_of_memory("receive_xattr"); ++ int i, count = read_int(f); + for (i = 0; i < count; i++) { -+ size_t name_len; -+ size_t datum_len; -+ char *ptr; -+ -+ name_len = read_int(f); -+ datum_len = read_int(f); -+ ptr = new_array(char, name_len + datum_len); ++ char *ptr, *name; ++ 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; ++ 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); + if (!ptr) + out_of_memory("receive_xattr"); -+ read_buf(f, ptr, name_len); -+ read_buf(f, ptr + name_len, datum_len); -+ curr_rsync_xal.rxas[i].name_len = name_len; -+ curr_rsync_xal.rxas[i].datum_len = datum_len; -+ curr_rsync_xal.rxas[i].name = ptr; -+ curr_rsync_xal.rxas[i].datum = ptr + name_len; ++ name = ptr + datum_len + extra_len; ++ read_buf(f, name, name_len); ++ 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; ++ } ++#else ++ /* This OS only has a user namespace, so we either ++ * strip the user prefix, or we put a non-user ++ * namespace inside our rsync hierarchy. */ ++ if (HAS_PREFIX(name, USER_PREFIX)) { ++ name += UPRE_LEN; ++ name_len -= UPRE_LEN; ++ } else if (am_root) { ++ name -= RPRE_LEN; ++ name_len += RPRE_LEN; ++ memcpy(name, RSYNC_PREFIX, RPRE_LEN); ++ } else { ++ free(ptr); ++ continue; ++ } ++#endif ++ rxa = EXPAND_ITEM_LIST(&temp_xattr, rsync_xa, count); ++ rxa->name = name; ++ rxa->datum = ptr; ++ rxa->name_len = name_len; ++ rxa->datum_len = datum_len; + } -+ rsync_xal_store(); -+ } else { -+ size_t index; -+ -+ index = read_int(f); -+ if (index >= rsync_xal_l.count) { -+ rprintf(FERROR, "%s: receive_xattr: xa index %lu out of range\n", -+ fname, (unsigned long)index); ++ ndx = rsync_xal_l.count; /* pre-incremented count */ ++ rsync_xal_store(&temp_xattr); /* adds item to rsync_xal_l */ ++ } else if (tag == 'x') { ++ ndx = read_int(f); ++ if (ndx < 0 || (size_t)ndx >= rsync_xal_l.count) { ++ rprintf(FERROR, "receive_xattr: xa index %d out of" ++ " range for %s\n", ndx, f_name(file, NULL)); + exit_cleanup(RERR_STREAMIO); + } -+ fxil.filexalidxs[fxil.count].xalidx = index; -+ } -+ fxil.count++; -+} -+ -+static int rsync_xal_set(const char *fname, rsync_xal *x) -+{ -+ size_t i; -+ int ret = 0; -+ -+ for (i = 0; i < x->count; i++) { -+ int status = sys_lsetxattr(fname, x->rxas[i].name, x->rxas[i].datum, x->rxas[i].datum_len, 0); -+ if (status < 0) { -+ rprintf(FERROR, "%s: rsync_xal_set: lsetxattr %s failed: %s\n", -+ fname, x->rxas[i].name, strerror(errno)); -+ ret = -1; -+ } ++ } else { ++ rprintf(FERROR, "receive_xattr: unknown extended attribute" ++ " type tag (%c) for %s\n", tag, f_name(file, NULL)); ++ exit_cleanup(RERR_STREAMIO); + } -+ return ret; -+} -+ -+/* for duplicating xattrs on backups when using backup_dir */ -+ -+int dup_xattr(const char *orig, const char *bak) -+{ -+ int ret; -+ -+ if (rsync_xal_get(orig, &backup_xal) < 0) -+ ret = rsync_xal_set(bak, &backup_xal); -+ else -+ ret = 0; -+ rsync_xal_free(&backup_xal); + -+ return ret; -+} -+ -+void push_keep_backup_xattr(const struct file_struct *file, const char *orig, const char *dest) -+{ -+ backup_orig_file = file; -+ backup_orig_fname = orig; -+ backup_dest_fname = dest; -+ rsync_xal_get(orig, &backup_xal); ++ F_XATTR(file) = ndx; +} + -+static int set_keep_backup_xal(void) ++/* Turn the xattr data in statx into cached xattr data, setting the index ++ * values in the file struct. */ ++void cache_xattr(struct file_struct *file, statx *sxp) +{ -+ return rsync_xal_set(backup_dest_fname, &backup_xal); -+} -+ -+void cleanup_keep_backup_xattr(void) -+{ -+ backup_orig_file = NULL; -+ backup_orig_fname = null_string; -+ backup_dest_fname = null_string; -+ rsync_xal_free(&backup_xal); -+} ++ int ndx; + -+static int file_xal_index_compare(const void *x1, const void *x2) -+{ -+ const file_xal_index *xa1; -+ const file_xal_index *xa2; ++ if (!sxp->xattr) ++ return; + -+ xa1 = x1; -+ xa2 = x2; -+ return xa1->file == xa2->file ? 0 : xa1->file < xa2->file ? -1 : 1; -+} ++ ndx = find_matching_xattr(sxp->xattr); ++ if (ndx == -1) ++ rsync_xal_store(sxp->xattr); /* adds item to rsync_xal_l */ ++ free_xattr(sxp); + -+void sort_file_xattr_index_lists(void) -+{ -+ qsort(fxil.filexalidxs, fxil.count, sizeof (file_xal_index), file_xal_index_compare); ++ F_XATTR(file) = ndx; +} + -+static int find_file_xal_index(const struct file_struct *file) ++static int rsync_xal_set(const char *fname, item_list *xalp) +{ -+ int low = 0, high = fxil.count; -+ const struct file_struct *file_mid; ++ rsync_xa *rxas = xalp->items; ++ size_t i; ++ int ret = 0; + -+ if (!high--) { -+ rprintf(FERROR, "find_file_xal_index: no entries\n"); -+ exit_cleanup(RERR_STREAMIO); -+ return -1; -+ } -+ do { -+ int mid = (high + low) / 2; -+ file_mid = fxil.filexalidxs[mid].file; -+ if (file_mid == file) -+ return fxil.filexalidxs[mid].xalidx; -+ if (file_mid > file) -+ high = mid - 1; -+ else -+ low = mid + 1; -+ } while (low < high); -+ if (low == high) { -+ file_mid = fxil.filexalidxs[low].file; -+ if (file_mid == file) -+ return fxil.filexalidxs[low].xalidx; ++ for (i = 0; i < xalp->count; i++) { ++ int status = sys_lsetxattr(fname, rxas[i].name, rxas[i].datum, rxas[i].datum_len); ++ if (status < 0) { ++ rsyserr(FERROR, errno, ++ "rsync_xal_set: lsetxattr(\"%s\",\"%s\") failed", ++ fname, rxas[i].name); ++ ret = -1; ++ } + } -+ rprintf(FERROR, -+ "find_file_xal_index: can't find entry for file in list\n"); -+ exit_cleanup(RERR_STREAMIO); -+ return -1; ++ return ret; +} + -+/* set extended attributes on rsync-ed or keep_backup-ed file */ -+ -+int set_xattr(const char *fname, const struct file_struct *file) ++/* Set extended attributes on indicated filename. */ ++int set_xattr(const char *fname, const struct file_struct *file, UNUSED(statx *sxp)) +{ -+ int xalidx; -+ rsync_xal *x; ++ int ndx; ++ item_list *lst = rsync_xal_l.items; + + if (dry_run) + return 1; /* FIXME: --dry-run needs to compute this value */ + -+ if (file == backup_orig_file) { -+ if (!strcmp(fname, backup_dest_fname)) -+ return set_keep_backup_xal(); ++ if (read_only || list_only) { ++ errno = EROFS; ++ return -1; + } -+ xalidx = find_file_xal_index(file); -+ x = &(rsync_xal_l.rxals[xalidx]); + -+ return rsync_xal_set(fname, x); ++ ndx = F_XATTR(file); ++ return rsync_xal_set(fname, lst + ndx); /* TODO: This needs to return 1 if no xattrs changed! */ +} + +#endif /* SUPPORT_XATTRS */