summaryrefslogtreecommitdiff
path: root/archival/tar.c
diff options
context:
space:
mode:
authorEric Andersen <andersen@codepoet.org>2001-06-20 20:47:31 +0000
committerEric Andersen <andersen@codepoet.org>2001-06-20 20:47:31 +0000
commitaff5e2eaef3a625bfa7760e59a6cf29de5cb0282 (patch)
tree0037911818b382882261c44a8de57c08d1cfb17b /archival/tar.c
parent06a71cced288126a5dfa2ff37b8138bebe8c1c42 (diff)
downloadbusybox-w32-aff5e2eaef3a625bfa7760e59a6cf29de5cb0282.tar.gz
busybox-w32-aff5e2eaef3a625bfa7760e59a6cf29de5cb0282.tar.bz2
busybox-w32-aff5e2eaef3a625bfa7760e59a6cf29de5cb0282.zip
it turs out, there was a nasty tar bug where bb tar would create leading dirs
with mode 0777 in all cases due to usask issues. Thanks to Matt Kraai for noticing and spotting the culprit. This makes bb tar behave just like GNU tar once again. -Erik
Diffstat (limited to 'archival/tar.c')
-rw-r--r--archival/tar.c8
1 files changed, 0 insertions, 8 deletions
diff --git a/archival/tar.c b/archival/tar.c
index 0cb638a7f..bca5a0906 100644
--- a/archival/tar.c
+++ b/archival/tar.c
@@ -576,10 +576,6 @@ static int readTarFile(int tarFd, int extractFlag, int listFlag,
576 TarHeader rawHeader; 576 TarHeader rawHeader;
577 TarInfo header; 577 TarInfo header;
578 578
579 /* Set the umask for this process so it doesn't
580 * screw up permission setting for us later. */
581 umask(0);
582
583 /* Read the tar file, and iterate over it one file at a time */ 579 /* Read the tar file, and iterate over it one file at a time */
584 while ( (status = full_read(tarFd, (char*)&rawHeader, TAR_BLOCK_SIZE)) == TAR_BLOCK_SIZE ) { 580 while ( (status = full_read(tarFd, (char*)&rawHeader, TAR_BLOCK_SIZE)) == TAR_BLOCK_SIZE ) {
585 581
@@ -1109,10 +1105,6 @@ static int writeTarFile(const char* tarName, int verboseFlag, char **argv,
1109 if (fstat(tbInfo.tarFd, &tbInfo.statBuf) < 0) 1105 if (fstat(tbInfo.tarFd, &tbInfo.statBuf) < 0)
1110 error_msg_and_die(io_error, tarName, strerror(errno)); 1106 error_msg_and_die(io_error, tarName, strerror(errno));
1111 1107
1112 /* Set the umask for this process so it doesn't
1113 * screw up permission setting for us later. */
1114 umask(0);
1115
1116 /* Read the directory/files and iterate over them one at a time */ 1108 /* Read the directory/files and iterate over them one at a time */
1117 while (*argv != NULL) { 1109 while (*argv != NULL) {
1118 if (recursive_action(*argv++, TRUE, FALSE, FALSE, 1110 if (recursive_action(*argv++, TRUE, FALSE, FALSE,