diff options
author | Peter Stephenson <p.stephenson@samsung.com> | 2018-06-20 12:09:43 +0100 |
---|---|---|
committer | Peter Stephenson <p.stephenson@samsung.com> | 2018-06-20 12:09:43 +0100 |
commit | eada7e1138a3fca90f085dd764ad86098e58c9ac (patch) | |
tree | ec7df71375da7ac82ea9c67f2fa6685b92a42239 /Src/builtin.c | |
parent | 7caec474de577e59a6cf45952e5b87712967aa45 (diff) | |
download | zsh-eada7e1138a3fca90f085dd764ad86098e58c9ac.tar.gz zsh-eada7e1138a3fca90f085dd764ad86098e58c9ac.tar.xz zsh-eada7e1138a3fca90f085dd764ad86098e58c9ac.zip |
43077: Fix shift builtin status.
If the math evaulation to get the shift count failed the status wasn't passed back from the builtin.
Diffstat (limited to 'Src/builtin.c')
-rw-r--r-- | Src/builtin.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/Src/builtin.c b/Src/builtin.c index 1cba97dec..93fa9112c 100644 --- a/Src/builtin.c +++ b/Src/builtin.c @@ -5318,8 +5318,13 @@ bin_shift(char *name, char **argv, Options ops, UNUSED(int func)) /* optional argument can be either numeric or an array */ queue_signals(); - if (*argv && !getaparam(*argv)) + if (*argv && !getaparam(*argv)) { num = mathevali(*argv++); + if (errflag) { + unqueue_signals(); + return 1; + } + } if (num < 0) { unqueue_signals(); |