diff options
| author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-07-20 23:03:23 +0000 |
|---|---|---|
| committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-07-20 23:03:23 +0000 |
| commit | 7ae1cc16b4ecec930cd776f9ed688b6d4d53a166 (patch) | |
| tree | dea5f9edc2fabb62c126930df82e7e1b4d443b73 /shell/msh.c | |
| parent | 2b68c8f74420c08ff3bb44d6745833ea715b6590 (diff) | |
| download | busybox-7ae1cc16b4ecec930cd776f9ed688b6d4d53a166.tar.gz | |
msh: fix "while...continue" bug 3884.
Diffstat (limited to 'shell/msh.c')
| -rw-r--r-- | shell/msh.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/shell/msh.c b/shell/msh.c index 44213c657..eb17eb668 100644 --- a/shell/msh.c +++ b/shell/msh.c @@ -2573,6 +2573,10 @@ static int execute(struct op *t, int *pin, int *pout, int no_fork) while (setjmp(bc.brkpt)) if (isbreak) goto broken; + /* Restore areanum value. It may be incremented by execute() + * below, and then "continue" may jump back to setjmp above */ + areanum = a + 1; + freearea(areanum + 1); brkset(&bc); for (t1 = t->left; i-- && *wp != NULL;) { setval(vp, *wp++); @@ -2586,6 +2590,10 @@ static int execute(struct op *t, int *pin, int *pout, int no_fork) while (setjmp(bc.brkpt)) if (isbreak) goto broken; + /* Restore areanum value. It may be incremented by execute() + * below, and then "continue" may jump back to setjmp above */ + areanum = a + 1; + freearea(areanum + 1); brkset(&bc); t1 = t->left; while ((execute(t1, pin, pout, /* no_fork: */ 0) == 0) == (t->op_type == TWHILE)) |