Fail silently if a partial tar header is read as tar.bz2 is leaving trailing junk (not sure why), add some missing files

7 files changed, 105 insertions, 5 deletions

diff --git a/archival/libunarchive/Makefile.in b/archival/libunarchive/Makefile.in
index 469004d..5675d09 100644
--- a/archival/libunarchive/Makefile.in
+++ b/archival/libunarchive/Makefile.in
@@ -36,6 +36,7 @@ LIBUNARCHIVE-y:= \
 \
 	get_header_ar.o \
 	get_header_tar.o \
+	get_header_tar_bz2.o \
 	get_header_tar_gz.o \
 \
 	header_skip.o \
diff --git a/archival/libunarchive/decompress_bunzip2.c b/archival/libunarchive/decompress_bunzip2.c
index 00ae5a4..4b611b8 100644
--- a/archival/libunarchive/decompress_bunzip2.c
+++ b/archival/libunarchive/decompress_bunzip2.c
@@ -1274,7 +1274,7 @@ save_state_and_return:
 	return retVal;   
 }
 
-static void BZ2_bzReadClose(void)
+extern void BZ2_bzReadClose(void)
 {
 	if (bzf->initialisedOk) {
 		bz_stream *strm = &(bzf->strm);
diff --git a/archival/libunarchive/filter_accept_list_reassign.c b/archival/libunarchive/filter_accept_list_reassign.c
new file mode 100644
index 0000000..fa294c7
--- /dev/null
+++ b/archival/libunarchive/filter_accept_list_reassign.c
@@ -0,0 +1,59 @@
+/*
+ *  Copyright (C) 2002 by Glenn McGrath
+ *
+ *  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
+ *  the Free Software Foundation; either version 2 of the License, or
+ *  (at your option) any later version.
+ *
+ *  This program is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with this program; if not, write to the Free Software
+ *  Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ */
+
+#include <stdlib.h>
+#include <string.h>
+#include <unistd.h>
+
+#include "libbb.h"
+#include "unarchive.h"
+
+/*
+ *	Reassign the subarchive metadata parser based on the filename extension
+ *  e.g. if its a .tar.gz modify archive_handle->sub_archive to process a .tar.gz
+ *       or if its a .tar.bz2 make archive_handle->sub_archive handle that 
+ */
+extern char filter_accept_list_reassign(archive_handle_t *archive_handle)
+{
+	/* Check the file entry is in the accept list */
+	if (find_list_entry(archive_handle->accept, archive_handle->file_header->name)) {
+		const char *name_ptr;
+
+		/* Extract the last 2 extensions */
+		name_ptr = strrchr(archive_handle->file_header->name, '.');
+
+		/* Modify the subarchive handler based on the extension */
+#ifdef CONFIG_FEATURE_DEB_TAR_GZ
+		if (strcmp(name_ptr, ".gz") == 0) {
+			archive_handle->sub_archive->read = read;
+			archive_handle->action_data_subarchive = get_header_tar_gz;
+			return(EXIT_SUCCESS);
+		}
+#endif
+#ifdef CONFIG_FEATURE_DEB_TAR_BZ2
+		if (strcmp(name_ptr, ".bz2") == 0) {
+			archive_handle->sub_archive->read = read_bz2;
+//			BZ2_bzReadOpen(archive_handle->sub_archive->src_fd, NULL, 0);
+			BZ2_bzReadOpen(archive_handle->src_fd, NULL, 0);
+			archive_handle->action_data_subarchive = get_header_tar;
+			return(EXIT_SUCCESS);
+		}
+#endif
+	}
+	return(EXIT_FAILURE);
+}
diff --git a/archival/libunarchive/get_header_tar.c b/archival/libunarchive/get_header_tar.c
index 20451d9..d3ff160 100644
--- a/archival/libunarchive/get_header_tar.c
+++ b/archival/libunarchive/get_header_tar.c
@@ -52,8 +52,8 @@ extern char get_header_tar(archive_handle_t *archive_handle)
 	/* Align header */
 	data_align(archive_handle, 512);
 
-	if (archive_xread_all_eof(archive_handle, tar.raw, 512) == 0) {
-		/* End of file */
+	if (archive_xread(archive_handle, tar.raw, 512) != 512) {
+		/* Assume end of file */
 		return(EXIT_FAILURE);
 	}
 	archive_handle->offset += 512;
diff --git a/archival/libunarchive/get_header_tar_bz2.c b/archival/libunarchive/get_header_tar_bz2.c
new file mode 100644
index 0000000..4e47166
--- /dev/null
+++ b/archival/libunarchive/get_header_tar_bz2.c
@@ -0,0 +1,39 @@
+/*
+ *  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
+ *  the Free Software Foundation; either version 2 of the License, or
+ *  (at your option) any later version.
+ *
+ *  This program is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU Library General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with this program; if not, write to the Free Software
+ *  Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ */
+
+#include <sys/types.h>
+#include <signal.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <unistd.h>
+#include "libbb.h"
+#include "unarchive.h"
+
+extern char get_header_tar_bz2(archive_handle_t *archive_handle)
+{
+	BZ2_bzReadOpen(archive_handle->src_fd, NULL, 0);
+
+	archive_handle->offset = 0;
+	while (get_header_tar(archive_handle) == EXIT_SUCCESS);
+
+	/* Cleanup */
+	BZ2_bzReadClose();
+	
+	/* Can only do one tar.bz2 per archive */
+	return(EXIT_FAILURE);
+}
+
diff --git a/archival/tar.c b/archival/tar.c
index e1e121a..95ef33b 100644
--- a/archival/tar.c
+++ b/archival/tar.c
@@ -748,8 +748,7 @@ int tar_main(int argc, char **argv)
 #endif /* CONFIG_FEATURE_TAR_GZIP */
 #ifdef CONFIG_FEATURE_TAR_BZIP2
 		if (tar_handle->read == read_bz2) {
-			BZ2_bzReadOpen(tar_handle->src_fd, NULL, 0);
-			while (get_header_tar(tar_handle) == EXIT_SUCCESS);
+			get_header_tar_bz2(tar_handle);
 		} else
 #endif /* CONFIG_FEATURE_TAR_BZIP2 */
 
diff --git a/include/unarchive.h b/include/unarchive.h
index 18bf089..b4e8e63 100644
--- a/include/unarchive.h
+++ b/include/unarchive.h
@@ -91,6 +91,7 @@ extern void check_trailer_gzip(int src_fd);
 
 extern char get_header_ar(archive_handle_t *archive_handle);
 extern char get_header_tar(archive_handle_t *archive_handle);
+extern char get_header_tar_bz2(archive_handle_t *archive_handle);
 extern char get_header_tar_gz(archive_handle_t *archive_handle);
 
 extern void seek_by_jump(const archive_handle_t *archive_handle, const unsigned int amount);
@@ -108,6 +109,7 @@ extern const llist_t *find_list_entry(const llist_t *list, const char *filename)
 
 extern ssize_t read_bz2(int fd, void *buf, size_t count);
 extern void BZ2_bzReadOpen(int fd, void *unused, int nUnused);
+extern void BZ2_bzReadClose(void);
 extern unsigned char uncompressStream(int src_fd, int dst_fd);
 
 #endif