diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2017-04-10 09:46:09 +0900 |
|---|---|---|
| committer | Yukihiro "Matz" Matsumoto <[email protected]> | 2017-04-10 09:46:09 +0900 |
| commit | 5c114c91d4ff31859fcd84cf8bf349b737b90d99 (patch) | |
| tree | 5da81f2ce21f82e2d52c1e501f81b98ce910bcf4 | |
| parent | 491d68bb3004eb8d7deec4a3a682b25de0d4afc2 (diff) | |
| download | mruby-5c114c91d4ff31859fcd84cf8bf349b737b90d99.tar.gz mruby-5c114c91d4ff31859fcd84cf8bf349b737b90d99.zip | |
Clear unused stack region that may refer freed objects; fix #3596
| -rw-r--r-- | src/gc.c | 13 |
1 files changed, 7 insertions, 6 deletions
@@ -544,6 +544,7 @@ mark_context_stack(mrb_state *mrb, struct mrb_context *c) { size_t i; size_t e; + mrb_value nil; if (c->stack == NULL) return; e = c->stack - c->stbase; @@ -553,14 +554,14 @@ mark_context_stack(mrb_state *mrb, struct mrb_context *c) mrb_value v = c->stbase[i]; if (!mrb_immediate_p(v)) { - if (mrb_basic_ptr(v)->tt == MRB_TT_FREE) { - c->stbase[i] = mrb_nil_value(); - } - else { - mrb_gc_mark(mrb, mrb_basic_ptr(v)); - } + mrb_gc_mark(mrb, mrb_basic_ptr(v)); } } + e = c->stend - c->stbase; + nil = mrb_nil_value(); + for (; i<e; i++) { + c->stbase[i] = nil; + } } static void |
