]> git.ipfire.org Git - thirdparty/tar.git/commitdiff
Fewer uses of size_t in extract.c
authorPaul Eggert <eggert@cs.ucla.edu>
Fri, 1 Nov 2024 02:53:25 +0000 (19:53 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Sat, 2 Nov 2024 06:47:23 +0000 (23:47 -0700)
* src/extract.c (struct delayed_set_stat, struct delayed_link)
(delay_set_stat, apply_nonancestor_delayed_set_stat)
(extract_file): Prefer idx_t to size_t.
(struct delayed_set_stat): Remove unused member xattr_map_size.

src/extract.c

index 1121e5961050edd91d74278d667f983370cbbd87..89d1999fffa4ca8608199dd229dfb14ec84e40fb 100644 (file)
@@ -120,13 +120,12 @@ struct delayed_set_stat
     /* extended attributes*/
     char *cntx_name;
     char *acls_a_ptr;
-    size_t acls_a_len;
+    idx_t acls_a_len;
     char *acls_d_ptr;
-    size_t acls_d_len;
-    size_t xattr_map_size;
+    idx_t acls_d_len;
     struct xattr_map xattr_map;
     /* Length and contents of name.  */
-    size_t file_name_len;
+    idx_t file_name_len;
     char *file_name;
   };
 
@@ -176,9 +175,9 @@ struct delayed_link
 
     /* ACLs */
     char *acls_a_ptr;
-    size_t acls_a_len;
+    idx_t acls_a_len;
     char *acls_d_ptr;
-    size_t acls_d_len;
+    idx_t acls_d_len;
 
     struct xattr_map xattr_map;
 
@@ -527,7 +526,7 @@ delay_set_stat (char const *file_name, struct tar_stat_info const *st,
                mode_t current_mode, mode_t current_mode_mask,
                mode_t mode, int atflag)
 {
-  size_t file_name_len = strlen (file_name);
+  idx_t file_name_len = strlen (file_name);
   struct delayed_set_stat *data;
 
   if (! (delayed_set_stat_table
@@ -959,7 +958,7 @@ set_xattr (char const *file_name, struct tar_stat_info const *st,
 static void
 apply_nonancestor_delayed_set_stat (char const *file_name, bool after_links)
 {
-  size_t file_name_len = strlen (file_name);
+  idx_t file_name_len = strlen (file_name);
   bool check_for_renamed_directories = 0;
 
   while (delayed_set_stat_head)
@@ -1277,7 +1276,6 @@ extract_file (char *file_name, int typeflag)
   off_t size;
   union block *data_block;
   int status;
-  size_t written;
   bool interdir_made = false;
   mode_t mode = (current_stat_info.stat.st_mode & MODE_RWX
                 & ~ (0 < same_owner_option ? S_IRWXG | S_IRWXO : 0));
@@ -1342,7 +1340,7 @@ extract_file (char *file_name, int typeflag)
            break;              /* FIXME: What happens, then?  */
          }
 
-       written = available_space_after (data_block);
+       idx_t written = available_space_after (data_block);
 
        if (written > size)
          written = size;