4 files changed, 41 insertions, 37 deletions

diff --git a/archival/libunarchive/data_extract_all.c b/archival/libunarchive/data_extract_all.c
index 0bb5bfe33..76e7edf24 100644
--- a/archival/libunarchive/data_extract_all.c
+++ b/archival/libunarchive/data_extract_all.c
@@ -25,7 +25,8 @@ void data_extract_all(archive_handle_t *archive_handle)
                  && (unlink(file_header->name) == -1)
                  && (errno != ENOENT)
                 ) {
-                        bb_perror_msg_and_die("cannot remove old file");
+                        bb_perror_msg_and_die("cannot remove old file %s",
+                                        file_header->name);
                 }
         }
         else if (archive_handle->flags & ARCHIVE_EXTRACT_NEWER) {
@@ -52,13 +53,16 @@ void data_extract_all(archive_handle_t *archive_handle)
 
         /* Handle hard links separately
          * We identified hard links as regular files of size 0 with a symlink */
-        if (S_ISREG(file_header->mode) && (file_header->link_name)
+        if (S_ISREG(file_header->mode) && (file_header->link_target)
          && (file_header->size == 0)
         ) {
                 /* hard link */
-                res = link(file_header->link_name, file_header->name);
+                res = link(file_header->link_target, file_header->name);
                 if ((res == -1) && !(archive_handle->flags & ARCHIVE_EXTRACT_QUIET)) {
-                        bb_perror_msg("cannot create hard link");
+                        bb_perror_msg("cannot create %slink "
+                                        "from %s to %s", "hard",
+                                        file_header->name,
+                                        file_header->link_target);
                 }
         } else {
                 /* Create the filesystem entry */
@@ -73,22 +77,22 @@ void data_extract_all(archive_handle_t *archive_handle)
                 }
                 case S_IFDIR:
                         res = mkdir(file_header->name, file_header->mode);
-                        if ((errno != EISDIR) && (res == -1)
+                        if ((res == -1) && (errno != EISDIR)
                          && !(archive_handle->flags & ARCHIVE_EXTRACT_QUIET)
                         ) {
-                                bb_perror_msg("extract_archive: %s", file_header->name);
+                                bb_perror_msg("cannot make dir %s", file_header->name);
                         }
                         break;
                 case S_IFLNK:
                         /* Symlink */
-                        res = symlink(file_header->link_name, file_header->name);
+                        res = symlink(file_header->link_target, file_header->name);
                         if ((res == -1)
                          && !(archive_handle->flags & ARCHIVE_EXTRACT_QUIET)
                         ) {
-                                bb_perror_msg("cannot create symlink "
-                                        "from %s to '%s'",
+                                bb_perror_msg("cannot create %slink "
+                                        "from %s to %s", "sym",
                                         file_header->name,
-                                        file_header->link_name);
+                                        file_header->link_target);
                         }
                         break;
                 case S_IFSOCK:
@@ -110,18 +114,18 @@ void data_extract_all(archive_handle_t *archive_handle)
         if (!(archive_handle->flags & ARCHIVE_NOPRESERVE_OWN)) {
                 lchown(file_header->name, file_header->uid, file_header->gid);
         }
-        /* uclibc has no lchmod, glibc is even stranger -
-         * it has lchmod which seems to do nothing!
-         * so we use chmod... */
-        if (!(archive_handle->flags & ARCHIVE_NOPRESERVE_PERM)
-         && (file_header->mode & S_IFMT) != S_IFLNK
-        ) {
-                chmod(file_header->name, file_header->mode);
-        }
-
-        if (archive_handle->flags & ARCHIVE_PRESERVE_DATE) {
-                struct utimbuf t;
-                t.actime = t.modtime = file_header->mtime;
-                utime(file_header->name, &t);
+        if ((file_header->mode & S_IFMT) != S_IFLNK) {
+                /* uclibc has no lchmod, glibc is even stranger -
+                 * it has lchmod which seems to do nothing!
+                 * so we use chmod... */
+                if (!(archive_handle->flags & ARCHIVE_NOPRESERVE_PERM)) {
+                        chmod(file_header->name, file_header->mode);
+                }
+                /* same for utime */
+                if (archive_handle->flags & ARCHIVE_PRESERVE_DATE) {
+                        struct utimbuf t;
+                        t.actime = t.modtime = file_header->mtime;
+                        utime(file_header->name, &t);
+                }
         }
 }
diff --git a/archival/libunarchive/get_header_cpio.c b/archival/libunarchive/get_header_cpio.c
index 6fd134018..724368be2 100644
--- a/archival/libunarchive/get_header_cpio.c
+++ b/archival/libunarchive/get_header_cpio.c
@@ -30,7 +30,7 @@ char get_header_cpio(archive_handle_t *archive_handle)
                 tmp = saved_hardlinks;
                 oldtmp = NULL;
 
-                file_header->link_name = file_header->name;
+                file_header->link_target = file_header->name;
                 file_header->size = 0;
 
                 while (tmp) {
@@ -56,7 +56,7 @@ char get_header_cpio(archive_handle_t *archive_handle)
                                 saved_hardlinks = tmp;
                 }
 
-                file_header->name = file_header->link_name;
+                file_header->name = file_header->link_target;
 
                 if (pending_hardlinks > 1) {
                         bb_error_msg("error resolving hardlink: archive made by GNU cpio 2.0-2.2?");
@@ -122,12 +122,12 @@ char get_header_cpio(archive_handle_t *archive_handle)
         }
 
         if (S_ISLNK(file_header->mode)) {
-                file_header->link_name = xzalloc(file_header->size + 1);
-                xread(archive_handle->src_fd, file_header->link_name, file_header->size);
+                file_header->link_target = xzalloc(file_header->size + 1);
+                xread(archive_handle->src_fd, file_header->link_target, file_header->size);
                 archive_handle->offset += file_header->size;
                 file_header->size = 0; /* Stop possible seeks in future */
         } else {
-                file_header->link_name = NULL;
+                file_header->link_target = NULL;
         }
         if (nlink > 1 && !S_ISDIR(file_header->mode)) {
                 if (file_header->size == 0) { /* Put file on a linked list for later */
@@ -154,7 +154,7 @@ char get_header_cpio(archive_handle_t *archive_handle)
 
         archive_handle->offset += file_header->size;
 
-        free(file_header->link_name);
+        free(file_header->link_target);
 
         return EXIT_SUCCESS;
 }
diff --git a/archival/libunarchive/get_header_tar.c b/archival/libunarchive/get_header_tar.c
index b3efdec94..d42f4c276 100644
--- a/archival/libunarchive/get_header_tar.c
+++ b/archival/libunarchive/get_header_tar.c
@@ -141,13 +141,13 @@ char get_header_tar(archive_handle_t *archive_handle)
                 unsigned major = GET_OCTAL(tar.devmajor);
                 file_header->device = makedev(major, minor);
         }
-        file_header->link_name = NULL;
+        file_header->link_target = NULL;
         if (!linkname && parse_names && tar.linkname[0]) {
                 /* we trash magic[0] here, it's ok */
                 tar.linkname[sizeof(tar.linkname)] = '\0';
-                file_header->link_name = xstrdup(tar.linkname);
-                /* FIXME: what if we have non-link object with link_name? */
-                /* Will link_name be free()ed? */
+                file_header->link_target = xstrdup(tar.linkname);
+                /* FIXME: what if we have non-link object with link_target? */
+                /* Will link_target be free()ed? */
         }
         file_header->mtime = GET_OCTAL(tar.mtime);
         file_header->size = GET_OCTAL(tar.size);
@@ -248,7 +248,7 @@ char get_header_tar(archive_handle_t *archive_handle)
                 longname = NULL;
         }
         if (linkname) {
-                file_header->link_name = linkname;
+                file_header->link_target = linkname;
                 linkname = NULL;
         }
 #endif
@@ -277,7 +277,7 @@ char get_header_tar(archive_handle_t *archive_handle)
         }
         archive_handle->offset += file_header->size;
 
-        free(file_header->link_name);
+        free(file_header->link_target);
         /* Do not free(file_header->name)! */
 
         return EXIT_SUCCESS;
diff --git a/archival/libunarchive/header_verbose_list.c b/archival/libunarchive/header_verbose_list.c
index 7b97e524c..f3b0d8c5c 100644
--- a/archival/libunarchive/header_verbose_list.c
+++ b/archival/libunarchive/header_verbose_list.c
@@ -23,8 +23,8 @@ void header_verbose_list(const file_header_t *file_header)
                 mtime->tm_sec,
                 file_header->name);
 
-        if (file_header->link_name) {
-                printf(" -> %s", file_header->link_name);
+        if (file_header->link_target) {
+                printf(" -> %s", file_header->link_target);
         }
         /* putchar isnt used anywhere else i dont think */
         puts("");