diff options
Diffstat (limited to 'shell/hush.c')
-rw-r--r-- | shell/hush.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/shell/hush.c b/shell/hush.c index f6b50dec6..0ade2ccca 100644 --- a/shell/hush.c +++ b/shell/hush.c | |||
@@ -9377,7 +9377,18 @@ static int FAST_FUNC builtin_shift(char **argv) | |||
9377 | int n = 1; | 9377 | int n = 1; |
9378 | argv = skip_dash_dash(argv); | 9378 | argv = skip_dash_dash(argv); |
9379 | if (argv[0]) { | 9379 | if (argv[0]) { |
9380 | n = atoi(argv[0]); | 9380 | n = bb_strtou(argv[0], NULL, 10); |
9381 | if (errno || n < 0) { | ||
9382 | /* shared string with ash.c */ | ||
9383 | bb_error_msg("Illegal number: %s", argv[0]); | ||
9384 | /* | ||
9385 | * ash aborts in this case. | ||
9386 | * bash prints error message and set $? to 1. | ||
9387 | * Interestingly, for "shift 99999" bash does not | ||
9388 | * print error message, but does set $? to 1 | ||
9389 | * (and does no shifting at all). | ||
9390 | */ | ||
9391 | } | ||
9381 | } | 9392 | } |
9382 | if (n >= 0 && n < G.global_argc) { | 9393 | if (n >= 0 && n < G.global_argc) { |
9383 | if (G_global_args_malloced) { | 9394 | if (G_global_args_malloced) { |