busybox-w32/shell/ash_test, branch su_cmd2

ash: improve trap and jobs builtins in child shells

2023-03-31T12:22:56+00:00

The trap and jobs builtins can be used to report information about traps and jobs. This works when they're called from the current shell but in a child shell the required information is usually cleared. Special hacks allow: - trap to work with command substitution; - jobs to work with command substitution or in a pipeline. Neither works with process substitution. - Relax the test for the trap hack so it also supports pipelines. - Pass the command to be evaluated to forkshell() in evalbackcmd() so trap and jobs both work with process substitution. function old new delta forkchild 629 640 +11 argstr 1502 1496 -6 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 1/1 up/down: 11/-6) Total: 5 bytes Signed-off-by: Ron Yorston Signed-off-by: Denys Vlasenko

ash: fix ifs cleanup on error paths

2022-08-02T09:18:11+00:00

Patch by Alex Gorinson function old new delta evalvar 477 495 +18 varvalue 603 618 +15 subevalvar 1557 1572 +15 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 3/0 up/down: 48/0) Total: 48 bytes Signed-off-by: Denys Vlasenko

ash: don't read past end of var in subvareval for bash substitutions

2022-03-01T07:47:43+00:00

Without this patch, BusyBox handles bash pattern substitutions without a terminating '/' character incorrectly. Consider the following shell script: _bootstrapver=5.0.211-r0 _referencesdir="/usr/${_bootstrapver/-*}/Sources" echo $_referencesdir This should output `/usr/5.0.211/Sources`. However, without this patch it instead outputs `/usr/5.0.211Sources`. This is due to the fact that BusyBox expects the bash pattern substitutions to always be terminated with a '/' (at least in this part of subvareval) and thus reads passed the substitution itself and consumes the '/' character which is part of the literal string. If there is no '/' after the substitution then BusyBox might perform an out-of-bounds read under certain circumstances. When replacing the bash pattern substitution with `${_bootstrapver/-*/}`, or with this patch applied, ash outputs the correct value. Signed-off-by: Sören Tempel Signed-off-by: Denys Vlasenko

shell: fix script's comm field if ENABLE_FEATURE_PREFER_APPLETS=y

2021-10-11T16:44:00+00:00

function old new delta re_execed_comm - 46 +46 main 72 86 +14 ------------------------------------------------------------------------------ (add/remove: 1/0 grow/shrink: 1/0 up/down: 60/0) Total: 60 bytes Signed-off-by: Denys Vlasenko

shell: fix parsing of $(( (v)++ + NUM ))

2021-09-26T11:29:25+00:00

function old new delta evaluate_string 988 1011 +23 Signed-off-by: Denys Vlasenko

shell: enable more tests which are passing now

2021-09-25T20:35:17+00:00

Signed-off-by: Denys Vlasenko

shell: fix arithmentic evaluation of "++7" and such (it is + + 7, i.e. 7)

2021-09-25T20:04:45+00:00

function old new delta evaluate_string 945 988 +43 Signed-off-by: Denys Vlasenko

ash: parser: Fix alias expansion after heredoc or newlines

2021-09-07T23:43:12+00:00

Upstream commit: Date: Wed, 29 Apr 2020 00:19:59 +1000 parser: Fix alias expansion after heredoc or newlines This script should print OK: alias a="case x in " b=x a b) echo BAD;; esac alias BEGIN={ END=} BEGIN cat <<- EOF > /dev/null $(:) EOF END : <<- EOF && $(:) EOF BEGIN echo OK END However, because the value of checkkwd is either zeroed when it shouldn't, or isn't zeroed when it should, dash currently gets it wrong in every case. This patch fixes it by saving checkkwd and zeroing it where needed. function old new delta readtoken 157 176 +19 Signed-off-by: Denys Vlasenko

shell/ash_test/run-all: unset locale/language variables

2021-09-07T16:24:08+00:00

Signed-off-by: Denys Vlasenko

ash: LINENO starts from 0 in -c SCRIPT mode

2021-09-07T16:16:45+00:00

The var_LINENO3.tests fails for hush: it does start from 0, but does not increment. Signed-off-by: Denys Vlasenko