# [PATCH] textconv: stop leaking file descriptors

1 messages from 2009-12-30 to 2009-12-30. Participants: Jeff King.
Thread: https://gitlist.dev/t/22060

## Jeff King, 2009-12-30 09:02

Subject: [PATCH] textconv: stop leaking file descriptors
Message-ID: <20091230090253.GA24049@coredump.intra.peff.net>
URL: https://gitlist.dev/e/20091230090253.GA24049%40coredump.intra.peff.net

```
We read the output from textconv helpers over a pipe, but we
never actually closed our end of the pipe after using it.

Signed-off-by: Jeff King <peff@peff.net>
---
This should go onto 'maint'. Now I'll just go find a brown paper bag...

 diff.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/diff.c b/diff.c
index d14a575..aad4b39 100644
--- a/diff.c
+++ b/diff.c
@@ -3823,11 +3823,13 @@ static char *run_textconv(const char *pgm, struct diff_filespec *spec,
 	if (start_command(&child) != 0 ||
 	    strbuf_read(&buf, child.out, 0) < 0 ||
 	    finish_command(&child) != 0) {
+		close(child.out);
 		strbuf_release(&buf);
 		remove_tempfile();
 		error("error running textconv command '%s'", pgm);
 		return NULL;
 	}
+	close(child.out);
 	remove_tempfile();
 
 	return strbuf_detach(&buf, outsize);
-- 
1.6.6.320.g7c9b3

```