Browse files

stream_to_pack: xread does not guarantee to read all requested bytes

The deflate loop in bulk-checkin::stream_to_pack expects to get all bytes
from a file that it requests to read in a single function call. But it
used xread(), which does not give that guarantee. Replace it by

Signed-off-by: Johannes Sixt <>
Signed-off-by: Junio C Hamano <>
  • Loading branch information...
1 parent 1599999 commit e92527c97cfa26bf6e42381cf1b7035950c36e69 @j6t j6t committed with gitster Aug 20, 2013
Showing with 1 addition and 1 deletion.
  1. +1 −1 bulk-checkin.c
@@ -114,7 +114,7 @@ static int stream_to_pack(struct bulk_checkin_state *state,
if (size && !s.avail_in) {
ssize_t rsize = size < sizeof(ibuf) ? size : sizeof(ibuf);
- if (xread(fd, ibuf, rsize) != rsize)
+ if (read_in_full(fd, ibuf, rsize) != rsize)
die("failed to read %d bytes from '%s'",
(int)rsize, path);
offset += rsize;

0 comments on commit e92527c

Please sign in to comment.