diff options
author | Eric Andersen <andersen@codepoet.org> | 2005-04-27 14:51:38 +0400 |
---|---|---|
committer | Eric Andersen <andersen@codepoet.org> | 2005-04-27 14:51:38 +0400 |
commit | fef32b570bbf9226778482e2f3f693b83d4e9c65 (patch) | |
tree | 6b1e7a75c9cbf9fde7e9dabd1508e84632f04d2c /libbb/copyfd.c | |
parent | edb3fbc30517883a1e7ec5eacdefd610305069fa (diff) |
Correct errors preventing busybox tar from working properly,
fixing bug http://bugs.uclibc.org/view.php?id=231
Diffstat (limited to 'libbb/copyfd.c')
-rw-r--r-- | libbb/copyfd.c | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/libbb/copyfd.c b/libbb/copyfd.c index baf99df51..27d65a419 100644 --- a/libbb/copyfd.c +++ b/libbb/copyfd.c @@ -39,7 +39,7 @@ static size_t bb_full_fd_action(int src_fd, int dst_fd, const size_t size2) int status; size_t xread, wrote, total, size = size2; - if ((dst_fd < 0) || (src_fd < 0)) { + if (src_fd < 0) { return -1; } @@ -56,11 +56,16 @@ static size_t bb_full_fd_action(int src_fd, int dst_fd, const size_t size2) while (total < size) { xread = BUFSIZ; - if (size < (wrote + BUFSIZ)) - xread = size - wrote; + if (size < (total + BUFSIZ)) + xread = size - total; xread = bb_full_read(src_fd, buffer, xread); if (xread > 0) { - wrote = bb_full_write(dst_fd, buffer, xread); + if (dst_fd < 0) { + /* A -1 dst_fd means we need to fake it... */ + wrote = xread; + } else { + wrote = bb_full_write(dst_fd, buffer, xread); + } if (wrote < xread) { bb_perror_msg(bb_msg_write_error); break; @@ -78,8 +83,8 @@ static size_t bb_full_fd_action(int src_fd, int dst_fd, const size_t size2) RELEASE_CONFIG_BUFFER(buffer); } - if (status == 0 || wrote) - return wrote; + if (status == 0 || total) + return total; /* Some sortof error occured */ return -1; } |