Move get_unaligned_le32() macros to platform.h

[oweals/busybox.git] / archival / libarchive / data_extract_all.c

diff --git a/archival/libarchive/data_extract_all.c b/archival/libarchive/data_extract_all.c
index bd51d2ad3dd93499794c31e8c74802734a907203..1830ffb8d0f786be7abe4ac4636fde932c87d6a5 100644 (file)
--- a/archival/libarchive/data_extract_all.c
+++ b/archival/libarchive/data_extract_all.c
@@ -8,16 +8,14 @@
 
 void FAST_FUNC data_extract_all(archive_handle_t *archive_handle)
 {
-
        file_header_t *file_header = archive_handle->file_header;
        int dst_fd;
        int res;
+       char *hard_link;
 #if ENABLE_FEATURE_TAR_LONG_OPTIONS
        char *dst_name;
-       char *dst_link;
 #else
 # define dst_name (file_header->name)
-# define dst_link (file_header->link_target)
 #endif
 
 #if ENABLE_FEATURE_TAR_SELINUX
@@ -31,37 +29,39 @@ void FAST_FUNC data_extract_all(archive_handle_t *archive_handle)
        }
 #endif
 
+       /* Hard links are encoded as regular files of size 0
+        * with a nonempty link field */
+       hard_link = NULL;
+       if (S_ISREG(file_header->mode) && file_header->size == 0)
+               hard_link = file_header->link_target;
+
 #if ENABLE_FEATURE_TAR_LONG_OPTIONS
        dst_name = file_header->name;
-       dst_link = file_header->link_target;
        if (archive_handle->tar__strip_components) {
                unsigned n = archive_handle->tar__strip_components;
                do {
                        dst_name = strchr(dst_name, '/');
                        if (!dst_name || dst_name[1] == '\0') {
                                data_skip(archive_handle);
-                               return;
+                               goto ret;
                        }
                        dst_name++;
                        /*
                         * Link target is shortened only for hardlinks:
                         * softlinks restored unchanged.
                         */
-                       if (S_ISREG(file_header->mode)
-                        && file_header->size == 0
-                        && dst_link
-                       ) {
+                       if (hard_link) {
 // GNU tar 1.26 does not check that we reached end of link name:
 // if "dir/hardlink" is hardlinked to "file",
 // tar xvf a.tar --strip-components=1 says:
 //  tar: hardlink: Cannot hard link to '': No such file or directory
 // and continues processing. We silently skip such entries.
-                               dst_link = strchr(dst_link, '/');
-                               if (!dst_link || dst_link[1] == '\0') {
+                               hard_link = strchr(hard_link, '/');
+                               if (!hard_link || hard_link[1] == '\0') {
                                        data_skip(archive_handle);
-                                       return;
+                                       goto ret;
                                }
-                               dst_link++;
+                               hard_link++;
                        }
                } while (--n != 0);
        }
@@ -79,12 +79,7 @@ void FAST_FUNC data_extract_all(archive_handle_t *archive_handle)
        if (archive_handle->ah_flags & ARCHIVE_UNLINK_OLD) {
                /* Remove the entry if it exists */
                if (!S_ISDIR(file_header->mode)) {
-                       /* Is it hardlink?
-                        * We encode hard links as regular files of size 0 with a symlink */
-                       if (S_ISREG(file_header->mode)
-                        && file_header->size == 0
-                        && dst_link
-                       ) {
+                       if (hard_link) {
                                /* Ugly special case:
                                 * tar cf t.tar hardlink1 hardlink2 hardlink1
                                 * results in this tarball structure:
@@ -92,7 +87,7 @@ void FAST_FUNC data_extract_all(archive_handle_t *archive_handle)
                                 * hardlink2 -> hardlink1
                                 * hardlink1 -> hardlink1 <== !!!
                                 */
-                               if (strcmp(dst_link, dst_name) == 0)
+                               if (strcmp(hard_link, dst_name) == 0)
                                        goto ret;
                        }
                        /* Proceed with deleting */
@@ -128,19 +123,15 @@ void FAST_FUNC data_extract_all(archive_handle_t *archive_handle)
                }
        }
 
-       /* Handle hard links separately
-        * We encode hard links as regular files of size 0 with a symlink */
-       if (S_ISREG(file_header->mode)
-        && file_header->size == 0
-        && dst_link
-       ) {
-               /* Hard link */
-               res = link(dst_link, dst_name);
+       /* Handle hard links separately */
+       if (hard_link) {
+               res = link(hard_link, dst_name);
                if (res != 0 && !(archive_handle->ah_flags & ARCHIVE_EXTRACT_QUIET)) {
+                       /* shared message */
                        bb_perror_msg("can't create %slink "
-                                       "from %s to %s", "hard",
+                                       "%s to %s", "hard",
                                        dst_name,
-                                       dst_link);
+                                       hard_link);
                }
                /* Hardlinks have no separate mode/ownership, skip chown/chmod */
                goto ret;
@@ -191,8 +182,9 @@ void FAST_FUNC data_extract_all(archive_handle_t *archive_handle)
                if (res != 0
                 && !(archive_handle->ah_flags & ARCHIVE_EXTRACT_QUIET)
                ) {
+                       /* shared message */
                        bb_perror_msg("can't create %slink "
-                               "from %s to %s", "sym",
+                               "%s to %s", "sym",
                                dst_name,
                                file_header->link_target);
                }