diff options
| author | Christopher Aue <[email protected]> | 2017-08-12 23:59:35 +0200 |
|---|---|---|
| committer | Christopher Aue <[email protected]> | 2017-08-12 23:59:40 +0200 |
| commit | d5ac785b315530bcdba7b0a5bf9fd49f1daaaad7 (patch) | |
| tree | adce38dfcc6249d69f1ac4c43d44a35e5ae8eed3 /src | |
| parent | 198df6103924149d74f9668b5bb63c2e8e9e79ab (diff) | |
| download | mruby-d5ac785b315530bcdba7b0a5bf9fd49f1daaaad7.tar.gz mruby-d5ac785b315530bcdba7b0a5bf9fd49f1daaaad7.zip | |
Reintroduced not storing converted proc directly in the stack
Diffstat (limited to 'src')
| -rw-r--r-- | src/vm.c | 10 |
1 files changed, 8 insertions, 2 deletions
@@ -1345,7 +1345,10 @@ RETRY_TRY_BLOCK: else { blk = regs[bidx]; if (!mrb_nil_p(blk) && mrb_type(blk) != MRB_TT_PROC) { - blk = regs[bidx] = mrb_convert_type(mrb, blk, MRB_TT_PROC, "Proc", "to_proc"); + /* store the converted proc not directly in the stack because the stack + might have been reallocated during mrb_convert_type(), see #3622 */ + blk = mrb_convert_type(mrb, blk, MRB_TT_PROC, "Proc", "to_proc"); + regs[bidx] = blk; } } c = mrb_class(mrb, recv); @@ -1528,7 +1531,10 @@ RETRY_TRY_BLOCK: recv = regs[0]; blk = regs[bidx]; if (!mrb_nil_p(blk) && mrb_type(blk) != MRB_TT_PROC) { - blk = regs[bidx] = mrb_convert_type(mrb, blk, MRB_TT_PROC, "Proc", "to_proc"); + /* store the converted proc not directly in the stack because the stack + might have been reallocated during mrb_convert_type(), see #3622 */ + blk = mrb_convert_type(mrb, blk, MRB_TT_PROC, "Proc", "to_proc"); + regs[bidx] = blk; } c = ci->target_class->super; m = mrb_method_search_vm(mrb, &c, mid); |
