diff options
| author | Denys Vlasenko | 2017-07-25 15:18:57 +0200 |
|---|---|---|
| committer | Denys Vlasenko | 2017-07-25 15:18:57 +0200 |
| commit | 645c697372b714f1293a37a185aa62965f600479 (patch) | |
| tree | 5033362b96737f83526b776818bf4c9a4ef8a9ef /shell/hush.c | |
| parent | 64925384c9cf5e0d986e183577da286bb3207ce7 (diff) | |
| download | busybox-645c697372b714f1293a37a185aa62965f600479.zip busybox-645c697372b714f1293a37a185aa62965f600479.tar.gz | |
hush: treat ${#?} as "length of $?"
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell/hush.c')
| -rw-r--r-- | shell/hush.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/shell/hush.c b/shell/hush.c index 20b0923..f9dad07 100644 --- a/shell/hush.c +++ b/shell/hush.c @@ -5559,8 +5559,10 @@ static NOINLINE const char *expand_one_var(char **to_be_freed_pp, char *arg, cha first_char = arg[0] = arg0 & 0x7f; exp_op = 0; - if (first_char == '#' /* ${#... */ - && arg[1] && !exp_saveptr /* not ${#} and not ${#<op_char>...} */ + if (first_char == '#' && arg[1] /* ${#... but not ${#} */ + && (!exp_saveptr /* and (not ${#<op_char>...} */ + || (arg[1] == '?' && arg[2] == '\0') /* or ${#?} - "len of $?") */ + ) ) { /* It must be length operator: ${#var} */ var++; @@ -5797,7 +5799,11 @@ static NOINLINE const char *expand_one_var(char **to_be_freed_pp, char *arg, cha /* mimic bash message */ die_if_script("%s: %s", var, - exp_word[0] ? exp_word : "parameter null or not set" + exp_word[0] + ? exp_word + : "parameter null or not set" + /* ash has more specific messages, a-la: */ + /*: (exp_save == ':' ? "parameter null or not set" : "parameter not set")*/ ); //TODO: how interactive bash aborts expansion mid-command? } else { |