summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko2014-03-16 12:34:53 +0100
committerDenys Vlasenko2014-03-16 12:34:53 +0100
commit438b4ac2860f4aec5b5fb82c47f75a0cb80d9cb4 (patch)
tree83dd96e62dfe975fa9ea3f738fe32e220862b2fa
parent29f352604868c5d17c81948ea30e332491cecada (diff)
downloadbusybox-438b4ac2860f4aec5b5fb82c47f75a0cb80d9cb4.zip
busybox-438b4ac2860f4aec5b5fb82c47f75a0cb80d9cb4.tar.gz
script: make it work even if fd 0 is closed
Testcase: script -q -c "echo hey" /dev/null 0>&- Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--libbb/getpty.c2
-rw-r--r--util-linux/script.c7
2 files changed, 8 insertions, 1 deletions
diff --git a/libbb/getpty.c b/libbb/getpty.c
index 435e4d0..391d729 100644
--- a/libbb/getpty.c
+++ b/libbb/getpty.c
@@ -16,7 +16,7 @@ int FAST_FUNC xgetpty(char *line)
#if ENABLE_FEATURE_DEVPTS
p = open("/dev/ptmx", O_RDWR);
- if (p > 0) {
+ if (p >= 0) {
grantpt(p); /* chmod+chown corresponding slave pty */
unlockpt(p); /* (what does this do?) */
# ifndef HAVE_PTSNAME_R
diff --git a/util-linux/script.c b/util-linux/script.c
index 8fb991d..abcd73b 100644
--- a/util-linux/script.c
+++ b/util-linux/script.c
@@ -77,8 +77,15 @@ int script_main(int argc UNUSED_PARAM, char **argv)
if (!(opt & OPT_q)) {
printf("Script started, file is %s\n", fname);
}
+
shell = get_shell_name();
+ /* Some people run "script ... 0>&-".
+ * Our code assumes that STDIN_FILENO != pty.
+ * Ensure STDIN_FILENO is not closed:
+ */
+ bb_sanitize_stdio();
+
pty = xgetpty(pty_line);
/* get current stdin's tty params */