diff options
| author |   deraadt <deraadt@openbsd.org> | 2009-06-01 18:20:33 +0000 |
|---|---|---|
| committer | deraadt <deraadt@openbsd.org> | 2009-06-01 18:20:33 +0000 |
| commit | 795a79f78d94ab3cd3a1275cfa44c3bf5416e8b8 (patch) | |
| tree | c45e37da143c52e6492c044a8c4a19d92e35a2c5 | |
| parent | Sort argument list of udl_draw_char similar as udl_fb_* friends. (diff) | |
| download | wireguard-openbsd-795a79f78d94ab3cd3a1275cfa44c3bf5416e8b8.tar.xz wireguard-openbsd-795a79f78d94ab3cd3a1275cfa44c3bf5416e8b8.zip | |
"let --" was crashing ksh; found by phy0@rambler.ru. Various other expressions
involving ++ and -- also ran into this. Insufficient checks for end of parse in
the tokenizer made it assume that an lvalue had been found
ok millert otto
| -rw-r--r-- | bin/ksh/expr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/bin/ksh/expr.c b/bin/ksh/expr.c index 4ee87318073..13474fda656 100644 --- a/bin/ksh/expr.c +++ b/bin/ksh/expr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: expr.c,v 1.19 2006/04/10 14:38:59 jaredy Exp $ */ +/* $OpenBSD: expr.c,v 1.20 2009/06/01 18:20:33 deraadt Exp $ */ /* * Korn expression evaluation @@ -541,7 +541,7 @@ do_ppmm(Expr_state *es, enum token op, struct tbl *vasn, bool is_prefix) static void assign_check(Expr_state *es, enum token op, struct tbl *vasn) { - if (vasn->name[0] == '\0' && !(vasn->flag & EXPRLVALUE)) + if (es->tok == END || (vasn->name[0] == '\0' && !(vasn->flag & EXPRLVALUE))) evalerr(es, ET_LVALUE, opinfo[(int) op].name); else if (vasn->flag & RDONLY) evalerr(es, ET_RDONLY, opinfo[(int) op].name); |