summaryrefslogtreecommitdiff
path: root/archival/tar.c
diff options
context:
space:
mode:
authorEric Andersen2004-05-05 10:37:49 +0000
committerEric Andersen2004-05-05 10:37:49 +0000
commitaaff79a8dd00f2a9652a4bce8fbfe77715804ac2 (patch)
tree2a6a8e172ed56b1d5083626bf93028ac2678a8e9 /archival/tar.c
parent6c8161d69fe9fce0f862b678aaa84866aaaeff8f (diff)
downloadbusybox-aaff79a8dd00f2a9652a4bce8fbfe77715804ac2.zip
busybox-aaff79a8dd00f2a9652a4bce8fbfe77715804ac2.tar.gz
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/tar.c')
-rw-r--r--archival/tar.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/archival/tar.c b/archival/tar.c
index 9d50a10..efdc465 100644
--- a/archival/tar.c
+++ b/archival/tar.c
@@ -477,8 +477,8 @@ static inline int writeTarFile(const int tar_fd, const int verboseFlag,
dup2(gzipDataPipe[0], 0);
close(gzipDataPipe[1]);
- if (tbInfo.tarFd != 1);
- dup2(tbInfo.tarFd, 1);
+ if (tbInfo.tarFd != 1)
+ dup2(tbInfo.tarFd, 1);
close(gzipStatusPipe[0]);
fcntl(gzipStatusPipe[1], F_SETFD, FD_CLOEXEC); /* close on exec shows sucess */