diff options
author | Eric Andersen | 2004-03-23 23:15:36 +0000 |
---|---|---|
committer | Eric Andersen | 2004-03-23 23:15:36 +0000 |
commit | 97310d025390e96f34140cff13034fcd2b5da18f (patch) | |
tree | be9b3406b9b0598d0bc42d6912a7855f7ee11819 /testsuite/grep/grep-handles-binary-files | |
parent | 6bb80870b839cc812d55b28f886bf3e001e4a00e (diff) | |
download | busybox-97310d025390e96f34140cff13034fcd2b5da18f.zip busybox-97310d025390e96f34140cff13034fcd2b5da18f.tar.gz |
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
Diffstat (limited to 'testsuite/grep/grep-handles-binary-files')
0 files changed, 0 insertions, 0 deletions