My last change to this file broke tar x. Sigh. (Trying to make sure we work

on file sizes between 2 and 4 gigs on 32 bit machines.  For supporting >4 gigs,
just use a 64 bit machine already...)

1 files changed, 5 insertions, 4 deletions

diff --git a/libbb/copyfd.c b/libbb/copyfd.c
index c1962e3c6..0b850884b 100644
--- a/libbb/copyfd.c
+++ b/libbb/copyfd.c
@@ -29,11 +29,13 @@ static ssize_t bb_full_fd_action(int src_fd, int dst_fd, size_t size)
 	RESERVE_CONFIG_BUFFER(buffer,BUFSIZ);
 
 	if (src_fd < 0) goto out;
-
 	while (!size || total < size)
 	{
-		ssize_t wrote, xread = (size && size < BUFSIZ) ? size : BUFSIZ;
-		xread = safe_read(src_fd, buffer, xread);
+		ssize_t wrote, xread;
+		
+		xread = safe_read(src_fd, buffer,
+				(!size || size - total > BUFSIZ) ? BUFSIZ : size - total);
+
 		if (xread > 0) {
 			/* A -1 dst_fd means we need to fake it... */
 			wrote = (dst_fd < 0) ? xread : bb_full_write(dst_fd, buffer, xread);
@@ -42,7 +44,6 @@ static ssize_t bb_full_fd_action(int src_fd, int dst_fd, size_t size)
 				break;
 			}
 			total += wrote;
-			size -= wrote;
 		} else if (xread < 0) {
 			bb_perror_msg(bb_msg_read_error);
 			break;