diff options
| author |  Johan Herland <johan@herland.net> | 2007-05-15 14:39:25 +0200 |
|---|---|---|
| committer |  Junio C Hamano <junkio@cox.net> | 2007-05-15 21:15:54 -0700 |
| commit | 2924415f4fb081d9dde687092248c86ec0c40195 (patch) | |
| tree | e81cf399b019741668bf0fb422953fe8c4c00a3e | |
| parent | Use $Id$ as the ident attribute keyword rather than $ident$ to be consistent ... (diff) | |
| download | tgif-2924415f4fb081d9dde687092248c86ec0c40195.tar.xz | |
Fix signedness on return value from xread()
The return value from xread() is ssize_t.
Paolo Teti <paolo.teti@gmail.com> pointed out that in this case, the
signed return value was assigned to an unsigned type (size_t). This patch
fixes that.
Signed-off-by: Johan Herland <johan@herland.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
| -rw-r--r-- | pack-write.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pack-write.c b/pack-write.c index de72f44dc8..ae2e481e55 100644 --- a/pack-write.c +++ b/pack-write.c @@ -25,7 +25,7 @@ void fixup_pack_header_footer(int pack_fd, buf = xmalloc(buf_sz); for (;;) { - size_t n = xread(pack_fd, buf, buf_sz); + ssize_t n = xread(pack_fd, buf, buf_sz); if (!n) break; if (n < 0) |