Put file descriptor arg at the start of the arg list for consistency.
[rsync/rsync.git] / acls.c
diff --git a/acls.c b/acls.c
index a233ee1..24dc177 100644 (file)
--- a/acls.c
+++ b/acls.c
@@ -3,7 +3,7 @@
  *
  * Copyright (C) 1996 Andrew Tridgell
  * Copyright (C) 1996 Paul Mackerras
- * Copyright (C) 2006 Wayne Davison
+ * Copyright (C) 2006-2009 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
@@ -88,6 +88,9 @@ static const rsync_acl empty_rsync_acl = {
 static item_list access_acl_list = EMPTY_ITEM_LIST;
 static item_list default_acl_list = EMPTY_ITEM_LIST;
 
+static size_t prior_access_count = (size_t)-1;
+static size_t prior_default_count = (size_t)-1;
+
 /* === Calculations on ACL types === */
 
 static const char *str_acl_type(SMB_ACL_TYPE_T type)
@@ -112,10 +115,11 @@ static int calc_sacl_entries(const rsync_acl *racl)
        /* A System ACL always gets user/group/other permission entries. */
        return racl->names.count
 #ifdef ACLS_NEED_MASK
-            + 4;
+            + 1
 #else
-            + (racl->mask_obj != NO_ENTRY) + 3;
+            + (racl->mask_obj != NO_ENTRY)
 #endif
+            + 3;
 }
 
 /* Extracts and returns the permission bits from the ACL.  This cannot be
@@ -137,7 +141,8 @@ static void rsync_acl_strip_perms(rsync_acl *racl)
        else {
                if (racl->group_obj == racl->mask_obj)
                        racl->group_obj = NO_ENTRY;
-               racl->mask_obj = NO_ENTRY;
+               if (racl->names.count != 0)
+                       racl->mask_obj = NO_ENTRY;
        }
        racl->other_obj = NO_ENTRY;
 }
@@ -557,7 +562,7 @@ int get_acl(const char *fname, stat_x *sxp)
 /* === Send functions === */
 
 /* Send the ida list over the file descriptor. */
-static void send_ida_entries(const ida_entries *idal, int f)
+static void send_ida_entries(int f, const ida_entries *idal)
 {
        id_access *ida;
        size_t count = idal->count;
@@ -583,8 +588,8 @@ static void send_ida_entries(const ida_entries *idal, int f)
        }
 }
 
