diff options
| author | Denys Vlasenko | 2023-07-04 14:35:41 +0200 |
|---|---|---|
| committer | Denys Vlasenko | 2023-07-04 14:38:25 +0200 |
| commit | 07a95cfcabb0706a22599b4440b495b6cfa8123e (patch) | |
| tree | ee074cb7517d0d2673fb3563cbd19ed4f8e5ed40 /shell/hush.c | |
| parent | e5692e2342c68092ee3d4d895ea847cf7d13fa57 (diff) | |
| download | busybox-07a95cfcabb0706a22599b4440b495b6cfa8123e.zip busybox-07a95cfcabb0706a22599b4440b495b6cfa8123e.tar.gz | |
ash: disable check for "good" function name, bash does not check this
function old new delta
.rodata 105304 105261 -43
parse_command 1696 1633 -63
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-106) Total: -106 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell/hush.c')
| -rw-r--r-- | shell/hush.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/shell/hush.c b/shell/hush.c index ec4f3a2..1b7e546 100644 --- a/shell/hush.c +++ b/shell/hush.c @@ -4306,7 +4306,7 @@ static int done_word(struct parse_context *ctx) || endofname(command->argv[0])[0] != '\0' ) { /* bash says just "not a valid identifier" */ - syntax_error("bad variable name in for"); + syntax_error("bad for loop variable"); return 1; } /* Force FOR to have just one word (variable name) */ @@ -4683,6 +4683,11 @@ static int parse_group(struct parse_context *ctx, syntax_error_unexpected_ch(ch); return -1; } +//bash allows functions named "123", "..", "return"! +// if (endofname(command->argv[0])[0] != '\0') { +// syntax_error("bad function name"); +// return -1; +// } nommu_addchr(&ctx->as_string, ch); command->cmd_type = CMD_FUNCDEF; goto skip; |