On Sonntag, 28. März 2010, Jeff King wrote:
> + if (start_command(&child) != 0 ||
> + strbuf_read(&buf, child.out, 0) < 0 ||
> + finish_command(&child) != 0) {
This conditional is somewhat dubious. If strbuf_read fails, you do not wait
for the child, and a zombie remains.
The have this sequence already in run_textconv().
-- Hannes
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html