Re: [PATCH v2 1/1] Don't free remote->name after fetch

2016-06-14 Thread Junio C Hamano
Keith McGuigan writes: > Yeah that was the only place I found where it was doing the strdup > already (and in that situation, it has to). All the other places just > grabbed remote->name. > > Yes, sorry, I can sign off on this. Do you want me to resend with the > header in place, or is this con

Re: [PATCH v2 1/1] Don't free remote->name after fetch

2016-06-14 Thread Keith McGuigan
Yeah that was the only place I found where it was doing the strdup already (and in that situation, it has to). All the other places just grabbed remote->name. Yes, sorry, I can sign off on this. Do you want me to resend with the header in place, or is this confirmation good enough? -- - Keith

Re: [PATCH v2 1/1] Don't free remote->name after fetch

2016-06-14 Thread Junio C Hamano
kmcgui...@twopensource.com writes: > From: Keith McGuigan > > Make fetch's string_list of remote names owns all of its string items > (strdup'ing when necessary) so that it can deallocate them safely when > clearing. > > --- OK. When I pointed out the call to string_list_append() in get_remote_

[PATCH v2 1/1] Don't free remote->name after fetch

2016-06-14 Thread kmcguigan
From: Keith McGuigan Make fetch's string_list of remote names owns all of its string items (strdup'ing when necessary) so that it can deallocate them safely when clearing. --- builtin/fetch.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/builtin/fetch.c b/builtin/fet