diff options
author | Eric Andersen <andersen@codepoet.org> | 2004-05-05 10:37:49 +0000 |
---|---|---|
committer | Eric Andersen <andersen@codepoet.org> | 2004-05-05 10:37:49 +0000 |
commit | aaff79a8dd00f2a9652a4bce8fbfe77715804ac2 (patch) | |
tree | 2a6a8e172ed56b1d5083626bf93028ac2678a8e9 /archival | |
parent | 6c8161d69fe9fce0f862b678aaa84866aaaeff8f (diff) | |
download | busybox-w32-aaff79a8dd00f2a9652a4bce8fbfe77715804ac2.tar.gz busybox-w32-aaff79a8dd00f2a9652a4bce8fbfe77715804ac2.tar.bz2 busybox-w32-aaff79a8dd00f2a9652a4bce8fbfe77715804ac2.zip |
Steve Grubb writes:
Hello,
I found and patched 2 more bugs. The first is a misplaced semi-colon. The second
one is a buffer overflow. I doubt the buffer overflow is triggered in real life.
But you never know what those wily hackers are up to.
Thanks,
Steve Grubb
Diffstat (limited to 'archival')
-rw-r--r-- | archival/tar.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/archival/tar.c b/archival/tar.c index 9d50a101d..efdc46587 100644 --- a/archival/tar.c +++ b/archival/tar.c | |||
@@ -477,8 +477,8 @@ static inline int writeTarFile(const int tar_fd, const int verboseFlag, | |||
477 | dup2(gzipDataPipe[0], 0); | 477 | dup2(gzipDataPipe[0], 0); |
478 | close(gzipDataPipe[1]); | 478 | close(gzipDataPipe[1]); |
479 | 479 | ||
480 | if (tbInfo.tarFd != 1); | 480 | if (tbInfo.tarFd != 1) |
481 | dup2(tbInfo.tarFd, 1); | 481 | dup2(tbInfo.tarFd, 1); |
482 | 482 | ||
483 | close(gzipStatusPipe[0]); | 483 | close(gzipStatusPipe[0]); |
484 | fcntl(gzipStatusPipe[1], F_SETFD, FD_CLOEXEC); /* close on exec shows sucess */ | 484 | fcntl(gzipStatusPipe[1], F_SETFD, FD_CLOEXEC); /* close on exec shows sucess */ |