Fix tar -j support

Use the old fork() method of tar compression support, rather than
read_bz2....
 - (*uncompress)(int in, int out) seems like a more natural interface
for compression code.
 - it might improve performance by seperating the work into one cpu
bound and one io bound process.
 - There is extra code required to do read_[gz|bunzip] since (*uncompress)(int in,
int out) will normally be used by the standalone compression applet.

There have been problems with this method so if you see a "Short read"
error let me know.

3 files changed, 43 insertions, 23 deletions

diff --git a/archival/libunarchive/decompress_bunzip2.c b/archival/libunarchive/decompress_bunzip2.c
index c66df5e6d..83232fbe8 100644
--- a/archival/libunarchive/decompress_bunzip2.c
+++ b/archival/libunarchive/decompress_bunzip2.c
@@ -590,22 +590,6 @@ extern int uncompressStream(int src_fd, int dst_fd)
 	return i;
 }
 
-/* This new version is not yet properly integrated with tar */
-extern ssize_t read_bz2(int fd, void *buf, size_t count)
-{
-#warning FIXME "bzip2 tar support is broken!"
-	return(0);
-}
-
-extern void BZ2_bzReadOpen(int fd, void *unused, int nUnused)
-{
-#warning FIXME "bzip2 tar support is broken!"
-}
-extern void BZ2_bzReadClose(void)
-{
-#warning FIXME "bzip2 tar support is broken!"
-}
-
 #ifdef TESTING
 
 static char * const bunzip_errors[]={NULL,"Bad file checksum","Not bzip data",
diff --git a/archival/libunarchive/get_header_tar.c b/archival/libunarchive/get_header_tar.c
index cf5ac6b62..7fa0a0088 100644
--- a/archival/libunarchive/get_header_tar.c
+++ b/archival/libunarchive/get_header_tar.c
@@ -65,7 +65,7 @@ extern char get_header_tar(archive_handle_t *archive_handle)
 
 	/* If there is no filename its an empty header */
 	if (tar.formated.name[0] == 0) {
-		return(EXIT_SUCCESS);
+		return(EXIT_FAILURE);
 	}
 
 	/* Check header has valid magic, "ustar" is for the proper tar
diff --git a/archival/libunarchive/get_header_tar_bz2.c b/archival/libunarchive/get_header_tar_bz2.c
index b49ccae18..3d50aecd3 100644
--- a/archival/libunarchive/get_header_tar_bz2.c
+++ b/archival/libunarchive/get_header_tar_bz2.c
@@ -15,6 +15,7 @@
  */
 
 #include <sys/types.h>
+#include <sys/wait.h>
 #include <signal.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -25,18 +26,53 @@
 
 extern char get_header_tar_bz2(archive_handle_t *archive_handle)
 {
-	BZ2_bzReadOpen(archive_handle->src_fd, NULL, 0);
+	int fd_pipe[2];
+	int pid;
 
-	archive_handle->read = read_bz2;
+	/* Cant lseek over pipe's */
+	archive_handle->read = safe_read;
 	archive_handle->seek = seek_by_char;
 
+	if (pipe(fd_pipe) != 0) {
+		bb_error_msg_and_die("Can't create pipe");
+	}
+
+	pid = fork();
+	if (pid == -1) {
+		bb_error_msg_and_die("Fork failed\n");
+	}
+
+	if (pid == 0) {
+		/* child process */
+	    close(fd_pipe[0]); /* We don't wan't to read from the pipe */
+	    uncompressStream(archive_handle->src_fd, fd_pipe[1]);
+		check_trailer_gzip(archive_handle->src_fd);
+	    close(fd_pipe[1]); /* Send EOF */
+	    exit(0);
+	    /* notreached */
+	}
+	/* parent process */
+	close(fd_pipe[1]); /* Don't want to write down the pipe */
+	close(archive_handle->src_fd);
+
+	archive_handle->src_fd = fd_pipe[0];
+
 	archive_handle->offset = 0;
 	while (get_header_tar(archive_handle) == EXIT_SUCCESS);
 
-	/* Cleanup */
-	BZ2_bzReadClose();
+	close(fd_pipe[0]);
+printf("finished\n");
+#if 0
+	if (kill(pid, SIGTERM) == -1) {
+		bb_error_msg_and_die("Couldnt kill gunzip process");
+	}
+#endif
 	
-	/* Can only do one tar.bz2 per archive */
+	/* I dont think this is needed */
+	if (waitpid(pid, NULL, 0) == -1) {
+		bb_error_msg("Couldnt wait ?");
+	}
+
+	/* Can only do one file at a time */
 	return(EXIT_FAILURE);
 }
-