> } else {
> + struct strbuf cmd = STRBUF_INIT;
> +
...
> strbuf_addstr(&cmd, prog);
> strbuf_addch(&cmd, ' ');
> sq_quote_buf(&cmd, path);
...
> if (start_command(conn))
> die("unable to fork");
...
> strbuf_release(&cmd);
Here I reread the cover letter, your intent is only to fix memleaks
on non-die code paths. Ok. (unlike the grand vision of an
"always valgrind clean program")
Thanks,
Stefan

