Re: [PATCH] builtin/notes: remove unnecessary free

2018-11-12 Thread Junio C Hamano
Johan Herland writes: > On Sun, Nov 11, 2018 at 10:49 AM Carlo Marcelo Arenas Belón > wrote: >> >> 511726e4b1 ("builtin/notes: fix premature failure when trying to add >> the empty blob", 2014-11-09) removed the check for !len but left a >> call to free the buffer that will be otherwise NULL

Re: [PATCH] builtin/notes: remove unnecessary free

2018-11-11 Thread Johan Herland
On Sun, Nov 11, 2018 at 10:49 AM Carlo Marcelo Arenas Belón wrote: > > 511726e4b1 ("builtin/notes: fix premature failure when trying to add > the empty blob", 2014-11-09) removed the check for !len but left a > call to free the buffer that will be otherwise NULL > > Signed-off-by: Carlo Marcelo

[PATCH] builtin/notes: remove unnecessary free

2018-11-11 Thread Carlo Marcelo Arenas Belón
511726e4b1 ("builtin/notes: fix premature failure when trying to add the empty blob", 2014-11-09) removed the check for !len but left a call to free the buffer that will be otherwise NULL Signed-off-by: Carlo Marcelo Arenas Belón --- builtin/notes.c | 4 +--- 1 file changed, 1 insertion(+), 3