Hi Stefan,
On Wed, 30 May 2018, Stefan Beller wrote:
> Signed-off-by: Stefan Beller <sbel...@google.com>
> ---
> builtin/submodule--helper.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
> index 7c3cd9dbeba..96024fee1b1 100644
> --- a/builtin/submodule--helper.c
> +++ b/builtin/submodule--helper.c
> @@ -63,6 +63,7 @@ static int print_default_remote(int argc, const char
> **argv, const char *prefix)
> if (remote)
> printf("%s\n", remote);
>
> + free(remote);
Makes sense.
Out of curiosity (and because a cover letter is missing): how did you
stumble over these? Coverity?
Ciao,
Dscho
> return 0;
> }
>
> --
> 2.17.1.1185.g55be947832-goog
>
>
