On Tue, Oct 10, 2017 at 11:23:19AM +0200, Simon Ruderich wrote:
> On Tue, Oct 10, 2017 at 09:00:09AM +0900, Junio C Hamano wrote:
> >> --- a/entry.c
> >> +++ b/entry.c
> >> @@ -283,6 +283,7 @@ static int write_entry(struct cache_entry *ce,
> >> if (dco && dco->state != CE_NO_DELAY) {
> >> /* Do not send the blob in case of a retry. */
> >> if (dco->state == CE_RETRY) {
> >> + free(new);
> >> new = NULL;
> >> size = 0;
> >> }
>
> FREE_AND_NULL(new)?
Ah, yeah, I forgot we had that now. It would work here, but note that
this code ends up going away later in the series.
-Peff