"Johannes Schindelin via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > From: Johannes Schindelin <johannes.schindelin@xxxxxx> > > 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@xxxxxx> > --- > 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);
