[git commit] getty: fix for NOCTTY killing us with SIGHUP

Denys Vlasenko vda.linux at googlemail.com
Thu Apr 26 08:53:48 UTC 2012


commit: http://git.busybox.net/busybox/commit/?id=eced0c78a54bbecc61f1717d33f64ee7d99804bb
branch: http://git.busybox.net/busybox/commit/?id=refs/heads/master

Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
 loginutils/getty.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/loginutils/getty.c b/loginutils/getty.c
index afb411b..bbb5a96 100644
--- a/loginutils/getty.c
+++ b/loginutils/getty.c
@@ -561,8 +561,14 @@ int getty_main(int argc UNUSED_PARAM, char **argv)
 		 */
 		fd = open("/dev/tty", O_RDWR | O_NONBLOCK);
 		if (fd >= 0) {
+			/* TIOCNOTTY sends SIGHUP to the foreground
+			 * process group - which may include us!
+			 * Make sure to not die on it:
+			 */
+			sighandler_t old = signal(SIGHUP, SIG_IGN);
 			ioctl(fd, TIOCNOTTY);
 			close(fd);
+			signal(SIGHUP, old);
 		}
 	}
 


More information about the busybox-cvs mailing list