- patch from Denis Vlasenko to add and use bb_xopen3()
[oweals/busybox.git] / archival / tar.c
index b18f1f87f2d6a36c04c9dcd3c8908ca26f9cb7c6..dcc2f163e4a695f6298f5da0e867048272764f43 100644 (file)
@@ -1,16 +1,15 @@
 /* vi: set sw=4 ts=4: */
 /*
- * Mini tar implementation for busybox 
+ * Mini tar implementation for busybox
  *
- * Modifed to use common extraction code used by ar, cpio, dpkg-deb, dpkg
- *  Glenn McGrath <bug1@optushome.com.au>
+ * Modified to use common extraction code used by ar, cpio, dpkg-deb, dpkg
+ *  Glenn McGrath <bug1@iinet.net.au>
  *
  * Note, that as of BusyBox-0.43, tar has been completely rewritten from the
- * ground up.  It still has remnents of the old code lying about, but it is
+ * ground up.  It still has remnants of the old code lying about, but it is
  * very different now (i.e., cleaner, less global variables, etc.)
  *
- * Copyright (C) 1999,2000 by Lineo, inc. and Erik Andersen
- * Copyright (C) 1999-2002 by Erik Andersen <andersee@debian.org>
+ * Copyright (C) 1999-2004 by Erik Andersen <andersen@codepoet.org>
  *
  * Based in part in the tar implementation in sash
  *  Copyright (c) 1999 by David I. Bell
  *
  * Based in part on the tar implementation from busybox-0.28
  *  Copyright (C) 1995 Bruce Perens
- *  This is free software under the GNU General Public License.
- *
- * 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
  *
+ * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
 #include <fcntl.h>
 #include <fnmatch.h>
 #include <string.h>
 #include <errno.h>
+#include <signal.h>
+#include <sys/wait.h>
+#include <sys/socket.h>
+#include <sys/sysmacros.h>     /* major() and minor() */
 #include "unarchive.h"
 #include "busybox.h"
 
 #ifdef CONFIG_FEATURE_TAR_CREATE
 
 /* Tar file constants  */
-# define TAR_MAGIC          "ustar"        /* ustar and a null */
-# define TAR_VERSION        "  "           /* Be compatable with GNU tar format */
-
-# ifndef MAJOR
-#  define MAJOR(dev) (((dev)>>8)&0xff)
-#  define MINOR(dev) ((dev)&0xff)
-# endif
+# define TAR_MAGIC          "ustar"    /* ustar and a null */
+# define TAR_VERSION        "  "       /* Be compatible with GNU tar format */
 
-static const int TAR_BLOCK_SIZE = 512;
-static const int TAR_MAGIC_LEN = 6;
-static const int TAR_VERSION_LEN = 2;
+#define TAR_BLOCK_SIZE         512
+#define TAR_MAGIC_LEN          6
+#define TAR_VERSION_LEN                2
 
 /* POSIX tar Header Block, from POSIX 1003.1-1990  */
