Jacob Keller <jacob.e.kel...@intel.com> writes:

> @@ -523,8 +523,10 @@ char *git_pathdup_submodule(const char *path, const char 
> *fmt, ...)
>       va_start(args, fmt);
>       err = do_submodule_path(&buf, path, fmt, args);
>       va_end(args);
> -     if (err)
> +     if (err) {
> +             strbuf_release(&buf);
>               return NULL;
> +     }
>       return strbuf_detach(&buf, NULL);
>  }

Thanks.  My copy was lacking this hunk.  Will replace.

Reply via email to