summaryrefslogtreecommitdiffabout
authorLars Hjemli <hjemli@gmail.com>2008-05-20 15:56:47 (UTC)
committer Lars Hjemli <hjemli@gmail.com>2008-05-20 15:56:47 (UTC)
commitdd7c172542440170b5b1aca8be43d2ad6dae7227 (patch) (side-by-side diff)
treed991f45be79cec1d3d031bac70413146f593a018
parentaf2e75616d1bfb7dc79d299d10ae0bd39bef47bc (diff)
downloadcgit-dd7c172542440170b5b1aca8be43d2ad6dae7227.zip
cgit-dd7c172542440170b5b1aca8be43d2ad6dae7227.tar.gz
cgit-dd7c172542440170b5b1aca8be43d2ad6dae7227.tar.bz2
cache.c: fix error checking in print_slot()
The change to print_slot() in cdc6b2f8e7a8d43dcfe0475a9d3498333ea686b8 made the function return correct errno for read errors while ignoring write errors, which is not what was intended. This patch tries to rectify things. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Diffstat (more/less context) (show whitespace changes)
-rw-r--r--cache.c11
1 files changed, 7 insertions, 4 deletions
diff --git a/cache.c b/cache.c
index aa97ae1..9f02cf5 100644
--- a/cache.c
+++ b/cache.c
@@ -83,3 +83,3 @@ static int print_slot(struct cache_slot *slot)
{
- ssize_t i;
+ ssize_t i, j;
@@ -89,6 +89,9 @@ static int print_slot(struct cache_slot *slot)
- while((i = xread(slot->cache_fd, slot->buf, sizeof(slot->buf))) > 0)
- i = xwrite(STDOUT_FILENO, slot->buf, i);
+ do {
+ i = j = xread(slot->cache_fd, slot->buf, sizeof(slot->buf));
+ if (i > 0)
+ j = xwrite(STDOUT_FILENO, slot->buf, i);
+ } while (i > 0 && j == i);
- if (i < 0)
+ if (i < 0 || j != i)
return errno;