Jim Meyering <jim@xxxxxxxxxxxx> writes:
> Signed-off-by: Jim Meyering <meyering@xxxxxxxxxx>
> ---
> Coverity spotted this.
Thanks, will apply, but it seems that Coverity does not know or care that
this codepath is used only once in the process and the process immediately
exits after the small leak that we didn't bother spending two extra lines
to plug, which is kind of sad.
> builtin/fetch.c | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/builtin/fetch.c b/builtin/fetch.c
> index f9c41da..93c9938 100644
> --- a/builtin/fetch.c
> +++ b/builtin/fetch.c
> @@ -875,6 +875,7 @@ static int fetch_one(struct remote *remote, int argc, const char **argv)
> {
> int i;
> static const char **refs = NULL;
> + struct refspec *refspec;
> int ref_nr = 0;
> int exit_code;
>
> @@ -915,8 +916,9 @@ static int fetch_one(struct remote *remote, int argc, const char **argv)
>
> sigchain_push_common(unlock_pack_on_signal);
> atexit(unlock_pack);
> - exit_code = do_fetch(transport,
> - parse_fetch_refspec(ref_nr, refs), ref_nr);
> + refspec = parse_fetch_refspec(ref_nr, refs);
> + exit_code = do_fetch(transport, refspec, ref_nr);
> + free(refspec);
> transport_disconnect(transport);
> transport = NULL;
> return exit_code;
> --
> 1.7.6.rc0.293.g40857
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html