Fix .deb unpack and experimental TAR support

The previous ar.c fialed to recognise one record in .debs

Experimental tar support, this is currently *very* cheap because ar and
tar are functionally similar. It  will need lots of testing so by
default tar support code is defined out.
To test uncomment the line "#define BB_AR_EXPERIMENTAL_UNTAR"

2 files changed, 356 insertions, 174 deletions

diff --git a/ar.c b/ar.c
index 61ce83029..d82763df1 100644
--- a/ar.c
+++ b/ar.c
@@ -51,6 +51,30 @@
 //#define bb_need_io_error
 //#include "messages.c"
 
+//#define BB_AR_EXPERIMENTAL_UNTAR
+
+#if defined BB_AR_EXPERIMENTAL_UNTAR
+typedef struct rawTarHeader {
+        char name[100];               /*   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[100];           /* 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 */
+} rawTarHeader_t;
+#endif
+
 typedef struct rawArHeader {    /* Byte Offset */
         char name[16];          /*  0-15 */
         char date[12];          /* 16-27 */
@@ -71,92 +95,136 @@ typedef struct headerL {
         struct headerL *next;
 } headerL_t;
 
+#if defined BB_AR_EXPERIMENTAL_UNTAR
 /*
- * identify Ar header (magic) and set srcFd to first header entry 
+ * identify Tar header (magic field) and reset srcFd to entry position
  */
-static int checkArMagic(int srcFd)
+static int checkTarMagic(int srcFd)
 {
-        char arMagic[8];
-        if (fullRead(srcFd, arMagic, 8) != 8)
-                return (FALSE);
-        
-        if (strncmp(arMagic,"!<arch>",7) != 0)
+        off_t headerStart;
+        char magic[6];
+
+        headerStart = lseek(srcFd, 0, SEEK_CUR);
+        lseek(srcFd, (off_t) 257, SEEK_CUR);
+        fullRead(srcFd, magic, 6);
+        lseek(srcFd, headerStart, SEEK_SET);
+        if (strncmp(magic, "ustar", 5)!=0)
+                return(FALSE);
+        return(TRUE);
+}
+
+
+static int readTarHeader(int srcFd, headerL_t *current)
+{
+        rawTarHeader_t rawTarHeader;
+        unsigned char *temp = (unsigned char *) &rawTarHeader;
+        long sum = 0;
+        int i;
+        off_t initialOffset;
+
+        initialOffset = lseek(srcFd, 0, SEEK_CUR);
+        if (fullRead(srcFd, (char *) &rawTarHeader, 512) != 512) {
+                lseek(srcFd, initialOffset, SEEK_SET);
                 return(FALSE);
-        return(TRUE);
+        }
+        for (i =  0; i < 148 ; i++)
+        sum += temp[i];
+        sum += ' ' * 8;
+        for (i =  156; i < 512 ; i++)
+                sum += temp[i];
+        if (sum!= strtol(rawTarHeader.chksum, NULL, 8))
+                return(FALSE);
+        sscanf(rawTarHeader.name, "%s", current->name);
+        current->size = strtol(rawTarHeader.size, NULL, 8);
+        current->uid = strtol(rawTarHeader.uid, NULL, 8);
+        current->gid = strtol(rawTarHeader.gid, NULL, 8);
+        current->mode = strtol(rawTarHeader.mode, NULL, 8);
+        current->mtime = strtol(rawTarHeader.mtime, NULL, 8);
+        current->offset = lseek(srcFd, 0 , SEEK_CUR);
+
+        current->next = (headerL_t *) xmalloc(sizeof(headerL_t));
+        current = current->next;
+        return(TRUE);
 }
+#endif
 
 /*
- * read, convert and check the raw ar header
- * srcFd should be pointing to the start of header prior to entry
- * srcFd will be pointing at the start of data after successful exit
- * if returns FALSE srcFd is reset to initial position
+ * identify Ar header (magic) and reset srcFd to entry position
  */
-static int readRawArHeader(int srcFd, headerL_t *header)
+static int checkArMagic(int srcFd)
 {
-        rawArHeader_t rawArHeader;
-        off_t   initialOffset;
-        size_t nameLength;
-        
-        initialOffset = lseek(srcFd, 0, SEEK_CUR);
-        if (fullRead(srcFd, (char *) &rawArHeader, 60) != 60) {
-                lseek(srcFd, initialOffset, SEEK_SET);
-                return(FALSE);
-        }
-        if ((rawArHeader.fmag[0]!='`') || (rawArHeader.fmag[1]!='\n')) {
-                lseek(srcFd, initialOffset, SEEK_SET);
-                return(FALSE);
-        }
+        off_t headerStart;
+        char arMagic[8];
 
-        strncpy(header->name, rawArHeader.name, 16);
-        nameLength=strcspn(header->name, " \\");
-        header->name[nameLength]='\0';
-        parse_mode(rawArHeader.mode, &header->mode);
-        header->mtime = atoi(rawArHeader.date);
-        header->uid = atoi(rawArHeader.uid);
-        header->gid = atoi(rawArHeader.gid);
-        header->size = (size_t) atoi(rawArHeader.size);
-        header->offset = initialOffset + (off_t) 60;
-        return(TRUE); 
+        headerStart = lseek(srcFd, 0, SEEK_CUR);
+        if (fullRead(srcFd, arMagic, 8) != 8) {
+                printf("fatal error/n");
+                return (FALSE);
+        }
+        lseek(srcFd, headerStart, SEEK_SET);
+
+        if (strncmp(arMagic,"!<arch>",7) != 0)
+                return(FALSE);
+        return(TRUE);
 }
 
 /*
  * get, check and correct the converted header
  */ 
-static int readArEntry(int srcFd, headerL_t *newEntry)
+static int readArEntry(int srcFd, headerL_t *entry)
 {
         size_t nameLength;
+        rawArHeader_t rawArHeader;
+        off_t   initialOffset;
 
-        if(readRawArHeader(srcFd, newEntry)==FALSE)
-                return(FALSE);
-        
-        nameLength = strcspn(newEntry->name, "/");
+        initialOffset = lseek(srcFd, 0, SEEK_CUR);
+        if (fullRead(srcFd, (char *) &rawArHeader, 60) != 60) {
+                lseek(srcFd, initialOffset, SEEK_SET);
+                return(FALSE);
+        }
+        if ((rawArHeader.fmag[0]!='`') || (rawArHeader.fmag[1]!='\n')) {
+                lseek(srcFd, initialOffset, SEEK_SET);
+                return(FALSE);
+        }
+
+        strncpy(entry->name, rawArHeader.name, 16);
+        nameLength=strcspn(entry->name, " \\");
+        entry->name[nameLength]='\0';
+        parse_mode(rawArHeader.mode, &entry->mode);
+        entry->mtime = atoi(rawArHeader.date);
+        entry->uid = atoi(rawArHeader.uid);
+        entry->gid = atoi(rawArHeader.gid);
+        entry->size = (size_t) atoi(rawArHeader.size);
+        entry->offset = initialOffset + (off_t) 60;
+
+        nameLength = strcspn(entry->name, "/");
         
         /* handle GNU style short filenames, strip trailing '/' */
         if (nameLength > 0)
-                newEntry->name[nameLength]='\0';
+                entry->name[nameLength]='\0';
         
         /* handle GNU style long filenames */ 
         if (nameLength == 0) {
                 /* escape from recursive call */
-                if (newEntry->name[1]=='0') 
+                if (entry->name[1]=='0') 
                         return(TRUE);
 
                 /* the data section contains the real filename */
-                if (newEntry->name[1]=='/') {
+                if (entry->name[1]=='/') {
                         char tempName[MAX_NAME_LENGTH];
 
-                        if (newEntry->size > MAX_NAME_LENGTH)
-                                newEntry->size = MAX_NAME_LENGTH;
-                        fullRead(srcFd, tempName, newEntry->size);
-                        tempName[newEntry->size-3]='\0';
+                        if (entry->size > MAX_NAME_LENGTH)
+                                entry->size = MAX_NAME_LENGTH;
+                        fullRead(srcFd, tempName, entry->size);
+                        tempName[entry->size-3]='\0';
                         
                         /* read the second header for this entry */
                         /* be carefull, this is recursive */
-                        if (readArEntry(srcFd, newEntry)==FALSE)
+                        if (readArEntry(srcFd, entry)==FALSE)
                                 return(FALSE);
                 
-                        if ((newEntry->name[0]='/') && (newEntry->name[1]='0'))
-                                strcpy(newEntry->name, tempName);
+                        if ((entry->name[0]='/') && (entry->name[1]='0'))
+                                strcpy(entry->name, tempName);
                         else {
                                 errorMsg("Invalid long filename\n");
                                 return(FALSE);
@@ -171,17 +239,54 @@ static int readArEntry(int srcFd, headerL_t *newEntry)
  */
 static headerL_t *getHeaders(int srcFd, headerL_t *head, int funct)
 {
+#if defined BB_AR_EXPERIMENTAL_UNTAR
+        int tar=FALSE;
+#endif
+        int ar=FALSE;
         headerL_t *list;
+        off_t initialOffset;
+
         list = (headerL_t *) malloc(sizeof(headerL_t));
+        initialOffset=lseek(srcFd, 0, SEEK_CUR);
+        if (checkArMagic(srcFd)==TRUE) 
+                ar=TRUE;
+
+#if defined BB_AR_EXPERIMENTAL_UNTAR
+        if (checkTarMagic(srcFd)==TRUE)
+                tar=TRUE;
 
-        if (checkArMagic(srcFd)==TRUE) {
-                while(readArEntry(srcFd, list) == TRUE) {
+        if (tar==TRUE) {
+                while(readTarHeader(srcFd, list)==TRUE) {
+                        off_t tarOffset;
+                        list->next = (headerL_t *) malloc(sizeof(headerL_t));
+                        *list->next = *head;
+                        *head = *list;
+
+                        /* recursive check for sub-archives */
+                        if ((funct & RECURSIVE) == RECURSIVE)
+                                head = getHeaders(srcFd, head, funct);
+                        tarOffset = (off_t) head->size/512;
+                        if ( head->size % 512 > 0)
+                                tarOffset++;
+                        tarOffset=tarOffset*512;
+                        lseek(srcFd, head->offset + tarOffset, SEEK_SET);
+                }
+        }
+#endif
+
+        if (ar==TRUE) {
+                lseek(srcFd, 8, SEEK_CUR); 
+                while(1) {
+                        if (readArEntry(srcFd, list) == FALSE) {
+                                lseek(srcFd, ++initialOffset, SEEK_CUR); 
+                                if (readArEntry(srcFd, list) == FALSE)
+                                        return(head);
+                        }
                         list->next = (headerL_t *) malloc(sizeof(headerL_t));
                         *list->next = *head;
                         *head = *list;
-                
                         /* recursive check for sub-archives */
-                        if ( funct & RECURSIVE ) 
+                        if (funct & RECURSIVE)  
                                 head = getHeaders(srcFd, head, funct);
                         lseek(srcFd, head->offset + head->size, SEEK_SET);
                 }
@@ -202,27 +307,6 @@ static headerL_t *findEntry(headerL_t *head, const char *filename)
         return(NULL);
 }
 
-/*
- * populate linked list with all ar file entries and offset 
- */
-static int displayEntry(headerL_t *head, int funct)
-{
-        if ( funct & VERBOSE ) {
-                printf("%s %d/%d %8d %s ", modeString(head->mode), head->uid, head->gid, head->size, timeString(head->mtime));
-        }
-        printf("%s\n", head->name);
-        head = head->next;
-        return(TRUE);
-}
-
-static int extractAr(int srcFd, int dstFd, headerL_t *file)
-{
-        lseek(srcFd, file->offset, SEEK_SET);
-        if (copySubFile(srcFd, dstFd, (size_t) file->size) == TRUE)
-                return(TRUE);   
-        return(FALSE);
-}
-
 extern int ar_main(int argc, char **argv)
 {
         int funct = 0, opt=0;
@@ -270,7 +354,6 @@ extern int ar_main(int argc, char **argv)
         extractList = (headerL_t *) malloc(sizeof(headerL_t));  
 
         header = getHeaders(srcFd, header, funct);
-        
         /* find files to extract or display */
         if (optind<argc) {
                 /* only handle specified files */
@@ -283,20 +366,28 @@ extern int ar_main(int argc, char **argv)
                         optind++;
                 }       
         }
-        else 
-                /* extract everything */
+        else  
                 extractList = header;
         
-       while(extractList->next != NULL) {       
-                if ( funct & EXT_TO_FILE ) {
+        while(extractList->next != NULL) {      
+                if (funct & EXT_TO_FILE) {
+                        if (isDirectory(extractList->name, TRUE, NULL)==FALSE)
+                                createPath(extractList->name, 0666);
                         dstFd = open(extractList->name, O_WRONLY | O_CREAT, extractList->mode);
-                        
-                        extractAr(srcFd, dstFd, extractList);
+                        lseek(srcFd, extractList->offset, SEEK_SET);
+                        copySubFile(srcFd, dstFd, (size_t) extractList->size);
+                }
+                if (funct & EXT_TO_STDOUT) {    
+                        lseek(srcFd, extractList->offset, SEEK_SET);
+                        copySubFile(srcFd, fileno(stdout), (size_t) extractList->size);
+                }
+                if ( (funct & DISPLAY) || (funct & VERBOSE)) {
+                        if (funct & VERBOSE)
+                                printf("%s %d/%d %8d %s ", modeString(extractList->mode), 
+                                        extractList->uid, extractList->gid,
+                                        extractList->size, timeString(extractList->mtime));
+                        printf("%s\n", extractList->name);
                 }
-                if ( funct & EXT_TO_STDOUT )    
-                        extractAr(srcFd, fileno(stdout), extractList);  
-                if ( (funct & DISPLAY) || (funct & VERBOSE))
-                        displayEntry(extractList, funct);
                 extractList=extractList->next;
         }
         return (TRUE);
diff --git a/archival/ar.c b/archival/ar.c
index 61ce83029..d82763df1 100644
--- a/archival/ar.c
+++ b/archival/ar.c
@@ -51,6 +51,30 @@
 //#define bb_need_io_error
 //#include "messages.c"
 
+//#define BB_AR_EXPERIMENTAL_UNTAR
+
+#if defined BB_AR_EXPERIMENTAL_UNTAR
+typedef struct rawTarHeader {
+        char name[100];               /*   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[100];           /* 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 */
+} rawTarHeader_t;
+#endif
+
 typedef struct rawArHeader {    /* Byte Offset */
         char name[16];          /*  0-15 */
         char date[12];          /* 16-27 */
@@ -71,92 +95,136 @@ typedef struct headerL {
         struct headerL *next;
 } headerL_t;
 
+#if defined BB_AR_EXPERIMENTAL_UNTAR
 /*
- * identify Ar header (magic) and set srcFd to first header entry 
+ * identify Tar header (magic field) and reset srcFd to entry position
  */
-static int checkArMagic(int srcFd)
+static int checkTarMagic(int srcFd)
 {
-        char arMagic[8];
-        if (fullRead(srcFd, arMagic, 8) != 8)
-                return (FALSE);
-        
-        if (strncmp(arMagic,"!<arch>",7) != 0)
+        off_t headerStart;
+        char magic[6];
+
+        headerStart = lseek(srcFd, 0, SEEK_CUR);
+        lseek(srcFd, (off_t) 257, SEEK_CUR);
+        fullRead(srcFd, magic, 6);
+        lseek(srcFd, headerStart, SEEK_SET);
+        if (strncmp(magic, "ustar", 5)!=0)
+                return(FALSE);
+        return(TRUE);
+}
+
+
+static int readTarHeader(int srcFd, headerL_t *current)
+{
+        rawTarHeader_t rawTarHeader;
+        unsigned char *temp = (unsigned char *) &rawTarHeader;
+        long sum = 0;
+        int i;
+        off_t initialOffset;
+
+        initialOffset = lseek(srcFd, 0, SEEK_CUR);
+        if (fullRead(srcFd, (char *) &rawTarHeader, 512) != 512) {
+                lseek(srcFd, initialOffset, SEEK_SET);
                 return(FALSE);
-        return(TRUE);
+        }
+        for (i =  0; i < 148 ; i++)
+        sum += temp[i];
+        sum += ' ' * 8;
+        for (i =  156; i < 512 ; i++)
+                sum += temp[i];
+        if (sum!= strtol(rawTarHeader.chksum, NULL, 8))
+                return(FALSE);
+        sscanf(rawTarHeader.name, "%s", current->name);
+        current->size = strtol(rawTarHeader.size, NULL, 8);
+        current->uid = strtol(rawTarHeader.uid, NULL, 8);
+        current->gid = strtol(rawTarHeader.gid, NULL, 8);
+        current->mode = strtol(rawTarHeader.mode, NULL, 8);
+        current->mtime = strtol(rawTarHeader.mtime, NULL, 8);
+        current->offset = lseek(srcFd, 0 , SEEK_CUR);
+
+        current->next = (headerL_t *) xmalloc(sizeof(headerL_t));
+        current = current->next;
+        return(TRUE);
 }
+#endif
 
 /*
- * read, convert and check the raw ar header
- * srcFd should be pointing to the start of header prior to entry
- * srcFd will be pointing at the start of data after successful exit
- * if returns FALSE srcFd is reset to initial position
+ * identify Ar header (magic) and reset srcFd to entry position
  */
-static int readRawArHeader(int srcFd, headerL_t *header)
+static int checkArMagic(int srcFd)
 {
-        rawArHeader_t rawArHeader;
-        off_t   initialOffset;
-        size_t nameLength;
-        
-        initialOffset = lseek(srcFd, 0, SEEK_CUR);
-        if (fullRead(srcFd, (char *) &rawArHeader, 60) != 60) {
-                lseek(srcFd, initialOffset, SEEK_SET);
-                return(FALSE);
-        }
-        if ((rawArHeader.fmag[0]!='`') || (rawArHeader.fmag[1]!='\n')) {
-                lseek(srcFd, initialOffset, SEEK_SET);
-                return(FALSE);
-        }
+        off_t headerStart;
+        char arMagic[8];
 
-        strncpy(header->name, rawArHeader.name, 16);
-        nameLength=strcspn(header->name, " \\");
-        header->name[nameLength]='\0';
-        parse_mode(rawArHeader.mode, &header->mode);
-        header->mtime = atoi(rawArHeader.date);
-        header->uid = atoi(rawArHeader.uid);
-        header->gid = atoi(rawArHeader.gid);
-        header->size = (size_t) atoi(rawArHeader.size);
-        header->offset = initialOffset + (off_t) 60;
-        return(TRUE); 
+        headerStart = lseek(srcFd, 0, SEEK_CUR);
+        if (fullRead(srcFd, arMagic, 8) != 8) {
+                printf("fatal error/n");
+                return (FALSE);
+        }
+        lseek(srcFd, headerStart, SEEK_SET);
+
+        if (strncmp(arMagic,"!<arch>",7) != 0)
+                return(FALSE);
+        return(TRUE);
 }
 
 /*
  * get, check and correct the converted header
  */ 
-static int readArEntry(int srcFd, headerL_t *newEntry)
+static int readArEntry(int srcFd, headerL_t *entry)
 {
         size_t nameLength;
+        rawArHeader_t rawArHeader;
+        off_t   initialOffset;
 
-        if(readRawArHeader(srcFd, newEntry)==FALSE)
-                return(FALSE);
-        
-        nameLength = strcspn(newEntry->name, "/");
+        initialOffset = lseek(srcFd, 0, SEEK_CUR);
+        if (fullRead(srcFd, (char *) &rawArHeader, 60) != 60) {
+                lseek(srcFd, initialOffset, SEEK_SET);
+                return(FALSE);
+        }
+        if ((rawArHeader.fmag[0]!='`') || (rawArHeader.fmag[1]!='\n')) {
+                lseek(srcFd, initialOffset, SEEK_SET);
+                return(FALSE);
+        }
+
+        strncpy(entry->name, rawArHeader.name, 16);
+        nameLength=strcspn(entry->name, " \\");
+        entry->name[nameLength]='\0';
+        parse_mode(rawArHeader.mode, &entry->mode);
+        entry->mtime = atoi(rawArHeader.date);
+        entry->uid = atoi(rawArHeader.uid);
+        entry->gid = atoi(rawArHeader.gid);
+        entry->size = (size_t) atoi(rawArHeader.size);
+        entry->offset = initialOffset + (off_t) 60;
+
+        nameLength = strcspn(entry->name, "/");
         
         /* handle GNU style short filenames, strip trailing '/' */
         if (nameLength > 0)
-                newEntry->name[nameLength]='\0';
+                entry->name[nameLength]='\0';
         
         /* handle GNU style long filenames */ 
         if (nameLength == 0) {
                 /* escape from recursive call */
-                if (newEntry->name[1]=='0') 
+                if (entry->name[1]=='0') 
                         return(TRUE);
 
                 /* the data section contains the real filename */
-                if (newEntry->name[1]=='/') {
+                if (entry->name[1]=='/') {
                         char tempName[MAX_NAME_LENGTH];
 
-                        if (newEntry->size > MAX_NAME_LENGTH)
-                                newEntry->size = MAX_NAME_LENGTH;
-                        fullRead(srcFd, tempName, newEntry->size);
-                        tempName[newEntry->size-3]='\0';
+                        if (entry->size > MAX_NAME_LENGTH)
+                                entry->size = MAX_NAME_LENGTH;
+                        fullRead(srcFd, tempName, entry->size);
+                        tempName[entry->size-3]='\0';
                         
                         /* read the second header for this entry */
                         /* be carefull, this is recursive */
-                        if (readArEntry(srcFd, newEntry)==FALSE)
+                        if (readArEntry(srcFd, entry)==FALSE)
                                 return(FALSE);
                 
-                        if ((newEntry->name[0]='/') && (newEntry->name[1]='0'))
-                                strcpy(newEntry->name, tempName);
+                        if ((entry->name[0]='/') && (entry->name[1]='0'))
+                                strcpy(entry->name, tempName);
                         else {
                                 errorMsg("Invalid long filename\n");
                                 return(FALSE);
@@ -171,17 +239,54 @@ static int readArEntry(int srcFd, headerL_t *newEntry)
  */
 static headerL_t *getHeaders(int srcFd, headerL_t *head, int funct)
 {
+#if defined BB_AR_EXPERIMENTAL_UNTAR
+        int tar=FALSE;
+#endif
+        int ar=FALSE;
         headerL_t *list;
+        off_t initialOffset;
+
         list = (headerL_t *) malloc(sizeof(headerL_t));
+        initialOffset=lseek(srcFd, 0, SEEK_CUR);
+        if (checkArMagic(srcFd)==TRUE) 
+                ar=TRUE;
+
+#if defined BB_AR_EXPERIMENTAL_UNTAR
+        if (checkTarMagic(srcFd)==TRUE)
+                tar=TRUE;
 
-        if (checkArMagic(srcFd)==TRUE) {
-                while(readArEntry(srcFd, list) == TRUE) {
+        if (tar==TRUE) {
+                while(readTarHeader(srcFd, list)==TRUE) {
+                        off_t tarOffset;
+                        list->next = (headerL_t *) malloc(sizeof(headerL_t));
+                        *list->next = *head;
+                        *head = *list;
+
+                        /* recursive check for sub-archives */
+                        if ((funct & RECURSIVE) == RECURSIVE)
+                                head = getHeaders(srcFd, head, funct);
+                        tarOffset = (off_t) head->size/512;
+                        if ( head->size % 512 > 0)
+                                tarOffset++;
+                        tarOffset=tarOffset*512;
+                        lseek(srcFd, head->offset + tarOffset, SEEK_SET);
+                }
+        }
+#endif
+
+        if (ar==TRUE) {
+                lseek(srcFd, 8, SEEK_CUR); 
+                while(1) {
+                        if (readArEntry(srcFd, list) == FALSE) {
+                                lseek(srcFd, ++initialOffset, SEEK_CUR); 
+                                if (readArEntry(srcFd, list) == FALSE)
+                                        return(head);
+                        }
                         list->next = (headerL_t *) malloc(sizeof(headerL_t));
                         *list->next = *head;
                         *head = *list;
-                
                         /* recursive check for sub-archives */
-                        if ( funct & RECURSIVE ) 
+                        if (funct & RECURSIVE)  
                                 head = getHeaders(srcFd, head, funct);
                         lseek(srcFd, head->offset + head->size, SEEK_SET);
                 }
@@ -202,27 +307,6 @@ static headerL_t *findEntry(headerL_t *head, const char *filename)
         return(NULL);
 }
 
-/*
- * populate linked list with all ar file entries and offset 
- */
-static int displayEntry(headerL_t *head, int funct)
-{
-        if ( funct & VERBOSE ) {
-                printf("%s %d/%d %8d %s ", modeString(head->mode), head->uid, head->gid, head->size, timeString(head->mtime));
-        }
-        printf("%s\n", head->name);
-        head = head->next;
-        return(TRUE);
-}
-
-static int extractAr(int srcFd, int dstFd, headerL_t *file)
-{
-        lseek(srcFd, file->offset, SEEK_SET);
-        if (copySubFile(srcFd, dstFd, (size_t) file->size) == TRUE)
-                return(TRUE);   
-        return(FALSE);
-}
-
 extern int ar_main(int argc, char **argv)
 {
         int funct = 0, opt=0;
@@ -270,7 +354,6 @@ extern int ar_main(int argc, char **argv)
         extractList = (headerL_t *) malloc(sizeof(headerL_t));  
 
         header = getHeaders(srcFd, header, funct);
-        
         /* find files to extract or display */
         if (optind<argc) {
                 /* only handle specified files */
@@ -283,20 +366,28 @@ extern int ar_main(int argc, char **argv)
                         optind++;
                 }       
         }
-        else 
-                /* extract everything */
+        else  
                 extractList = header;
         
-       while(extractList->next != NULL) {       
-                if ( funct & EXT_TO_FILE ) {
+        while(extractList->next != NULL) {      
+                if (funct & EXT_TO_FILE) {
+                        if (isDirectory(extractList->name, TRUE, NULL)==FALSE)
+                                createPath(extractList->name, 0666);
                         dstFd = open(extractList->name, O_WRONLY | O_CREAT, extractList->mode);
-                        
-                        extractAr(srcFd, dstFd, extractList);
+                        lseek(srcFd, extractList->offset, SEEK_SET);
+                        copySubFile(srcFd, dstFd, (size_t) extractList->size);
+                }
+                if (funct & EXT_TO_STDOUT) {    
+                        lseek(srcFd, extractList->offset, SEEK_SET);
+                        copySubFile(srcFd, fileno(stdout), (size_t) extractList->size);
+                }
+                if ( (funct & DISPLAY) || (funct & VERBOSE)) {
+                        if (funct & VERBOSE)
+                                printf("%s %d/%d %8d %s ", modeString(extractList->mode), 
+                                        extractList->uid, extractList->gid,
+                                        extractList->size, timeString(extractList->mtime));
+                        printf("%s\n", extractList->name);
                 }
-                if ( funct & EXT_TO_STDOUT )    
-                        extractAr(srcFd, fileno(stdout), extractList);  
-                if ( (funct & DISPLAY) || (funct & VERBOSE))
-                        displayEntry(extractList, funct);
                 extractList=extractList->next;
         }
         return (TRUE);