Re: [PATCH 07/11] submodule--helper: avoid memory leak when fetching submodules
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 16, 2022, 04:55 UTC
- Message-ID
- <xmqqedzpw6k8.fsf@gitster.g>
- In-Reply-To
- <ad3f23a9f0c2c92ac95406fcadc51eec1a9861ff.1655336146.git.gitgitgadget@gmail.com>
"Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com> writes:
Show 24 quoted lines
> From: Johannes Schindelin <johannes.schindelin@gmx.de> > > In c51f8f94e5b3 (submodule--helper: run update procedures from C, > 2021-08-24), we added code that first obtains the default remote, and > then adds that to a `strvec`. > > However, we never released the default remote's memory. > > Reported by Coverity. > > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> > --- > builtin/submodule--helper.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c > index d7b8004b933..9a8248ffe6a 100644 > --- a/builtin/submodule--helper.c > +++ b/builtin/submodule--helper.c > @@ -2208,6 +2208,7 @@ static int fetch_in_submodule(const char *module_path, int depth, int quiet, str > char *hex = oid_to_hex(oid); > char *remote = get_default_remote(); > strvec_pushl(&cp.args, remote, hex, NULL); > + free(remote);
Trivially correct. Will queue.
> } > > return run_command(&cp);