Remove the signal array stack as our pipe handling should be secure enough now. Also, move the normalize close_on_exec and setting non block.

1 files changed, 15 insertions, 1 deletions

diff --git a/common.c b/common.c
index 7eeb1d9d..5754d40f 100644
--- a/common.c
+++ b/common.c
@@ -171,7 +171,7 @@ fd_hasdata(int fd)
 }
 
 int
-close_on_exec(int fd)
+set_cloexec(int fd)
 {
 	int flags;
 
@@ -184,6 +184,20 @@ close_on_exec(int fd)
 	return 0;
 }
 
+int
+set_nonblock(int fd)
+{
+	int flags;
+
+	if ((flags = fcntl(fd, F_GETFL, 0)) == -1
+	    || fcntl(fd, F_SETFL, flags | O_NONBLOCK) == -1)
+	{
+		logger(LOG_ERR, "fcntl: %s", strerror(errno));
+		return -1;
+	}
+	return 0;
+}
+
 /* Handy function to get the time.
  * We only care about time advancements, not the actual time itself
  * Which is why we use CLOCK_MONOTONIC, but it is not available on all