-enum { NAME_SIZE = 100 }; /* because gcc won't let me use 'static const int' */
-struct TarHeader
-{                            /* byte offset */
-       char name[NAME_SIZE];         /*   0-99 */
-       char mode[8];                 /* 100-107 */
-       char uid[8];                  /* 108-115 */
-       char gid[8];                  /* 116-123 */
-       char size[12];                /* 124-135 */
-       char mtime[12];               /* 136-147 */
-       char chksum[8];               /* 148-155 */
-       char typeflag;                /* 156-156 */
-       char linkname[NAME_SIZE];     /* 157-256 */
-       char magic[6];                /* 257-262 */
-       char version[2];              /* 263-264 */
-       char uname[32];               /* 265-296 */
-       char gname[32];               /* 297-328 */
-       char devmajor[8];             /* 329-336 */
-       char devminor[8];             /* 337-344 */
-       char prefix[155];             /* 345-499 */
-       char padding[12];             /* 500-512 (pad to exactly the TAR_BLOCK_SIZE) */
+#define NAME_SIZE                      100
+struct TarHeader {             /* byte offset */
+       char name[NAME_SIZE];   /*   0-99 */
+       char mode[8];           /* 100-107 */
+       char uid[8];            /* 108-115 */
+       char gid[8];            /* 116-123 */
+       char size[12];          /* 124-135 */
+       char mtime[12];         /* 136-147 */
+       char chksum[8];         /* 148-155 */
+       char typeflag;          /* 156-156 */
+       char linkname[NAME_SIZE];       /* 157-256 */
+       char magic[6];          /* 257-262 */
+       char version[2];        /* 263-264 */
+       char uname[32];         /* 265-296 */
+       char gname[32];         /* 297-328 */
+       char devmajor[8];       /* 329-336 */
+       char devminor[8];       /* 337-344 */
+       char prefix[155];       /* 345-499 */
+       char padding[12];       /* 500-512 (pad to exactly the TAR_BLOCK_SIZE) */
 };
 typedef struct TarHeader TarHeader;
 
@@ -95,71 +78,64 @@ typedef struct TarHeader TarHeader;
 ** Even these functions use the xxxHardLinkInfo() functions.
 */
 typedef struct HardLinkInfo HardLinkInfo;
-struct HardLinkInfo
-{
-       HardLinkInfo *next;           /* Next entry in list */
-       dev_t dev;                    /* Device number */
-       ino_t ino;                    /* Inode number */
-       short linkCount;              /* (Hard) Link Count */
-       char name[1];                 /* Start of filename (must be last) */
+struct HardLinkInfo {
+       HardLinkInfo *next;     /* Next entry in list */
+       dev_t dev;                      /* Device number */
+       ino_t ino;                      /* Inode number */
+       short linkCount;        /* (Hard) Link Count */
+       char name[1];           /* Start of filename (must be last) */
 };
 
 /* Some info to be carried along when creating a new tarball */
-struct TarBallInfo
-{
-       char* fileName;               /* File name of the tarball */
-       int tarFd;                    /* Open-for-write file descriptor
-                                                                        for the tarball */
-       struct stat statBuf;          /* Stat info for the tarball, letting
-                                                                        us know the inode and device that the
-                                                                        tarball lives, so we can avoid trying 
-                                                                        to include the tarball into itself */
-       int verboseFlag;              /* Whether to print extra stuff or not */
-       char** excludeList;           /* List of files to not include */
-       HardLinkInfo *hlInfoHead;     /* Hard Link Tracking Information */
-       HardLinkInfo *hlInfo;         /* Hard Link Info for the current file */
+struct TarBallInfo {
+       char *fileName;                 /* File name of the tarball */
+       int tarFd;                              /* Open-for-write file descriptor
+                                                          for the tarball */
+       struct stat statBuf;    /* Stat info for the tarball, letting
+                                                          us know the inode and device that the
+                                                          tarball lives, so we can avoid trying
+                                                          to include the tarball into itself */
+       int verboseFlag;                /* Whether to print extra stuff or not */
+       const llist_t *excludeList;     /* List of files to not include */
+       HardLinkInfo *hlInfoHead;       /* Hard Link Tracking Information */
+       HardLinkInfo *hlInfo;   /* Hard Link Info for the current file */
 };
 typedef struct TarBallInfo TarBallInfo;
 
 /* A nice enum with all the possible tar file content types */
-enum TarFileType 
-{
-       REGTYPE  = '0',            /* regular file */
-       REGTYPE0 = '\0',           /* regular file (ancient bug compat)*/
-       LNKTYPE  = '1',            /* hard link */
-       SYMTYPE  = '2',            /* symbolic link */
-       CHRTYPE  = '3',            /* character special */
-       BLKTYPE  = '4',            /* block special */
-       DIRTYPE  = '5',            /* directory */
-       FIFOTYPE = '6',            /* FIFO special */
-       CONTTYPE = '7',            /* reserved */
-       GNULONGLINK = 'K',         /* GNU long (>100 chars) link name */
-       GNULONGNAME = 'L',         /* GNU long (>100 chars) file name */
+enum TarFileType {
+       REGTYPE = '0',          /* regular file */
+       REGTYPE0 = '\0',        /* regular file (ancient bug compat) */
+       LNKTYPE = '1',          /* hard link */
+       SYMTYPE = '2',          /* symbolic link */
+       CHRTYPE = '3',          /* character special */
+       BLKTYPE = '4',          /* block special */
+       DIRTYPE = '5',          /* directory */
+       FIFOTYPE = '6',         /* FIFO special */
+       CONTTYPE = '7',         /* reserved */
+       GNULONGLINK = 'K',      /* GNU long (>100 chars) link name */
+       GNULONGNAME = 'L',      /* GNU long (>100 chars) file name */
 };
 typedef enum TarFileType TarFileType;
 
 /* Might be faster (and bigger) if the dev/ino were stored in numeric order;) */
-static void
-addHardLinkInfo (HardLinkInfo **hlInfoHeadPtr, dev_t dev, ino_t ino,
-               short linkCount, const char *name)
+static inline void addHardLinkInfo(HardLinkInfo ** hlInfoHeadPtr,
+                                       struct stat *statbuf,
+                                       const char *name)
 {
        /* Note: hlInfoHeadPtr can never be NULL! */
        HardLinkInfo *hlInfo;
 
-       hlInfo = (HardLinkInfo *)xmalloc(sizeof(HardLinkInfo)+strlen(name)+1);
-       if (hlInfo) {
-               hlInfo->next = *hlInfoHeadPtr;
-               *hlInfoHeadPtr = hlInfo;
-               hlInfo->dev = dev;
-               hlInfo->ino = ino;
-               hlInfo->linkCount = linkCount;
-               strcpy(hlInfo->name, name);
-       }
-       return;
+       hlInfo = (HardLinkInfo *) xmalloc(sizeof(HardLinkInfo) + strlen(name));
+       hlInfo->next = *hlInfoHeadPtr;
+       *hlInfoHeadPtr = hlInfo;
+       hlInfo->dev = statbuf->st_dev;
+       hlInfo->ino = statbuf->st_ino;
+       hlInfo->linkCount = statbuf->st_nlink;
+       strcpy(hlInfo->name, name);
 }
 
-static void
-freeHardLinkInfo (HardLinkInfo **hlInfoHeadPtr)
+static void freeHardLinkInfo(HardLinkInfo ** hlInfoHeadPtr)
 {
        HardLinkInfo *hlInfo = NULL;
        HardLinkInfo *hlInfoNext = NULL;
@@ -177,21 +153,20 @@ freeHardLinkInfo (HardLinkInfo **hlInfoHeadPtr)
 }
 
 /* Might be faster (and bigger) if the dev/ino were stored in numeric order;) */
-static HardLinkInfo *
-findHardLinkInfo (HardLinkInfo *hlInfo, dev_t dev, ino_t ino)
+static inline HardLinkInfo *findHardLinkInfo(HardLinkInfo * hlInfo, struct stat *statbuf)
 {
-       while(hlInfo) {
-               if ((ino == hlInfo->ino) && (dev == hlInfo->dev))
+       while (hlInfo) {
+               if ((statbuf->st_ino == hlInfo->ino) && (statbuf->st_dev == hlInfo->dev))
                        break;
                hlInfo = hlInfo->next;
        }
-       return(hlInfo);
+       return (hlInfo);
 }
 
 /* Put an octal string into the specified buffer.
  * The number is zero and space padded and possibly null padded.
- * Returns TRUE if successful.  */ 
-static int putOctal (char *cp, int len, long value)
+ * Returns TRUE if successful.  */
+static int putOctal(char *cp, int len, long value)
 {
        int tempLength;
        char tempBuffer[32];
@@ -199,10 +174,10 @@ static int putOctal (char *cp, int len, long value)
 
        /* Create a string of the specified length with an initial space,
         * leading zeroes and the octal number, and a trailing null.  */
-       sprintf (tempString, "%0*lo", len - 1, value);
+       sprintf(tempString, "%0*lo", len - 1, value);
 
        /* If the string is too large, suppress the leading space.  */
-       tempLength = strlen (tempString) + 1;
+       tempLength = strlen(tempString) + 1;
        if (tempLength > len) {
                tempLength--;
                tempString++;
@@ -217,71 +192,74 @@ static int putOctal (char *cp, int len, long value)
                return FALSE;
 
        /* Copy the string to the field.  */
-       memcpy (cp, tempString, len);
+       memcpy(cp, tempString, len);
 
        return TRUE;
 }
 
 /* Write out a tar header for the specified file/directory/whatever */
-static int
-writeTarHeader(struct TarBallInfo *tbInfo, const char *header_name,
-               const char *real_name, struct stat *statbuf)
+static inline int writeTarHeader(struct TarBallInfo *tbInfo,
+               const char *header_name, const char *real_name, struct stat *statbuf)
 {
-       long chksum=0;
+       long chksum = 0;
        struct TarHeader header;
        const unsigned char *cp = (const unsigned char *) &header;
        ssize_t size = sizeof(struct TarHeader);
-               
-       memset( &header, 0, size);
 
-       strncpy(header.name, header_name, sizeof(header.name)); 
+       memset(&header, 0, size);
+
+       strncpy(header.name, header_name, sizeof(header.name));
 
        putOctal(header.mode, sizeof(header.mode), statbuf->st_mode);
        putOctal(header.uid, sizeof(header.uid), statbuf->st_uid);
        putOctal(header.gid, sizeof(header.gid), statbuf->st_gid);
-       putOctal(header.size, sizeof(header.size), 0); /* Regular file size is handled later */
+       putOctal(header.size, sizeof(header.size), 0);  /* Regular file size is handled later */
        putOctal(header.mtime, sizeof(header.mtime), statbuf->st_mtime);
-       strncpy(header.magic, TAR_MAGIC TAR_VERSION, 
-                       TAR_MAGIC_LEN + TAR_VERSION_LEN );
+       strncpy(header.magic, TAR_MAGIC TAR_VERSION,
+                       TAR_MAGIC_LEN + TAR_VERSION_LEN);
 
        /* Enter the user and group names (default to root if it fails) */
-       my_getpwuid(header.uname, statbuf->st_uid);
-       if (! *header.uname)
-               strcpy(header.uname, "root");
-       my_getgrgid(header.gname, statbuf->st_gid);
-       if (! *header.uname)
+       if (bb_getpwuid(header.uname, statbuf->st_uid, sizeof(header.uname)) == NULL)
                strcpy(header.uname, "root");
+       if (bb_getgrgid(header.gname, statbuf->st_gid, sizeof(header.gname)) == NULL)
+               strcpy(header.gname, "root");
 
        if (tbInfo->hlInfo) {
                /* This is a hard link */
                header.typeflag = LNKTYPE;
-               strncpy(header.linkname, tbInfo->hlInfo->name, sizeof(header.linkname));
+               strncpy(header.linkname, tbInfo->hlInfo->name,
+                               sizeof(header.linkname));
        } else if (S_ISLNK(statbuf->st_mode)) {
                char *lpath = xreadlink(real_name);
-               if (!lpath) /* Already printed err msg inside xreadlink() */
-                       return ( FALSE);
-               header.typeflag  = SYMTYPE;
-               strncpy(header.linkname, lpath, sizeof(header.linkname)); 
+
+               if (!lpath)             /* Already printed err msg inside xreadlink() */
+                       return (FALSE);
+               header.typeflag = SYMTYPE;
+               strncpy(header.linkname, lpath, sizeof(header.linkname));
                free(lpath);
        } else if (S_ISDIR(statbuf->st_mode)) {
-               header.typeflag  = DIRTYPE;
-               strncat(header.name, "/", sizeof(header.name)); 
+               header.typeflag = DIRTYPE;
+               strncat(header.name, "/", sizeof(header.name));
        } else if (S_ISCHR(statbuf->st_mode)) {
-               header.typeflag  = CHRTYPE;
-               putOctal(header.devmajor, sizeof(header.devmajor), MAJOR(statbuf->st_rdev));
-               putOctal(header.devminor, sizeof(header.devminor), MINOR(statbuf->st_rdev));
+               header.typeflag = CHRTYPE;
+               putOctal(header.devmajor, sizeof(header.devmajor),
+                                major(statbuf->st_rdev));
+               putOctal(header.devminor, sizeof(header.devminor),
+                                minor(statbuf->st_rdev));
        } else if (S_ISBLK(statbuf->st_mode)) {
-               header.typeflag  = BLKTYPE;
-               putOctal(header.devmajor, sizeof(header.devmajor), MAJOR(statbuf->st_rdev));
-               putOctal(header.devminor, sizeof(header.devminor), MINOR(statbuf->st_rdev));
+               header.typeflag = BLKTYPE;
+               putOctal(header.devmajor, sizeof(header.devmajor),
+                                major(statbuf->st_rdev));
+               putOctal(header.devminor, sizeof(header.devminor),
+                                minor(statbuf->st_rdev));
        } else if (S_ISFIFO(statbuf->st_mode)) {
-               header.typeflag  = FIFOTYPE;
+               header.typeflag = FIFOTYPE;
        } else if (S_ISREG(statbuf->st_mode)) {
-               header.typeflag  = REGTYPE;
+               header.typeflag = REGTYPE;
                putOctal(header.size, sizeof(header.size), statbuf->st_size);
        } else {
-               error_msg("%s: Unknown file type", real_name);
-               return ( FALSE);
+               bb_error_msg("%s: Unknown file type", real_name);
+               return (FALSE);
        }
 
        /* Calculate and store the checksum (i.e., the sum of all of the bytes of
@@ -294,38 +272,38 @@ writeTarHeader(struct TarBallInfo *tbInfo, const char *header_name,
        while (size-- > 0)
                chksum += *cp++;
        putOctal(header.chksum, 7, chksum);
-       
+
        /* Now write the header out to disk */
-       if ((size=full_write(tbInfo->tarFd, (char*)&header, sizeof(struct TarHeader))) < 0) {
-               error_msg(io_error, real_name, strerror(errno)); 
-               return ( FALSE);
+       if ((size =
+                bb_full_write(tbInfo->tarFd, (char *) &header,
+                                       sizeof(struct TarHeader))) < 0) {
+               bb_error_msg(bb_msg_io_error, real_name);
+               return (FALSE);
        }
        /* Pad the header up to the tar block size */
-       for (; size<TAR_BLOCK_SIZE; size++) {
+       for (; size < TAR_BLOCK_SIZE; size++) {
                write(tbInfo->tarFd, "\0", 1);
        }
        /* Now do the verbose thing (or not) */
-       if (tbInfo->verboseFlag==TRUE) {
+
+       if (tbInfo->verboseFlag) {
                FILE *vbFd = stdout;
-               if (tbInfo->tarFd == fileno(stdout))    // If the archive goes to stdout, verbose to stderr
+
+               if (tbInfo->tarFd == STDOUT_FILENO)     /* If the archive goes to stdout, verbose to stderr */
                        vbFd = stderr;
+
                fprintf(vbFd, "%s\n", header.name);
        }
 
-       return ( TRUE);
+       return (TRUE);
 }
 
-# if defined CONFIG_FEATURE_TAR_EXCLUDE
-static int exclude_file(char **excluded_files, const char *file)
+# ifdef CONFIG_FEATURE_TAR_FROM
+static inline int exclude_file(const llist_t *excluded_files, const char *file)
 {
-       int i;
-
-       if (excluded_files == NULL)
-               return 0;
-
-       for (i = 0; excluded_files[i] != NULL; i++) {
-               if (excluded_files[i][0] == '/') {
-                       if (fnmatch(excluded_files[i], file,
+       while (excluded_files) {
+               if (excluded_files->data[0] == '/') {
+                       if (fnmatch(excluded_files->data, file,
                                                FNM_PATHNAME | FNM_LEADING_DIR) == 0)
                                return 1;
                } else {
@@ -333,409 +311,564 @@ static int exclude_file(char **excluded_files, const char *file)
 
                        for (p = file; p[0] != '\0'; p++) {
                                if ((p == file || p[-1] == '/') && p[0] != '/' &&
-                                               fnmatch(excluded_files[i], p,
+                                       fnmatch(excluded_files->data, p,
                                                        FNM_PATHNAME | FNM_LEADING_DIR) == 0)
                                        return 1;
                        }
                }
+               excluded_files = excluded_files->link;
        }
 
        return 0;
 }
-#endif
+# else
+#define exclude_file(excluded_files, file) 0
+# endif
 
-static int writeFileToTarball(const char *fileName, struct stat *statbuf, void* userData)
+static int writeFileToTarball(const char *fileName, struct stat *statbuf,
+                                                         void *userData)
 {
-       struct TarBallInfo *tbInfo = (struct TarBallInfo *)userData;
+       struct TarBallInfo *tbInfo = (struct TarBallInfo *) userData;
        const char *header_name;
+       int inputFileFd = -1;
 
        /*
-       ** Check to see if we are dealing with a hard link.
-       ** If so -
-       ** Treat the first occurance of a given dev/inode as a file while
-       ** treating any additional occurances as hard links.  This is done
-       ** by adding the file information to the HardLinkInfo linked list.
-       */
+          ** Check to see if we are dealing with a hard link.
+          ** If so -
+          ** Treat the first occurance of a given dev/inode as a file while
+          ** treating any additional occurances as hard links.  This is done
+          ** by adding the file information to the HardLinkInfo linked list.
+        */
        tbInfo->hlInfo = NULL;
        if (statbuf->st_nlink > 1) {
-               tbInfo->hlInfo = findHardLinkInfo(tbInfo->hlInfoHead, statbuf->st_dev, 
-                               statbuf->st_ino);
+               tbInfo->hlInfo = findHardLinkInfo(tbInfo->hlInfoHead, statbuf);
                if (tbInfo->hlInfo == NULL)
-                       addHardLinkInfo (&tbInfo->hlInfoHead, statbuf->st_dev,
-                                       statbuf->st_ino, statbuf->st_nlink, fileName);
+                       addHardLinkInfo(&tbInfo->hlInfoHead, statbuf, fileName);
        }
 
        /* It is against the rules to archive a socket */
        if (S_ISSOCK(statbuf->st_mode)) {
-               error_msg("%s: socket ignored", fileName);
-               returnTRUE);
+               bb_error_msg("%s: socket ignored", fileName);
+               return (TRUE);
        }
 
        /* It is a bad idea to store the archive we are in the process of creating,
         * so check the device and inode to be sure that this particular file isn't
         * the new tarball */
        if (tbInfo->statBuf.st_dev == statbuf->st_dev &&
-                       tbInfo->statBuf.st_ino == statbuf->st_ino) {
-               error_msg("%s: file is the archive; skipping", fileName);
-               returnTRUE);
+               tbInfo->statBuf.st_ino == statbuf->st_ino) {
+               bb_error_msg("%s: file is the archive; skipping", fileName);
+               return (TRUE);
        }
 
        header_name = fileName;
        while (header_name[0] == '/') {
-               static int alreadyWarned=FALSE;
-               if (alreadyWarned==FALSE) {
-                       error_msg("Removing leading '/' from member names");
-                       alreadyWarned=TRUE;
+               static int alreadyWarned = FALSE;
+
+               if (alreadyWarned == FALSE) {
+                       bb_error_msg("Removing leading '/' from member names");
+                       alreadyWarned = TRUE;
                }
                header_name++;
        }
 
        if (strlen(fileName) >= NAME_SIZE) {
-               error_msg(name_longer_than_foo, NAME_SIZE);
-               return ( TRUE);
+               bb_error_msg(bb_msg_name_longer_than_foo, NAME_SIZE);
+               return (TRUE);
        }
 
        if (header_name[0] == '\0')
                return TRUE;
 
-# if defined CONFIG_FEATURE_TAR_EXCLUDE
-       if (exclude_file(tbInfo->excludeList, header_name)) {
+       if (ENABLE_FEATURE_TAR_FROM &&
+                       exclude_file(tbInfo->excludeList, header_name)) {
                return SKIP;
        }
-# endif //CONFIG_FEATURE_TAR_EXCLUDE
 
-       if (writeTarHeader(tbInfo, header_name, fileName, statbuf)==FALSE) {
-               return( FALSE);
-       } 
-
-       /* Now, if the file is a regular file, copy it out to the tarball */
-       if ((tbInfo->hlInfo == NULL)
-       &&  (S_ISREG(statbuf->st_mode))) {
-               int  inputFileFd;
-               char buffer[BUFSIZ];
-               ssize_t size=0, readSize=0;
+       /* Is this a regular file? */
+       if ((tbInfo->hlInfo == NULL) && (S_ISREG(statbuf->st_mode))) {
 
                /* open the file we want to archive, and make sure all is well */
                if ((inputFileFd = open(fileName, O_RDONLY)) < 0) {
-                       error_msg("%s: Cannot open: %s", fileName, strerror(errno));
-                       returnFALSE);
+                       bb_perror_msg("%s: Cannot open", fileName);
+                       return (FALSE);
                }
-               
+       }
+
+       /* Add an entry to the tarball */
+       if (writeTarHeader(tbInfo, header_name, fileName, statbuf) == FALSE) {
+               return (FALSE);
+       }
+
+       /* If it was a regular file, write out the body */
+       if (inputFileFd >= 0 ) {
+               ssize_t readSize = 0;
+
                /* write the file to the archive */
-               while ( (size = full_read(inputFileFd, buffer, sizeof(buffer))) > 0 ) {
-                       if (full_write(tbInfo->tarFd, buffer, size) != size ) {
-                               /* Output file seems to have a problem */
-                               error_msg(io_error, fileName, strerror(errno)); 
-                               return( FALSE);
-                       }
-                       readSize+=size;
-               }
-               if (size == -1) {
-                       error_msg(io_error, fileName, strerror(errno)); 
-                       return( FALSE);
-               }
+               readSize = bb_copyfd_eof(inputFileFd, tbInfo->tarFd);
+               close(inputFileFd);
+
                /* Pad the file up to the tar block size */
-               for (; (readSize%TAR_BLOCK_SIZE) != 0; readSize++) {
+               for (; (readSize % TAR_BLOCK_SIZE) != 0; readSize++)
                        write(tbInfo->tarFd, "\0", 1);
-               }
-               close( inputFileFd);
        }
 
-       returnTRUE);
+       return (TRUE);
 }
 
-static int writeTarFile(const char* tarName, int verboseFlag, char **argv,
-               char** excludeList)
+static inline int writeTarFile(const int tar_fd, const int verboseFlag,
+       const unsigned long dereferenceFlag, const llist_t *include,
+       const llist_t *exclude, const int gzip)
 {
-       int tarFd=-1;
-       int errorFlag=FALSE;
+       pid_t gzipPid = 0;
+
+       int errorFlag = FALSE;
        ssize_t size;
        struct TarBallInfo tbInfo;
-       tbInfo.verboseFlag = verboseFlag;
+
        tbInfo.hlInfoHead = NULL;
 
-       /* Make sure there is at least one file to tar up.  */
-       if (*argv == NULL)
-               error_msg_and_die("Cowardly refusing to create an empty archive");
-
-       /* Open the tar file for writing.  */
-       if (tarName == NULL)
-               tbInfo.tarFd = fileno(stdout);
-       else
-               tbInfo.tarFd = open (tarName, O_WRONLY | O_CREAT | O_TRUNC, 0644);
-       if (tbInfo.tarFd < 0) {
-               perror_msg( "Error opening '%s'", tarName);
-               freeHardLinkInfo(&tbInfo.hlInfoHead);
-               return ( FALSE);
-       }
-       tbInfo.excludeList=excludeList;
+       fchmod(tar_fd, 0644);
+       tbInfo.tarFd = tar_fd;
+       tbInfo.verboseFlag = verboseFlag;
+
        /* Store the stat info for the tarball's file, so
         * can avoid including the tarball into itself....  */
        if (fstat(tbInfo.tarFd, &tbInfo.statBuf) < 0)
-               error_msg_and_die(io_error, tarName, strerror(errno)); 
+               bb_perror_msg_and_die("Couldnt stat tar file");
+
+       if ((ENABLE_FEATURE_TAR_GZIP || ENABLE_FEATURE_TAR_BZIP2) && gzip) {
+               int gzipDataPipe[2] = { -1, -1 };
+               int gzipStatusPipe[2] = { -1, -1 };
+               volatile int vfork_exec_errno = 0;
+               char *zip_exec = (gzip == 1) ? "gzip" : "bzip2";
+
+
+               if (pipe(gzipDataPipe) < 0 || pipe(gzipStatusPipe) < 0)
+                       bb_perror_msg_and_die("create pipe");
+
+               signal(SIGPIPE, SIG_IGN);       /* we only want EPIPE on errors */
+
+# if __GNUC__
+                       /* Avoid vfork clobbering */
+                       (void) &include;
+                       (void) &errorFlag;
+                       (void) &zip_exec;
+# endif
+
+               gzipPid = vfork();
+
+               if (gzipPid == 0) {
+                       dup2(gzipDataPipe[0], 0);
+                       close(gzipDataPipe[1]);
+
+                       if (tbInfo.tarFd != 1)
+                               dup2(tbInfo.tarFd, 1);
+
+                       close(gzipStatusPipe[0]);
+                       fcntl(gzipStatusPipe[1], F_SETFD, FD_CLOEXEC);  /* close on exec shows success */
+
+                       execlp(zip_exec, zip_exec, "-f", NULL);
+                       vfork_exec_errno = errno;
+
+                       close(gzipStatusPipe[1]);
+                       exit(-1);
+               } else if (gzipPid > 0) {
+                       close(gzipDataPipe[0]);
+                       close(gzipStatusPipe[1]);
+
+                       while (1) {
+                               char buf;
+
+                               int n = bb_full_read(gzipStatusPipe[0], &buf, 1);
+
+                               if (n == 0 && vfork_exec_errno != 0) {
+                                       errno = vfork_exec_errno;
+                                       bb_perror_msg_and_die("Could not exec %s", zip_exec);
+                               } else if ((n < 0) && (errno == EAGAIN || errno == EINTR))
+                                       continue;       /* try it again */
+                               break;
+                       }
+                       close(gzipStatusPipe[0]);
+
+                       tbInfo.tarFd = gzipDataPipe[1];
+               } else bb_perror_msg_and_die("vfork gzip");
+       }
+
+       tbInfo.excludeList = exclude;
 
        /* Read the directory/files and iterate over them one at a time */
-       while (*argv != NULL) {
-               if (! recursive_action(*argv++, TRUE, FALSE, FALSE,
-                                       writeFileToTarball, writeFileToTarball, 
-                                       (void*) &tbInfo)) {
+       while (include) {
+               if (!recursive_action(include->data, TRUE, dereferenceFlag,
+                               FALSE, writeFileToTarball, writeFileToTarball, &tbInfo))
+               {
                        errorFlag = TRUE;
                }
+               include = include->link;
        }
        /* Write two empty blocks to the end of the archive */
-       for (size=0; size<(2*TAR_BLOCK_SIZE); size++) {
+       for (size = 0; size < (2 * TAR_BLOCK_SIZE); size++)
                write(tbInfo.tarFd, "\0", 1);
-       }
 
        /* To be pedantically correct, we would check if the tarball
         * is smaller than 20 tar blocks, and pad it if it was smaller,
         * but that isn't necessary for GNU tar interoperability, and
         * so is considered a waste of space */
 
+       /* Close so the child process (if any) will exit */
+       close(tbInfo.tarFd);
+
        /* Hang up the tools, close up shop, head home */
-       close(tarFd);
-       if (errorFlag) {
-               error_msg("Error exit delayed from previous errors");
+       if (ENABLE_FEATURE_CLEAN_UP)
                freeHardLinkInfo(&tbInfo.hlInfoHead);
-               return(FALSE);
-       }
-       freeHardLinkInfo(&tbInfo.hlInfoHead);
-       return( TRUE);
-}
-#endif //tar_create
 
-void append_file_to_list(const char *new_name, char ***list, int *list_count)
-{
-       *list = realloc(*list, sizeof(char *) * (*list_count + 2));
-       (*list)[*list_count] = xstrdup(new_name);
-       (*list_count)++;
-       (*list)[*list_count] = NULL;
-}
+       if (errorFlag)
+               bb_error_msg("Error exit delayed from previous errors");
+
+       if (gzipPid && waitpid(gzipPid, NULL, 0)==-1)
+               bb_error_msg("Couldnt wait");
 
-void append_file_list_to_list(char *filename, char ***name_list, int *num_of_entries)
+       return !errorFlag;
+}
+#else
+int writeTarFile(const int tar_fd, const int verboseFlag,
+       const unsigned long dereferenceFlag, const llist_t *include,
+       const llist_t *exclude, const int gzip);
+#endif /* tar_create */
+
+#ifdef CONFIG_FEATURE_TAR_FROM
+static llist_t *append_file_list_to_list(llist_t *list)
 {
        FILE *src_stream;
+       llist_t *cur = list;
+       llist_t *tmp;
        char *line;
-       
-       src_stream = xfopen(filename, "r");
-       while ((line = get_line_from_file(src_stream)) != NULL) {
-               chomp (line);
-               append_file_to_list(line, name_list, num_of_entries);
-               free(line);
+       llist_t *newlist = NULL;
+
+       while (cur) {
+               src_stream = bb_xfopen(cur->data, "r");
+               tmp = cur;
+               cur = cur->link;
+               free(tmp);
+               while ((line = bb_get_chomped_line_from_file(src_stream)) != NULL)
+                               newlist = llist_add_to(newlist, line);
+               fclose(src_stream);
        }
-       fclose(src_stream);
+       return newlist;
 }
+#else
+#define append_file_list_to_list(x)    0
+#endif
 
-#ifdef CONFIG_FEATURE_TAR_EXCLUDE
-/*
- * Create a list of names that are in the include list AND NOT in the exclude lists
- */
-char **list_and_not_list(char **include_list, char **exclude_list)
+#ifdef CONFIG_FEATURE_TAR_COMPRESS
+static char get_header_tar_Z(archive_handle_t *archive_handle)
 {
-       char **new_include_list = NULL;
-       int new_include_count = 0;
-       int include_count = 0;
-       int exclude_count;
+       /* Cant lseek over pipe's */
+       archive_handle->seek = seek_by_char;
 
-       if (include_list == NULL) {
-               return(NULL);
+       /* do the decompression, and cleanup */
+       if (bb_xread_char(archive_handle->src_fd) != 0x1f ||
+               bb_xread_char(archive_handle->src_fd) != 0x9d)
+       {
+               bb_error_msg_and_die("Invalid magic");
        }
-       
-       while (include_list[include_count] != NULL) {
-               int found = FALSE;
-               exclude_count = 0;
-               while (exclude_list[exclude_count] != NULL) {
-                       if (strcmp(include_list[include_count], exclude_list[exclude_count]) == 0) {
-                               found = TRUE;
-                               break;
-                       }
-                       exclude_count++;
-               }
 
-               if (! found) {
-                       new_include_list = realloc(new_include_list, sizeof(char *) * (include_count + 2));
-                       new_include_list[new_include_count] = include_list[include_count];
-                       new_include_count++;
-               } else {
-                       free(include_list[include_count]);
-               }
-               include_count++;
-       }
-       new_include_list[new_include_count] = NULL;
-       return(new_include_list);
+       archive_handle->src_fd = open_transformer(archive_handle->src_fd, uncompress);
+       archive_handle->offset = 0;
+       while (get_header_tar(archive_handle) == EXIT_SUCCESS);
+
+       /* Can only do one file at a time */
+       return(EXIT_FAILURE);
 }
+#else
+#define get_header_tar_Z       0
 #endif
 
-int tar_main(int argc, char **argv)
-{
-       enum untar_funct_e {
-               /* This is optional */
-               untar_unzip = 1,
-               /* Require one and only one of these */
-               untar_list = 2,
-               untar_create = 4,
-               untar_extract = 8
-       };
-
-       FILE *src_stream = NULL;
-       FILE *uncompressed_stream = NULL;
-       char **include_list = NULL;
-       char **exclude_list = NULL;
-       char *src_filename = NULL;
-       char *dst_prefix = NULL;
-       int opt;
-       unsigned short untar_funct = 0;
-       unsigned short untar_funct_required = 0;
-       unsigned short extract_function = 0;
-       int include_list_count = 0;
-#ifdef CONFIG_FEATURE_TAR_EXCLUDE
-       int exclude_list_count = 0;
+#define CTX_TEST                          (1 << 0)
+#define CTX_EXTRACT                       (1 << 1)
+#define TAR_OPT_BASEDIR                   (1 << 2)
+#define TAR_OPT_TARNAME                   (1 << 3)
+#define TAR_OPT_2STDOUT                   (1 << 4)
+#define TAR_OPT_P                         (1 << 5)
+#define TAR_OPT_VERBOSE                   (1 << 6)
+#define TAR_OPT_KEEP_OLD                  (1 << 7)
+
+#define TAR_OPT_AFTER_START               8
+
+#define CTX_CREATE                        (1 << (TAR_OPT_AFTER_START))
+#define TAR_OPT_DEREFERNCE                (1 << (TAR_OPT_AFTER_START + 1))
+#ifdef CONFIG_FEATURE_TAR_CREATE
+# define TAR_OPT_STR_CREATE               "ch"
+# define TAR_OPT_AFTER_CREATE             TAR_OPT_AFTER_START + 2
+#else
+# define TAR_OPT_STR_CREATE               ""
+# define TAR_OPT_AFTER_CREATE             TAR_OPT_AFTER_START
+#endif
+
+#define TAR_OPT_BZIP2                     (1 << (TAR_OPT_AFTER_CREATE))
+#ifdef CONFIG_FEATURE_TAR_BZIP2
+# define TAR_OPT_STR_BZIP2                "j"
+# define TAR_OPT_AFTER_BZIP2              TAR_OPT_AFTER_CREATE + 1
+#else
+# define TAR_OPT_STR_BZIP2                ""
+# define TAR_OPT_AFTER_BZIP2              TAR_OPT_AFTER_CREATE
 #endif
+
+#define TAR_OPT_LZMA                      (1 << (TAR_OPT_AFTER_BZIP2))
+#ifdef CONFIG_FEATURE_TAR_LZMA
+# define TAR_OPT_STR_LZMA                 "a"
+# define TAR_OPT_AFTER_LZMA               TAR_OPT_AFTER_BZIP2 + 1
+#else
+# define TAR_OPT_STR_LZMA                 ""
+# define TAR_OPT_AFTER_LZMA               TAR_OPT_AFTER_BZIP2
+#endif
+
+#define TAR_OPT_INCLUDE_FROM              (1 << (TAR_OPT_AFTER_LZMA))
+#define TAR_OPT_EXCLUDE_FROM              (1 << (TAR_OPT_AFTER_LZMA + 1))
+#ifdef CONFIG_FEATURE_TAR_FROM
+# define TAR_OPT_STR_FROM                 "T:X:"
+# define TAR_OPT_AFTER_FROM               TAR_OPT_AFTER_LZMA + 2
+#else
+# define TAR_OPT_STR_FROM                 ""
+# define TAR_OPT_AFTER_FROM               TAR_OPT_AFTER_LZMA
+#endif
+
+#define TAR_OPT_GZIP                      (1 << (TAR_OPT_AFTER_FROM))
 #ifdef CONFIG_FEATURE_TAR_GZIP
-       int gunzip_pid;
-       int gz_fd = 0;
+# define TAR_OPT_STR_GZIP                 "z"
+# define TAR_OPT_AFTER_GZIP               TAR_OPT_AFTER_FROM + 1
+#else
+# define TAR_OPT_STR_GZIP                 ""
+# define TAR_OPT_AFTER_GZIP               TAR_OPT_AFTER_FROM
 #endif
 
-       if (argc < 2) {
-               show_usage();
-       }
+#define TAR_OPT_UNCOMPRESS                (1 << (TAR_OPT_AFTER_GZIP))
+#ifdef CONFIG_FEATURE_TAR_COMPRESS
+# define TAR_OPT_STR_COMPRESS             "Z"
+# define TAR_OPT_AFTER_COMPRESS           TAR_OPT_AFTER_GZIP + 1
+#else
+# define TAR_OPT_STR_COMPRESS             ""
+# define TAR_OPT_AFTER_COMPRESS           TAR_OPT_AFTER_GZIP
+#endif
+
+#define TAR_OPT_NOPRESERVE_OWN            (1 << (TAR_OPT_AFTER_COMPRESS))
+#define TAR_OPT_NOPRESERVE_PERM           (1 << (TAR_OPT_AFTER_COMPRESS + 1))
+#define TAR_OPT_STR_NOPRESERVE            "\203\213"
+#define TAR_OPT_AFTER_NOPRESERVE          TAR_OPT_AFTER_COMPRESS + 2
+
+static const char tar_options[]="txC:f:Opvk" \
+       TAR_OPT_STR_CREATE \
+       TAR_OPT_STR_BZIP2 \
+       TAR_OPT_STR_LZMA \
+       TAR_OPT_STR_FROM \
+       TAR_OPT_STR_GZIP \
+       TAR_OPT_STR_COMPRESS \
+       TAR_OPT_STR_NOPRESERVE;
+
+#ifdef CONFIG_FEATURE_TAR_LONG_OPTIONS
+static const struct option tar_long_options[] = {
+       { "list",                               0,      NULL,   't' },
+       { "extract",                    0,      NULL,   'x' },
+       { "directory",                  1,      NULL,   'C' },
+       { "file",                               1,      NULL,   'f' },
+       { "to-stdout",                  0,      NULL,   'O' },
+       { "same-permissions",   0,      NULL,   'p' },
+       { "verbose",                    0,      NULL,   'v' },
+       { "keep-old",                   0,      NULL,   'k' },
+       { "no-same-owner",              0,      NULL,   '\203' },
+       { "no-same-permissions",0,      NULL,   '\213' },
+# ifdef CONFIG_FEATURE_TAR_CREATE
+       { "create",                             0,      NULL,   'c' },
+       { "dereference",                0,      NULL,   'h' },
+# endif
+# ifdef CONFIG_FEATURE_TAR_BZIP2
+       { "bzip2",                              0,      NULL,   'j' },
+# endif
+# ifdef CONFIG_FEATURE_TAR_LZMA
+       { "lzma",                               0,      NULL,   'a' },
+# endif
+# ifdef CONFIG_FEATURE_TAR_FROM
+       { "files-from",                 1,      NULL,   'T' },
+       { "exclude-from",               1,      NULL,   'X' },
+       { "exclude",                    1,      NULL,   '\n' },
+# endif
+# ifdef CONFIG_FEATURE_TAR_GZIP
+       { "gzip",                               0,      NULL,   'z' },
+# endif
+# ifdef CONFIG_FEATURE_TAR_COMPRESS
+       { "compress",                   0,      NULL,   'Z' },
+# endif
+       { 0,                                    0, 0, 0 }
+};
+#else
+#define tar_long_options       0
+#endif
+
+int tar_main(int argc, char **argv)
+{
+       char (*get_header_ptr)(archive_handle_t *) = get_header_tar;
+       archive_handle_t *tar_handle;
+       char *base_dir = NULL;
+       const char *tar_filename = "-";
+       unsigned long opt;
+       llist_t *excludes = NULL;
+
+       /* Initialise default values */
+       tar_handle = init_handle();
+       tar_handle->flags = ARCHIVE_CREATE_LEADING_DIRS | ARCHIVE_PRESERVE_DATE | ARCHIVE_EXTRACT_UNCONDITIONAL;
 
        /* Prepend '-' to the first argument if required */
-       if (argv[1][0] != '-') {
-               char *tmp = xmalloc(strlen(argv[1]) + 2);
-               tmp[0] = '-';
-               strcpy(tmp + 1, argv[1]);
-               argv[1] = tmp;
+       bb_opt_complementally = ENABLE_FEATURE_TAR_CREATE ?
+               "--:X::T::\n::c:t:x:?:c--tx:t--cx:x--ct" :
+               "--:X::T::\n::t:x:?:t--x:x--t";
+       if (ENABLE_FEATURE_TAR_LONG_OPTIONS)
+               bb_applet_long_options = tar_long_options;
+       opt = bb_getopt_ulflags(argc, argv, tar_options,
+                               &base_dir,      /* Change to dir <optarg> */
+                               &tar_filename /* archive filename */
+#ifdef CONFIG_FEATURE_TAR_FROM
+                               , &(tar_handle->accept),
+                               &(tar_handle->reject),
+                               &excludes
+#endif
+                               );
+
+       if (opt & CTX_TEST) {
+               if ((tar_handle->action_header == header_list) ||
+                       (tar_handle->action_header == header_verbose_list))
+               {
+                               tar_handle->action_header = header_verbose_list;
+               } else tar_handle->action_header = header_list;
+       }
+       if((opt & CTX_EXTRACT) && tar_handle->action_data != data_extract_to_stdout)
+               tar_handle->action_data = data_extract_all;
+
+       if (opt & TAR_OPT_2STDOUT)
+               tar_handle->action_data = data_extract_to_stdout;
+
+       if (opt & TAR_OPT_VERBOSE) {
+               if ((tar_handle->action_header == header_list) ||
+                       (tar_handle->action_header == header_verbose_list))
+               {
+                       tar_handle->action_header = header_verbose_list;
+               } else
+                       tar_handle->action_header = header_list;
        }
+       if (opt & TAR_OPT_KEEP_OLD)
+               tar_handle->flags &= ~ARCHIVE_EXTRACT_UNCONDITIONAL;
 
-       while ((opt = getopt(argc, argv, "ctxT:X:C:f:Opvz")) != -1) {
-               switch (opt) {
+       if (opt & TAR_OPT_NOPRESERVE_OWN)
+               tar_handle->flags |= ARCHIVE_NOPRESERVE_OWN;
 
-               /* One and only one of these is required */
-               case 'c':
-                       untar_funct_required |= untar_create;
-                       break;
-               case 't':
-                       untar_funct_required |= untar_list;
-                       extract_function |= extract_list |extract_unconditional;
-                       break;
-               case 'x':
-                       untar_funct_required |= untar_extract;
-                       extract_function |= (extract_all_to_fs | extract_unconditional | extract_create_leading_dirs);
-                       break;
+       if (opt & TAR_OPT_NOPRESERVE_PERM)
+               tar_handle->flags |= ARCHIVE_NOPRESERVE_PERM;
 
-               /* These are optional */
-               /* Exclude or Include files listed in <filename>*/
-#ifdef CONFIG_FEATURE_TAR_EXCLUDE
-               case 'X':
-                       append_file_list_to_list(optarg, &exclude_list, &exclude_list_count);
-                       break;
-#endif
-               case 'T':
-                       // by default a list is an include list
-                       append_file_list_to_list(optarg, &include_list, &include_list_count);
-                       break;
+       if (ENABLE_FEATURE_TAR_GZIP && (opt & TAR_OPT_GZIP))
+               get_header_ptr = get_header_tar_gz;
 
-               case 'C':       // Change to dir <optarg>
-                       /* Make sure dst_prefix ends in a '/' */
-                       dst_prefix = concat_path_file(optarg, "/");
-                       break;
-               case 'f':       // archive filename
-                       if (strcmp(optarg, "-") == 0) {
-                               src_filename = NULL;
-                       } else {
-                               src_filename = xstrdup(optarg);
-                       }
-                       break;
-               case 'O':
-                       extract_function |= extract_to_stdout;
-                       break;
-               case 'p':
-                       break;
-               case 'v':
-                       if (extract_function & extract_list) {
-                               extract_function |= extract_verbose_list;
-                       }
-                       extract_function |= extract_list;
-                       break;
-#ifdef CONFIG_FEATURE_TAR_GZIP
-               case 'z':
-                       untar_funct |= untar_unzip;
-                       break;
-#endif
-               default:
-                       show_usage();
+       if (ENABLE_FEATURE_TAR_BZIP2 && (opt & TAR_OPT_BZIP2))
+               get_header_ptr = get_header_tar_bz2;
+
+       if (ENABLE_FEATURE_TAR_LZMA && (opt & TAR_OPT_LZMA))
+               get_header_ptr = get_header_tar_lzma;
+
+       if (ENABLE_FEATURE_TAR_COMPRESS && (opt & TAR_OPT_UNCOMPRESS))
+               get_header_ptr = get_header_tar_Z;
+
+       if (ENABLE_FEATURE_TAR_FROM) {
+               tar_handle->reject = append_file_list_to_list(tar_handle->reject);
+               /* Append excludes to reject */
+               while (excludes) {
+                       llist_t *temp = excludes->link;
+                       excludes->link = tar_handle->reject;
+                       tar_handle->reject = excludes;
+                       excludes = temp;
                }
+               tar_handle->accept = append_file_list_to_list(tar_handle->accept);
        }
 
-       /* Make sure the valid arguments were passed */
-       if (untar_funct_required == 0) {
-               error_msg_and_die("You must specify one of the `-ctx' options");
-       }
-       if ((untar_funct_required != untar_create) && 
-                       (untar_funct_required != untar_extract) &&
-                       (untar_funct_required != untar_list)) {
-               error_msg_and_die("You may not specify more than one `ctx' option.");
+       /* Check if we are reading from stdin */
+       if (argv[optind] && *argv[optind] == '-') {
+               /* Default is to read from stdin, so just skip to next arg */
+               optind++;
        }
-       untar_funct |= untar_funct_required;
 
        /* Setup an array of filenames to work with */
+       /* TODO: This is the same as in ar, separate function ? */
        while (optind < argc) {
-               append_file_to_list(argv[optind], &include_list, &include_list_count);
+               char *filename_ptr = last_char_is(argv[optind], '/');
+               if (filename_ptr > argv[optind])
+                       *filename_ptr = '\0';
+
+               tar_handle->accept = llist_add_to(tar_handle->accept, argv[optind]);
                optind++;
        }
-       if (extract_function & (extract_list | extract_all_to_fs)) {
-               if (dst_prefix == NULL) {
-                       dst_prefix = xstrdup("./");
+
+       if ((tar_handle->accept) || (tar_handle->reject))
+               tar_handle->filter = filter_accept_reject_list;
+
+       /* Open the tar file */
+       {
+               FILE *tar_stream;
+               int flags;
+
+               if (ENABLE_FEATURE_TAR_CREATE && (opt & CTX_CREATE)) {
+                       /* Make sure there is at least one file to tar up.  */
+                       if (tar_handle->accept == NULL)
+                               bb_error_msg_and_die("empty archive");
+
+                       tar_stream = stdout;
+                       flags = O_WRONLY | O_CREAT | O_EXCL;
+                       unlink(tar_filename);
+               } else {
+                       tar_stream = stdin;
+                       flags = O_RDONLY;
                }
 
-               /* Setup the source of the tar data */
-               if (src_filename != NULL) {
-                       src_stream = xfopen(src_filename, "r");
+               if ((tar_filename[0] == '-') && (tar_filename[1] == '\0')) {
+                       tar_handle->src_fd = fileno(tar_stream);
+                       tar_handle->seek = seek_by_char;
                } else {
-                       src_stream = stdin;
+                       tar_handle->src_fd = bb_xopen(tar_filename, flags);
                }
-#ifdef CONFIG_FEATURE_TAR_GZIP
-               /* Get a binary tree of all the tar file headers */
-               if (untar_funct & untar_unzip) {
-                       uncompressed_stream = gz_open(src_stream, &gunzip_pid);
-               } else
-#endif // CONFIG_FEATURE_TAR_GZIP
-                       uncompressed_stream = src_stream;
-               
-               /* extract or list archive */
-               unarchive(uncompressed_stream, stdout, &get_header_tar, extract_function, dst_prefix, include_list, exclude_list);
-               fclose(uncompressed_stream);
        }
-#ifdef CONFIG_FEATURE_TAR_CREATE
+
+       if (base_dir)
+               bb_xchdir(base_dir);
+
        /* create an archive */
-       else if (untar_funct & untar_create) {
+       if (ENABLE_FEATURE_TAR_CREATE && (opt & CTX_CREATE)) {
                int verboseFlag = FALSE;
+               int zipMode = 0;
 
-#ifdef CONFIG_FEATURE_TAR_GZIP
-               if (untar_funct & untar_unzip) {
-                       error_msg_and_die("Creation of compressed tarfile not internally support by tar, pipe to busybox gunzip");
-               }
-#endif // CONFIG_FEATURE_TAR_GZIP
-               if (extract_function & extract_verbose_list) {
+               if (ENABLE_FEATURE_TAR_GZIP && get_header_ptr == get_header_tar_gz)
+                       zipMode = 1;
+               if (ENABLE_FEATURE_TAR_BZIP2 && get_header_ptr == get_header_tar_bz2)
+                       zipMode = 2;
+
+               if ((tar_handle->action_header == header_list) ||
+                               (tar_handle->action_header == header_verbose_list))
+               {
                        verboseFlag = TRUE;
                }
-               writeTarFile(src_filename, verboseFlag, include_list, exclude_list);
+               writeTarFile(tar_handle->src_fd, verboseFlag, opt & TAR_OPT_DEREFERNCE, tar_handle->accept,
+                       tar_handle->reject, zipMode);
+       } else {
+               while (get_header_ptr(tar_handle) == EXIT_SUCCESS);
+
+               /* Check that every file that should have been extracted was */
+               while (tar_handle->accept) {
+                       if (!find_list_entry(tar_handle->reject, tar_handle->accept->data)
+                               && !find_list_entry(tar_handle->passed, tar_handle->accept->data))
+                       {
+                               bb_error_msg_and_die("%s: Not found in archive", tar_handle->accept->data);
+                       }
+                       tar_handle->accept = tar_handle->accept->link;
+               }
        }
-#endif // CONFIG_FEATURE_TAR_CREATE
 
-       /* Cleanups */
-#ifdef CONFIG_FEATURE_TAR_GZIP
-       if (untar_funct & untar_unzip) {
-               fclose(src_stream);
-               close(gz_fd);
-               gz_close(gunzip_pid);
-       }
-#endif // CONFIG_FEATURE_TAR_GZIP
-#ifdef CONFIG_FEATURE_CLEAN_UP
-       if (src_filename) {
-               free(src_filename);
-       }
-#endif
+       if (ENABLE_FEATURE_CLEAN_UP && tar_handle->src_fd != STDIN_FILENO)
+               close(tar_handle->src_fd);
+
        return(EXIT_SUCCESS);
 }