From e421b5ebf6b2484455ea36bb6c98deff685088e3 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Mon, 17 Mar 2008 22:01:42 +0000 Subject: [PATCH] init: do not close all descriptors > 2. We were doing it - sometimes. Good choices are "do it, always" or "don't do it". Second is smaller. --- init/init.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/init/init.c b/init/init.c index eefb9df55..c4674a55f 100644 --- a/init/init.c +++ b/init/init.c @@ -214,8 +214,7 @@ static void console_init(void) if (fd >= 0) { dup2(fd, 0); dup2(fd, 1); - dup2(fd, 2); - while (fd > 2) close(fd--); + xmove_fd(fd, 2); } messageD(L_LOG, "console='%s'", s); } else { @@ -223,7 +222,7 @@ static void console_init(void) * (so that they won't be used by future opens) */ /* bb_sanitize_stdio(); - WRONG. - * Fail if "/dev/null" doesnt exist, and for init + * It fails if "/dev/null" doesnt exist, and for init * this is a real possibility! Open code it instead. */ int fd = open(bb_dev_null, O_RDWR); @@ -234,11 +233,11 @@ static void console_init(void) while ((unsigned)fd < 2) fd = dup(fd); if (fd > 2) - close (fd); + close(fd); } s = getenv("TERM"); - if (ioctl(0, TIOCGSERIAL, &sr) == 0) { + if (ioctl(STDIN_FILENO, TIOCGSERIAL, &sr) == 0) { /* Force the TERM setting to vt102 for serial console * if TERM is set to linux (the default) */ if (!s || strcmp(s, "linux") == 0)