Fix off-by-one error in OP_EPUSH and OP_ONERR; fix #4095

This commit is contained in:
Yukihiro "Matz" Matsumoto
2018-08-29 23:59:20 +09:00
parent e91b6a0378
commit f76cc483d5
+2 -2
View File
@@ -1240,7 +1240,7 @@ RETRY_TRY_BLOCK:
CASE(OP_ONERR, S) {
/* check rescue stack */
if (mrb->c->ci->ridx == UINT16_MAX) {
if (mrb->c->ci->ridx == UINT16_MAX-1) {
mrb_value exc = mrb_exc_new_str_lit(mrb, E_RUNTIME_ERROR, "too many nested rescues");
mrb_exc_set(mrb, exc);
goto L_RAISE;
@@ -1306,7 +1306,7 @@ RETRY_TRY_BLOCK:
p = mrb_closure_new(mrb, irep->reps[a]);
/* check ensure stack */
if (mrb->c->eidx == UINT16_MAX) {
if (mrb->c->eidx == UINT16_MAX-1) {
mrb_value exc = mrb_exc_new_str_lit(mrb, E_RUNTIME_ERROR, "too many nested ensures");
mrb_exc_set(mrb, exc);
goto L_RAISE;