just after calling inflate_unzip() there is this line:
count = bytebuffer_size - bytebuffer_offset;
if (count < 8) {
...
but count is a char (1 byte) i hit a situation where bytebuffer_size -
bytebuffer_offset was 2305 and gzip was complaining 'Short read' (there
is a bb_xread_all in the 'if' condition which tries to read from in-fd
which has already eof'd)
count being 1 byte interpreted 2305 as 7 and wrongly entered the 'if'
as far as functionality is concerned this happens _after_ the actual
un-gzip and only the trailers are affected (which tar happily neglects
as it knows its boundry limit).. but the code is bad and results in
wrong exit status, which affected my script..
have attached a small diff which changes the char to int.. the fix most
likely is fixing the symptom and not the root cause, or maybe the root
cause itself.
decompress_unzip.diff [^] (509 bytes) 08-31-05 10:40 
