From 47f1b2773dddb73cd779684c5e4b8a48464b8bad Mon Sep 17 00:00:00 2001
From: Ingo Schwarze <schwarze@openbsd.org>
Date: Sat, 18 Oct 2014 15:57:34 +0000
Subject: plug file descriptor leaks on read or write failure; hinted at by
 Steffen Nurpmeso <sdaoden at yandex dot com>.

---
 main.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'main.c')

diff --git a/main.c b/main.c
index 9e896fd3..2451ff5e 100644
--- a/main.c
+++ b/main.c
@@ -581,14 +581,15 @@ passthrough(const char *file, int fd)
 		for (off = 0; off < nr; off += nw)
 			if ((nw = write(STDOUT_FILENO, buf + off,
 			    (size_t)(nr - off))) == -1 || nw == 0) {
+				close(fd);
 				syscall = "write";
 				goto fail;
 			}
 
-        if (nr == 0) {
-		close(fd);
+	close(fd);
+
+	if (nr == 0)
 		return(MANDOCLEVEL_OK);
-        }
 
 	syscall = "read";
 fail:
-- 
cgit