From 88ca06769028e442bf873b270c176ca0e9f021f8 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Thu, 12 Oct 2006 22:44:13 +0000 Subject: [PATCH] xread/write can use full_read/write (smaller code) --- libbb/full_read.c | 2 +- libbb/full_write.c | 2 +- libbb/xfuncs.c | 16 ++++++---------- 3 files changed, 8 insertions(+), 12 deletions(-) diff --git a/libbb/full_read.c b/libbb/full_read.c index b5837d5bd..068d16698 100644 --- a/libbb/full_read.c +++ b/libbb/full_read.c @@ -24,7 +24,7 @@ ssize_t full_read(int fd, void *buf, size_t len) total = 0; - while (len > 0) { + while (len) { cc = safe_read(fd, buf, len); if (cc < 0) diff --git a/libbb/full_write.c b/libbb/full_write.c index d812d04b4..563f4a851 100644 --- a/libbb/full_write.c +++ b/libbb/full_write.c @@ -23,7 +23,7 @@ ssize_t full_write(int fd, const void *buf, size_t len) total = 0; - while (len > 0) { + while (len) { cc = safe_write(fd, buf, len); if (cc < 0) diff --git a/libbb/xfuncs.c b/libbb/xfuncs.c index d22cd279a..efc919491 100644 --- a/libbb/xfuncs.c +++ b/libbb/xfuncs.c @@ -113,24 +113,20 @@ int xopen3(const char *pathname, int flags, int mode) // Die with an error message if we can't read the entire buffer. void xread(int fd, void *buf, size_t count) { - while (count) { - ssize_t size = safe_read(fd, buf, count); - if (size < 1) + if (count) { + ssize_t size = full_read(fd, buf, count); + if (size != count) bb_error_msg_and_die("short read"); - count -= size; - buf = ((char *) buf) + size; } } // Die with an error message if we can't write the entire buffer. void xwrite(int fd, void *buf, size_t count) { - while (count) { - ssize_t size = safe_write(fd, buf, count); - if (size < 1) + if (count) { + ssize_t size = full_write(fd, buf, count); + if (size != count) bb_error_msg_and_die("short write"); - count -= size; - buf = ((char *) buf) + size; } }