From 97310d025390e96f34140cff13034fcd2b5da18f Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Tue, 23 Mar 2004 23:15:36 +0000 Subject: Brian Pomerantz writes: I've noticed a bug in the "autowidth" feature more, and is probably in others. The call to the function get_terminal_width_height() passes in a file descriptor but that file descriptor is never used, instead the ioctl() is called with 0. In more_main() the call to get_terminal_width_height() passes 0 as the file descriptor instead of fileno(cin). This isn't a problem when you more a file (e.g. "more /etc/passwd") but when you pipe a file to it (e.g. "cat /etc/passwd | more") the size of the terminal cannot be determined because file descriptor 0 is not a terminal. The fix is simple, I've attached a patch for more.c and get_terminal_width_height.c. BAPper --- util-linux/more.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'util-linux') diff --git a/util-linux/more.c b/util-linux/more.c index d7b7ce22f..04b29de17 100644 --- a/util-linux/more.c +++ b/util-linux/more.c @@ -67,6 +67,7 @@ extern int more_main(int argc, char **argv) int please_display_more_prompt = -1; struct stat st; FILE *file; + FILE *in_file = stdin; int len, page_height; argc--; @@ -78,6 +79,7 @@ extern int more_main(int argc, char **argv) cin = fopen(CURRENT_TTY, "r"); if (!cin) cin = bb_xfopen(CONSOLE_DEV, "r"); + in_file = cin; please_display_more_prompt = 0; #ifdef CONFIG_FEATURE_USE_TERMIOS getTermSettings(fileno(cin), &initial_settings); @@ -108,7 +110,7 @@ extern int more_main(int argc, char **argv) if(please_display_more_prompt>0) please_display_more_prompt = 0; - get_terminal_width_height(0, &terminal_width, &terminal_height); + get_terminal_width_height(fileno(in_file), &terminal_width, &terminal_height); if (terminal_height > 4) terminal_height -= 2; if (terminal_width > 0) -- cgit v1.2.3