-static void send_rsync_acl(rsync_acl *racl, SMB_ACL_TYPE_T type,
-                          item_list *racl_list, int f)
+static void send_rsync_acl(int f, rsync_acl *racl, SMB_ACL_TYPE_T type,
+                          item_list *racl_list)
 {
        int ndx = find_matching_rsync_acl(racl, type, racl_list);
 
@@ -617,7 +622,7 @@ static void send_rsync_acl(rsync_acl *racl, SMB_ACL_TYPE_T type,
                if (flags & XMIT_OTHER_OBJ)
                        write_varint(f, racl->other_obj);
                if (flags & XMIT_NAME_LIST)
-                       send_ida_entries(&racl->names, f);
+                       send_ida_entries(f, &racl->names);
 
                /* Give the allocated data to the new list object. */
                *new_racl = *racl;
@@ -627,7 +632,7 @@ static void send_rsync_acl(rsync_acl *racl, SMB_ACL_TYPE_T type,
 
 /* Send the ACL from the stat_x structure down the indicated file descriptor.
  * This also frees the ACL data. */
-void send_acl(stat_x *sxp, int f)
+void send_acl(int f, stat_x *sxp)
 {
        if (!sxp->acc_acl) {
                sxp->acc_acl = create_racl();
@@ -636,19 +641,19 @@ void send_acl(stat_x *sxp, int f)
        /* Avoid sending values that can be inferred from other data. */
        rsync_acl_strip_perms(sxp->acc_acl);
 
-       send_rsync_acl(sxp->acc_acl, SMB_ACL_TYPE_ACCESS, &access_acl_list, f);
+       send_rsync_acl(f, sxp->acc_acl, SMB_ACL_TYPE_ACCESS, &access_acl_list);
 
        if (S_ISDIR(sxp->st.st_mode)) {
                if (!sxp->def_acl)
                        sxp->def_acl = create_racl();
 
-               send_rsync_acl(sxp->def_acl, SMB_ACL_TYPE_DEFAULT, &default_acl_list, f);
+               send_rsync_acl(f, sxp->def_acl, SMB_ACL_TYPE_DEFAULT, &default_acl_list);
        }
 }
 
 /* === Receive functions === */
 
-static uint32 recv_acl_access(uchar *name_follows_ptr, int f)
+static uint32 recv_acl_access(int f, uchar *name_follows_ptr)
 {
        uint32 access = read_varint(f);
 
@@ -673,7 +678,7 @@ static uint32 recv_acl_access(uchar *name_follows_ptr, int f)
        return access;
 }
 
-static uchar recv_ida_entries(ida_entries *ent, int f)
+static uchar recv_ida_entries(int f, ida_entries *ent)
 {
        uchar computed_mask_bits = 0;
        int i, count = read_varint(f);
@@ -689,7 +694,7 @@ static uchar recv_ida_entries(ida_entries *ent, int f)
        for (i = 0; i < count; i++) {
                uchar has_name;
                id_t id = read_varint(f);
-               uint32 access = recv_acl_access(&has_name, f);
+               uint32 access = recv_acl_access(f, &has_name);
 
                if (has_name) {
                        if (access & NAME_IS_USER)
@@ -712,7 +717,7 @@ static uchar recv_ida_entries(ida_entries *ent, int f)
        return computed_mask_bits & ~NO_ENTRY;
 }
 
-static int recv_rsync_acl(item_list *racl_list, SMB_ACL_TYPE_T type, int f)
+static int recv_rsync_acl(int f, item_list *racl_list, SMB_ACL_TYPE_T type)
 {
        uchar computed_mask_bits = 0;
        acl_duo *duo_item;
@@ -735,28 +740,21 @@ static int recv_rsync_acl(item_list *racl_list, SMB_ACL_TYPE_T type, int f)
        flags = read_byte(f);
 
        if (flags & XMIT_USER_OBJ)
-               duo_item->racl.user_obj = recv_acl_access(NULL, f);
+               duo_item->racl.user_obj = recv_acl_access(f, NULL);
        if (flags & XMIT_GROUP_OBJ)
-               duo_item->racl.group_obj = recv_acl_access(NULL, f);
+               duo_item->racl.group_obj = recv_acl_access(f, NULL);
        if (flags & XMIT_MASK_OBJ)
-               duo_item->racl.mask_obj = recv_acl_access(NULL, f);
+               duo_item->racl.mask_obj = recv_acl_access(f, NULL);
        if (flags & XMIT_OTHER_OBJ)
-               duo_item->racl.other_obj = recv_acl_access(NULL, f);
+               duo_item->racl.other_obj = recv_acl_access(f, NULL);
        if (flags & XMIT_NAME_LIST)
-               computed_mask_bits |= recv_ida_entries(&duo_item->racl.names, f);
+               computed_mask_bits |= recv_ida_entries(f, &duo_item->racl.names);
 
 #ifdef HAVE_OSX_ACLS
        /* If we received a superfluous mask, throw it away. */
        duo_item->racl.mask_obj = NO_ENTRY;
 #else
-       if (!duo_item->racl.names.count) {
-               /* If we received a superfluous mask, throw it away. */
-               if (duo_item->racl.mask_obj != NO_ENTRY) {
-                       /* Mask off the group perms with it first. */
-                       duo_item->racl.group_obj &= duo_item->racl.mask_obj | NO_ENTRY;
-                       duo_item->racl.mask_obj = NO_ENTRY;
-               }
-       } else if (duo_item->racl.mask_obj == NO_ENTRY) /* Must be non-empty with lists. */
+       if (duo_item->racl.names.count && duo_item->racl.mask_obj == NO_ENTRY) /* Must be non-empty with lists. */
                duo_item->racl.mask_obj = (computed_mask_bits | duo_item->racl.group_obj) & ~NO_ENTRY;
 #endif
 
@@ -766,12 +764,12 @@ static int recv_rsync_acl(item_list *racl_list, SMB_ACL_TYPE_T type, int f)
 }
 
 /* Receive the ACL info the sender has included for this file-list entry. */
-void receive_acl(struct file_struct *file, int f)
+void receive_acl(int f, struct file_struct *file)
 {
-       F_ACL(file) = recv_rsync_acl(&access_acl_list, SMB_ACL_TYPE_ACCESS, f);
+       F_ACL(file) = recv_rsync_acl(f, &access_acl_list, SMB_ACL_TYPE_ACCESS);
 
        if (S_ISDIR(file->mode))
-               F_DIR_DEFACL(file) = recv_rsync_acl(&default_acl_list, SMB_ACL_TYPE_DEFAULT, f);
+               F_DIR_DEFACL(file) = recv_rsync_acl(f, &default_acl_list, SMB_ACL_TYPE_DEFAULT);
 }
 
 static int cache_rsync_acl(rsync_acl *racl, SMB_ACL_TYPE_T type, item_list *racl_list)
@@ -794,17 +792,50 @@ static int cache_rsync_acl(rsync_acl *racl, SMB_ACL_TYPE_T type, item_list *racl
 
 /* Turn the ACL data in stat_x into cached ACL data, setting the index
  * values in the file struct. */
-void cache_acl(struct file_struct *file, stat_x *sxp)
+void cache_tmp_acl(struct file_struct *file, stat_x *sxp)
 {
+       if (prior_access_count == (size_t)-1)
+               prior_access_count = access_acl_list.count;
+
        F_ACL(file) = cache_rsync_acl(sxp->acc_acl,
                                      SMB_ACL_TYPE_ACCESS, &access_acl_list);
 
        if (S_ISDIR(sxp->st.st_mode)) {
+               if (prior_default_count == (size_t)-1)
+                       prior_default_count = default_acl_list.count;
                F_DIR_DEFACL(file) = cache_rsync_acl(sxp->def_acl,
                                      SMB_ACL_TYPE_DEFAULT, &default_acl_list);
        }
 }
 
+static void uncache_duo_acls(item_list *duo_list, size_t start)
+{
+       acl_duo *duo_item = duo_list->items;
+       acl_duo *duo_start = duo_item + start;
+
+       duo_item += duo_list->count;
+       duo_list->count = start;
+
+       while (duo_item-- > duo_start) {
+               rsync_acl_free(&duo_item->racl);
+               if (duo_item->sacl)
+                       sys_acl_free_acl(duo_item->sacl);
+       }
+}
+
+void uncache_tmp_acls(void)
+{
+       if (prior_access_count != (size_t)-1) {
+               uncache_duo_acls(&access_acl_list, prior_access_count);
+               prior_access_count = (size_t)-1;
+       }
+
+       if (prior_default_count != (size_t)-1) {
+               uncache_duo_acls(&default_acl_list, prior_default_count);
+               prior_default_count = (size_t)-1;
+       }
+}
+
 #ifndef HAVE_OSX_ACLS
 static mode_t change_sacl_perms(SMB_ACL_T sacl, rsync_acl *racl, mode_t old_mode, mode_t mode)
 {
@@ -868,7 +899,7 @@ static mode_t change_sacl_perms(SMB_ACL_T sacl, rsync_acl *racl, mode_t old_mode
                        rsyserr(FERROR_XFER, errno, "change_sacl_perms: %s()",
                                errfun);
                }
-               return (mode_t)~0;
+               return (mode_t)-1;
        }
 
 #ifdef SMB_ACL_LOSES_SPECIAL_MODE_BITS
@@ -937,7 +968,7 @@ static int set_rsync_acl(const char *fname, acl_duo *duo_item,
                if (type == SMB_ACL_TYPE_ACCESS) {
                        cur_mode = change_sacl_perms(duo_item->sacl, &duo_item->racl,
                                                     cur_mode, mode);
-                       if (cur_mode == (mode_t)~0)
+                       if (cur_mode == (mode_t)-1)
                                return 0;
                }
 #endif
@@ -1081,7 +1112,7 @@ int default_perms_for_dir(const char *dir)
        /* Apply the permission-bit entries of the default ACL, if any. */
        if (racl.user_obj != NO_ENTRY) {
                perms = rsync_acl_get_perms(&racl);
-               if (verbose > 2)
+               if (DEBUG_GTE(ACL, 1))
                        rprintf(FINFO, "got ACL-based default perms %o for directory %s\n", perms, dir);
        }