[PATCH] daemon: disable SIGWINCH unless explicitly daemonized

[PATCH] daemon: disable SIGWINCH unless explicitly daemonized

[Eric Wong]

Checking stdin/stdout/stderr is not sufficient as the daemon
without setsid can still be under the control of a terminal.

Unfortunately this means systemd users cannot use SIGWINCH,
either.
---
 lib/PublicInbox/Daemon.pm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/lib/PublicInbox/Daemon.pm b/lib/PublicInbox/Daemon.pm
index a25dd90..512bb20 100644
--- a/lib/PublicInbox/Daemon.pm
+++ b/lib/PublicInbox/Daemon.pm
@@ -376,12 +376,12 @@ sub master_loop {
 				exit if $quit++;
 				kill_workers($s);
 			} elsif ($s eq 'WINCH') {
-				if (-t STDIN || -t STDOUT || -t STDERR) {
+				if ($daemonize) {
+					$worker_processes = 0;
+				} else {
 					warn
-"ignoring SIGWINCH while connected to terminal\n";
+"ignoring SIGWINCH since we are not daemonized\n";
 					$SIG{WINCH} = 'IGNORE';
-				} else {
-					$worker_processes = 0;
 				}
 			} elsif ($s eq 'HUP') {
 				$worker_processes = $set_workers;