diff options
Diffstat (limited to 'libbb')
| -rw-r--r-- | libbb/copyfd.c | 9 |
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) | |||
| 29 | RESERVE_CONFIG_BUFFER(buffer,BUFSIZ); | 29 | RESERVE_CONFIG_BUFFER(buffer,BUFSIZ); |
| 30 | 30 | ||
| 31 | if (src_fd < 0) goto out; | 31 | if (src_fd < 0) goto out; |
| 32 | |||
| 33 | while (!size || total < size) | 32 | while (!size || total < size) |
| 34 | { | 33 | { |
| 35 | ssize_t wrote, xread = (size && size < BUFSIZ) ? size : BUFSIZ; | 34 | ssize_t wrote, xread; |
| 36 | xread = safe_read(src_fd, buffer, xread); | 35 | |
| 36 | xread = safe_read(src_fd, buffer, | ||
| 37 | (!size || size - total > BUFSIZ) ? BUFSIZ : size - total); | ||
| 38 | |||
| 37 | if (xread > 0) { | 39 | if (xread > 0) { |
| 38 | /* A -1 dst_fd means we need to fake it... */ | 40 | /* A -1 dst_fd means we need to fake it... */ |
| 39 | wrote = (dst_fd < 0) ? xread : bb_full_write(dst_fd, buffer, xread); | 41 | 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) | |||
| 42 | break; | 44 | break; |
| 43 | } | 45 | } |
| 44 | total += wrote; | 46 | total += wrote; |
| 45 | size -= wrote; | ||
| 46 | } else if (xread < 0) { | 47 | } else if (xread < 0) { |
| 47 | bb_perror_msg(bb_msg_read_error); | 48 | bb_perror_msg(bb_msg_read_error); |
| 48 | break; | 49 | break; |
