From 51854c614873f59571e80da79f0dc0e8446cbf21 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bert=20M=C3=BCnnich?= Date: Thu, 27 Nov 2014 22:25:27 +0100 Subject: Fixed leakage of pipe descriptors in case of failing fork --- main.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/main.c b/main.c index fcbab7d..f8dbab8 100644 --- a/main.c +++ b/main.c @@ -262,20 +262,22 @@ void open_info(void) if (pipe(pfd) < 0) return; - pid = fork(); - if (pid > 0) { - close(pfd[1]); - fcntl(pfd[0], F_SETFL, O_NONBLOCK); - info.fd = pfd[0]; - info.i = info.lastsep = 0; - info.open = true; - } else if (pid == 0) { + if ((pid = fork()) == 0) { close(pfd[0]); dup2(pfd[1], 1); execl(info.cmd, info.cmd, files[fileidx].name, NULL); warn("could not exec: %s", info.cmd); exit(EXIT_FAILURE); } + close(pfd[1]); + if (pid < 0) { + close(pfd[0]); + } else { + fcntl(pfd[0], F_SETFL, O_NONBLOCK); + info.fd = pfd[0]; + info.i = info.lastsep = 0; + info.open = true; + } } void read_info(void) -- cgit v1.2.3