X-Git-Url: https://mattmccutchen.net/rsync/rsync-patches.git/blobdiff_plain/d38328008c1e3e2445c89d9a9a6af2c1333377a3..refs/heads/master:/adaptec_acl_mods.diff diff --git a/adaptec_acl_mods.diff b/adaptec_acl_mods.diff index cfc0c28..eaa1ed2 100644 --- a/adaptec_acl_mods.diff +++ b/adaptec_acl_mods.diff @@ -1,130 +1,86 @@ -Depends-On-Patch: acls.diff - -After applying the above patch and this one, run these commands for a -successful build: +To use this patch, run these commands for a successful build: + patch -p1 Attached is a small patch which is meant to be applied to a copy of -> rsync which has already been patched with acl support (the acls.diff -> file in the patches folder). It allows the preservation of the delete, +> Attached is a small patch which allows the preservation of the delete, > chmod, and chown bits which Adaptec has added to XFS on their SnapOS NAS > units. This is nice for backing up files between different NAS units > and preserving all of the Samba ACLs. -> +> > I'm not sure how useful this patch will be because I'm not sure if any > other NAS vendors have standardized on their extensions to POSIX ACLs to > support Samba in the same manner that Adaptec has. FWIW, though, this > will allow you to preserve acls when copying between different Adaptec > based NAS units running SnapOS. -I (Wayne) tweaked the patch for style and to avoid using SMB_* constants -with literal values were needed. +I (Wayne) tweaked the patch to work with the latest source. + +Todo: -I've also updated it to apply to the updated version of the acls.diff, -though I don't know if there might be some bits lost in the current -algorithm when using the file's mode bits to reconstruct a stripped ACL -entry. +Fix a bug that could lose some bits when stripping some (supposedly) +superfluous ACL info. ---- old/acls.c -+++ new/acls.c -@@ -84,10 +84,10 @@ static int calc_sacl_entries(const rsync +based-on: a01e3b490eb36ccf9e704840e1b6683dab867550 +diff --git a/lib/sysacls.c b/lib/sysacls.c +--- a/lib/sysacls.c ++++ b/lib/sysacls.c +@@ -31,6 +31,18 @@ + #endif + #define DEBUG(x,y) - static int rsync_acl_get_perms(const rsync_acl *racl) ++/* These are custom ACL bits used by Adaptec's modifications ++ * to XFS on their SnapOS units. */ ++#ifndef ACL_DELETE ++#define ACL_DELETE 8 ++#endif ++#ifndef ACL_CHMOD ++#define ACL_CHMOD 16 ++#endif ++#ifndef ACL_CHOWN ++#define ACL_CHOWN 32 ++#endif ++ + void SAFE_FREE(void *mem) { -- /* Note that (ACL_NO_ENTRY & 7) is 0. */ -- return ((racl->user_obj & 7) << 6) -- + (((racl->mask != ACL_NO_ENTRY ? racl->mask : racl->group_obj) & 7) << 3) -- + (racl->other & 7); -+ /* Note that (ACL_NO_ENTRY & 077) is 0. */ -+ return ((racl->user_obj & 077) << 6) -+ + (((racl->mask != ACL_NO_ENTRY ? racl->mask : racl->group_obj) & 077) << 3) -+ + (racl->other & 077); - } - - static void rsync_acl_strip_perms(rsync_acl *racl) -@@ -178,6 +178,9 @@ static BOOL unpack_smb_acl(rsync_acl *ra - } - access = (sys_acl_get_perm(permset, SMB_ACL_READ) ? 4 : 0) - | (sys_acl_get_perm(permset, SMB_ACL_WRITE) ? 2 : 0) -+ | (sys_acl_get_perm(permset, SMB_ACL_DELETE) ? 8 : 0) -+ | (sys_acl_get_perm(permset, SMB_ACL_CHMOD) ? 16 : 0) -+ | (sys_acl_get_perm(permset, SMB_ACL_CHOWN) ? 32 : 0) - | (sys_acl_get_perm(permset, SMB_ACL_EXECUTE) ? 1 : 0); - /* continue == done with entry; break == store in given idal */ - switch (tag_type) { -@@ -588,6 +591,12 @@ static int store_access_in_entry(uchar a - - COE( sys_acl_get_permset,(entry, &permset) ); - COE( sys_acl_clear_perms,(permset) ); -+ if (access & 32) -+ COE( sys_acl_add_perm(permset, SMB_ACL_CHOWN) ); -+ if (access & 16) -+ COE( sys_acl_add_perm(permset, SMB_ACL_CHMOD) ); -+ if (access & 8) -+ COE( sys_acl_add_perm(permset, SMB_ACL_DELETE) ); - if (access & 4) - COE( sys_acl_add_perm,(permset, SMB_ACL_READ) ); - if (access & 2) -@@ -620,7 +629,7 @@ static BOOL pack_smb_acl(SMB_ACL_T *smb_ + if (mem) +@@ -100,6 +112,9 @@ int sys_acl_get_info(SMB_ACL_ENTRY_T entry, SMB_ACL_TAG_T *tag_type_p, uint32 *b + return -1; - COE( sys_acl_create_entry,(smb_acl, &entry) ); - COE( sys_acl_set_tag_type,(entry, SMB_ACL_USER_OBJ) ); -- COE2( store_access_in_entry,(racl->user_obj & 7, entry) ); -+ COE2( store_access_in_entry,(racl->user_obj & 077, entry) ); + *bits_p = (acl_get_perm(permset, ACL_READ) ? 4 : 0) ++ | (acl_get_perm(permset, ACL_CHOWN) ? 32 : 0) ++ | (acl_get_perm(permset, ACL_CHMOD) ? 16 : 0) ++ | (acl_get_perm(permset, ACL_DELETE) ? 8 : 0) + | (acl_get_perm(permset, ACL_WRITE) ? 2 : 0) + | (acl_get_perm(permset, ACL_EXECUTE) ? 1 : 0); - for (ida = racl->users.idas, count = racl->users.count; - count--; ida++) { -@@ -632,7 +641,7 @@ static BOOL pack_smb_acl(SMB_ACL_T *smb_ +@@ -144,6 +159,12 @@ int sys_acl_set_access_bits(SMB_ACL_ENTRY_T entry, uint32 bits) + if ((rc = acl_get_permset(entry, &permset)) != 0) + return rc; + acl_clear_perms(permset); ++ if (bits & 32) ++ acl_add_perm(permset, ACL_CHOWN); ++ if (bits & 16) ++ acl_add_perm(permset, ACL_CHMOD); ++ if (bits & 8) ++ acl_add_perm(permset, ACL_DELETE); + if (bits & 4) + acl_add_perm(permset, ACL_READ); + if (bits & 2) +diff --git a/lib/sysacls.h b/lib/sysacls.h +--- a/lib/sysacls.h ++++ b/lib/sysacls.h +@@ -59,8 +59,8 @@ + #define SMB_ACL_TYPE_ACCESS ACL_TYPE_ACCESS + #define SMB_ACL_TYPE_DEFAULT ACL_TYPE_DEFAULT - COE( sys_acl_create_entry,(smb_acl, &entry) ); - COE( sys_acl_set_tag_type,(entry, SMB_ACL_GROUP_OBJ) ); -- COE2( store_access_in_entry,(racl->group_obj & 7, entry) ); -+ COE2( store_access_in_entry,(racl->group_obj & 077, entry) ); +-#define SMB_ACL_VALID_NAME_BITS (4 | 2 | 1) +-#define SMB_ACL_VALID_OBJ_BITS (4 | 2 | 1) ++#define SMB_ACL_VALID_NAME_BITS (32 | 16 | 8 | 4 | 2 | 1) ++#define SMB_ACL_VALID_OBJ_BITS (32 | 16 | 8 | 4 | 2 | 1) - for (ida = racl->groups.idas, count = racl->groups.count; - count--; ida++) { -@@ -653,7 +662,7 @@ static BOOL pack_smb_acl(SMB_ACL_T *smb_ - - COE( sys_acl_create_entry,(smb_acl, &entry) ); - COE( sys_acl_set_tag_type,(entry, SMB_ACL_OTHER) ); -- COE2( store_access_in_entry,(racl->other & 7, entry) ); -+ COE2( store_access_in_entry,(racl->other & 077, entry) ); - - #ifdef DEBUG - if (sys_acl_valid(*smb_acl) < 0) -@@ -762,7 +771,7 @@ static void receive_rsync_acl(rsync_acl - while (count--) { - char tag = read_byte(f); - uchar access = read_byte(f); -- if (access & ~ (4 | 2 | 1)) { -+ if (access & ~(32 | 16 | 8 | 4 | 2 | 1)) { - rprintf(FERROR, "receive_rsync_acl: bogus permset %o\n", - access); - exit_cleanup(RERR_STREAMIO); -@@ -835,7 +844,7 @@ static void receive_rsync_acl(rsync_acl - } else - #endif - if (racl->mask == ACL_NO_ENTRY) /* always made non-empty when needed */ -- racl->mask = computed_mask_bits | (racl->group_obj & 7); -+ racl->mask = computed_mask_bits | (racl->group_obj & 077); - } - - /* receive and build the rsync_acl_lists */ ---- old/smb_acls.h -+++ new/smb_acls.h -@@ -33,6 +33,11 @@ - #define SMB_ACL_READ ACL_READ - #define SMB_ACL_WRITE ACL_WRITE - #define SMB_ACL_EXECUTE ACL_EXECUTE -+/* These are custom ACL bits used by Adaptec's modifications -+ * to XFS on their SnapOS units. */ -+#define SMB_ACL_DELETE 0x08 -+#define SMB_ACL_CHMOD 0x10 -+#define SMB_ACL_CHOWN 0x20 + #define SMB_ACL_NEED_SORT - /* Types of ACLs. */ - #define SMB_ACL_USER ACL_USER