Re: [PATCH] packfile: Correct zlib buffer handling

2018-06-12 Thread Jeremy Linton
Hi, Sorry about the delay here (bit of a mix-up and didn't reply to the list). (see inline ) On Sun, May 27, 2018 at 9:41 PM, Junio C Hamano wrote: > Junio C Hamano writes: > >> Duy Nguyen writes: >> >>> On Sun, May 27, 2018 at 1:57 AM, Junio C Hamano wrote: (trimming) > > Specifically, I

Re: [PATCH] packfile: Correct zlib buffer handling

2018-05-27 Thread Junio C Hamano
Junio C Hamano writes: > Duy Nguyen writes: > >> On Sun, May 27, 2018 at 1:57 AM, Junio C Hamano wrote: >>> Duy Nguyen writes: >>> On Sat, May 26, 2018 at 12:56 AM, Jeremy Linton wrote: > @@ -1416,7 +1416,7 @@ static void *unpack_compressed_entry(struct > packed_git *p, >>

Re: [PATCH] packfile: Correct zlib buffer handling

2018-05-27 Thread Junio C Hamano
Duy Nguyen writes: > On Sun, May 27, 2018 at 1:57 AM, Junio C Hamano wrote: >> Duy Nguyen writes: >> >>> On Sat, May 26, 2018 at 12:56 AM, Jeremy Linton >>> wrote: @@ -1416,7 +1416,7 @@ static void *unpack_compressed_entry(struct packed_git *p, return NULL; >>>

Re: [PATCH] packfile: Correct zlib buffer handling

2018-05-26 Thread Duy Nguyen
On Sun, May 27, 2018 at 1:57 AM, Junio C Hamano wrote: > Duy Nguyen writes: > >> On Sat, May 26, 2018 at 12:56 AM, Jeremy Linton >> wrote: >>> @@ -1416,7 +1416,7 @@ static void *unpack_compressed_entry(struct >>> packed_git *p, >>> return NULL; >>> memset(&stream, 0, si

Re: [PATCH] packfile: Correct zlib buffer handling

2018-05-26 Thread Junio C Hamano
Duy Nguyen writes: > On Sat, May 26, 2018 at 12:56 AM, Jeremy Linton > wrote: >> @@ -1416,7 +1416,7 @@ static void *unpack_compressed_entry(struct packed_git >> *p, >> return NULL; >> memset(&stream, 0, sizeof(stream)); >> stream.next_out = buffer; >> - st

Re: [PATCH] packfile: Correct zlib buffer handling

2018-05-25 Thread Duy Nguyen
On Sat, May 26, 2018 at 12:56 AM, Jeremy Linton wrote: > @@ -1416,7 +1416,7 @@ static void *unpack_compressed_entry(struct packed_git > *p, > return NULL; > memset(&stream, 0, sizeof(stream)); > stream.next_out = buffer; > - stream.avail_out = size + 1; > +

Re: [PATCH] packfile: Correct zlib buffer handling

2018-05-25 Thread Todd Zullinger
Jeremy Linton wrote: > The buffer being passed to zlib includes a null terminator that > git needs to keep in place. unpack_compressed_entry() attempts to > detect the case that the source buffer hasn't been fully consumed > by checking to see if the destination buffer has been over consumed. > >

Re: [PATCH] packfile: Correct zlib buffer handling

2018-05-25 Thread Eric Sunshine
On Fri, May 25, 2018 at 7:17 PM, Jeremy Linton wrote: > The buffer being passed to zlib includes a null terminator that > git needs to keep in place. unpack_compressed_entry() attempts to > detect the case that the source buffer hasn't been fully consumed > by checking to see if the destination bu

[PATCH] packfile: Correct zlib buffer handling

2018-05-25 Thread Jeremy Linton
The buffer being passed to zlib includes a null terminator that git needs to keep in place. unpack_compressed_entry() attempts to detect the case that the source buffer hasn't been fully consumed by checking to see if the destination buffer has been over consumed. This yields two problems, first a

[PATCH] packfile: Correct zlib buffer handling

2018-05-25 Thread Jeremy Linton
The buffer being passed to zlib includes a null terminator that git needs to keep in place. unpack_compressed_entry() attempts to detect the case that the source buffer hasn't been fully consumed by checking to see if the destination buffer has been over consumed. This yields two problems, first a