Jacob Keller <jacob.e.keller@xxxxxxxxx> 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.