DO NOT REPLY [Bug 5565] New: xattrs not set on locked files that
already exist on target
samba-bugs at samba.org
samba-bugs at samba.org
Wed Jun 25 05:27:08 GMT 2008
https://bugzilla.samba.org/show_bug.cgi?id=5565
Summary: xattrs not set on locked files that already exist on
target
Product: rsync
Version: 3.0.3
Platform: Other
OS/Version: Mac OS X
Status: NEW
Severity: normal
Priority: P3
Component: core
AssignedTo: wayned at samba.org
ReportedBy: mike at bombich.com
QAContact: rsync-qa at samba.org
It appears that rsync isn't unlocking files before setting the file attributes
when those files already exist. This generates error messages on subsequent
backups such as:
rsync: rsync_xal_set: lsetxattr("locked_file","test_xattr") failed: Operation
not permitted (1)
rsync: rsync_xal_clear: lremovexattr("locked_file","test_xattr.temp") failed:
Operation not permitted (1)
I assume this would also affect ACLs. Presumably, this is what --force-change
is supposed to do, so I extended that functionality to apply to this test case.
Here's the test case (tested on 3.0.3pre3+fileflags+crtime):
######
#!/bin/sh
xattr=/usr/local/bin/xattr
rsync="/Users/bombich/Desktop/rsync-HEAD-20080412-0558GMT/rsync"
src=`mktemp -d /tmp/src.XXXXXX`
tgt=`mktemp -d /tmp/tgt.XXXXXX`
touch $src/locked_file
$xattr -s test_xattr "Test attribute" $src/locked_file
$xattr -s test_xattr.temp "Temp attribute" $src/locked_file
chflags uchg $src/locked_file
## First backup
printf "### Initial backup ###\n\n"
"$rsync" -vaX --fileflags --force-change $src/ $tgt/
## Change the source file
chflags nouchg $src/locked_file
$xattr -s test_xattr "Modified attribute" $src/locked_file
$xattr -d test_xattr.temp $src/locked_file
chflags uchg $src/locked_file
## Second backup
printf "\n### Second backup ###\n\n"
"$rsync" -vaX --fileflags --force-change $src/ $tgt/
# Cleanup
chflags -R nouchg $src
chflags -R nouchg $tgt
rm -rf $src $tgt
######
Example output:
[bombich:~] ~/Desktop/locked_xattrs.sh
### Initial backup ###
sending incremental file list
locked_file
sent 173 bytes received 32 bytes 410.00 bytes/sec
total size is 0 speedup is 0.00
### Second backup ###
sending incremental file list
rsync: rsync_xal_set: lsetxattr("locked_file","test_xattr") failed: Operation
not permitted (1)
rsync: rsync_xal_clear: lremovexattr("locked_file","test_xattr.temp") failed:
Operation not permitted (1)
sent 104 bytes received 16 bytes 240.00 bytes/sec
total size is 0 speedup is 0.00
rsync error: some files could not be transferred (code 23) at main.c(1031)
[sender=3.0.3dev]
PATCH:
diff -Naur rsync-3.0.3pre3_base/rsync.c rsync-3.0.3pre3_patched/rsync.c
--- rsync-3.0.3pre3_base/rsync.c 2008-06-23 22:31:58.000000000 -0700
+++ rsync-3.0.3pre3_patched/rsync.c 2008-06-23 22:59:03.000000000 -0700
@@ -31,6 +31,7 @@
extern int dry_run;
extern int preserve_acls;
extern int preserve_xattrs;
+extern int force_change;
extern int preserve_perms;
extern int preserve_fileflags;
extern int preserve_executability;
@@ -445,6 +446,11 @@
if (daemon_chmod_modes && !S_ISLNK(new_mode))
new_mode = tweak_mode(new_mode, daemon_chmod_modes);
+#ifdef SUPPORT_FORCE_CHANGE
+ if (force_change)
+ make_mutable(fname, sxp->st.st_mode, sxp->st.st_flags,
force_change);
+#endif
+
#ifdef SUPPORT_ACLS
if (preserve_acls && !S_ISLNK(file->mode) && !ACL_READY(*sxp))
get_acl(fname, sxp);
## Note: I'm not using undo_make_mutable because set_fileflags is called at the
end of this method if we care about fileflags.
--
Configure bugmail: https://bugzilla.samba.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the rsync
mailing list