diff options
author | Denys Vlasenko | 2010-03-11 21:17:55 +0100 |
---|---|---|
committer | Denys Vlasenko | 2010-03-11 21:17:55 +0100 |
commit | 58f108eb339957f58d5a6034d82b09c4d50b53e3 (patch) | |
tree | 4fdfae71136811f3b9c8ffd56656b78da19739da /testsuite | |
parent | b0a57abb79001b994115d2c96a7d9e1f2f511430 (diff) | |
download | busybox-58f108eb339957f58d5a6034d82b09c4d50b53e3.zip busybox-58f108eb339957f58d5a6034d82b09c4d50b53e3.tar.gz |
lineedit: fix another corner case with bad unicode input
function old new delta
read_key 607 646 +39
readit 50 55 +5
getch_nowait 290 295 +5
hash_find 233 234 +1
xstrtoul_range_sfx 231 230 -1
passwd_main 1058 1056 -2
builtin_exit 45 43 -2
cmp_main 649 645 -4
lineedit_read_key 257 245 -12
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 4/5 up/down: 50/-21) Total: 29 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'testsuite')
-rwxr-xr-x | testsuite/ash.tests | 26 |
1 files changed, 20 insertions, 6 deletions
diff --git a/testsuite/ash.tests b/testsuite/ash.tests index 4b6efe4..6b2caf3 100755 --- a/testsuite/ash.tests +++ b/testsuite/ash.tests @@ -10,33 +10,47 @@ # testing "test name" "options" "expected result" "file input" "stdin" testing "One byte which is not valid unicode char followed by valid input" \ - "script -q -c 'ash' /dev/null >/dev/null; cat output; rm output" \ + "script -q -c 'ash' /dev/null >/dev/null; cat ash.output" \ "\ 00000000 3f 2d 0a |?-.| 00000003 " \ "" \ - "echo \xff- | hexdump -C >output; exit; exit; exit; exit\n" \ + "echo \xff- | hexdump -C >ash.output; exit; exit; exit; exit\n" testing "30 bytes which are not valid unicode chars followed by valid input" \ - "script -q -c 'ash' /dev/null >/dev/null; cat output; rm output" \ + "script -q -c 'ash' /dev/null >/dev/null; cat ash.output" \ "\ 00000000 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f |????????????????| 00000010 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 3f 2d 0a |??????????????-.| 00000020 " \ "" \ - "echo \xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff- | hexdump -C >output; exit; exit; exit; exit\n" \ + "echo \xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff- | hexdump -C >ash.output; exit; exit; exit; exit\n" # Not sure this behavior is perfect: we lose all invalid input which precedes # arrow keys and such. In this example, \xff\xff are lost testing "2 bytes which are not valid unicode chars followed by left arrow key" \ - "script -q -c 'ash' /dev/null >/dev/null; cat output; rm output" \ + "script -q -c 'ash' /dev/null >/dev/null; cat ash.output" \ "\ 00000000 3d 2d 0a |=-.| 00000003 " \ "" \ - "echo =+\xff\xff\x1b\x5b\x44- | hexdump -C >output; exit; exit; exit; exit\n" \ + "echo =+\xff\xff\x1b\x5b\x44- | hexdump -C >ash.output; exit; exit; exit; exit\n" + +# ash should see "echo \xff\n",pause -> execute it as "echo ?" (which is +# not checked by the test), then read and execute the rest: "echo A | ..." +# The bug was that ash was eating the beginning of "echo A" despite the pause. +testing "Invalid unicode chars followed by a pause do not eat next chars" \ + "{ echo -ne 'echo \xff\n'; sleep 1; echo -ne 'echo A | hexdump -C >ash.output; exit; exit; exit; exit\n'; } \ + | script -q -c 'ash' /dev/null >/dev/null; cat ash.output" \ + "\ +00000000 41 0a |A.| +00000002 +" \ + "" "" + +rm ash.output exit $FAILCOUNT |