diff --git a/ChangeLog b/ChangeLog index 918fb5120..fe10578a6 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,16 @@ +Sat Mar 3 16:23:13 2007 Akinori MUSHA + + * env.h (SCOPE_CLONE): Introduce a new scope flag to prevent a + local_tbl region from getting freed many times; submitted by + Chikanaga Tomoyuki in + [ruby-dev:30460]. + + * eval.c (proc_invoke): Ditto. + + * gc.c (obj_free): Ditto. + + * parse.y (top_local_setup_gen): Ditto. + Sat Mar 3 16:07:02 2007 Akinori MUSHA * object.c (rb_obj_ivar_set): RDoc updated according to a diff --git a/env.h b/env.h index 311113323..c50103f71 100644 --- a/env.h +++ b/env.h @@ -43,6 +43,7 @@ extern struct SCOPE { #define SCOPE_MALLOC 1 #define SCOPE_NOSTACK 2 #define SCOPE_DONT_RECYCLE 4 +#define SCOPE_CLONE 8 extern int ruby_in_eval; diff --git a/eval.c b/eval.c index 598609e8c..64a43c5b5 100644 --- a/eval.c +++ b/eval.c @@ -8553,11 +8553,12 @@ proc_invoke(proc, args, self, klass) if (klass) _block.frame.last_class = klass; _block.frame.argc = RARRAY(tmp)->len; _block.frame.flags = ruby_frame->flags; - if (_block.frame.argc && (ruby_frame->flags & FRAME_DMETH)) { + if (_block.frame.argc && DMETHOD_P()) { NEWOBJ(scope, struct SCOPE); OBJSETUP(scope, tmp, T_SCOPE); scope->local_tbl = _block.scope->local_tbl; scope->local_vars = _block.scope->local_vars; + scope->flags |= SCOPE_CLONE; _block.scope = scope; } /* modify current frame */ diff --git a/gc.c b/gc.c index e3b088efd..9ac3bfbf6 100644 --- a/gc.c +++ b/gc.c @@ -1253,7 +1253,7 @@ obj_free(obj) if (RANY(obj)->as.scope.local_vars && RANY(obj)->as.scope.flags != SCOPE_ALLOCA) { VALUE *vars = RANY(obj)->as.scope.local_vars-1; - if (vars[0] == 0) + if (!(RANY(obj)->as.scope.flags & SCOPE_CLONE) && vars[0] == 0) RUBY_CRITICAL(free(RANY(obj)->as.scope.local_tbl)); if (RANY(obj)->as.scope.flags & SCOPE_MALLOC) RUBY_CRITICAL(free(vars)); diff --git a/parse.y b/parse.y index 9241220f5..c30e42315 100644 --- a/parse.y +++ b/parse.y @@ -5727,7 +5727,8 @@ top_local_setup() rb_mem_clear(ruby_scope->local_vars+i, len-i); } if (ruby_scope->local_tbl && ruby_scope->local_vars[-1] == 0) { - xfree(ruby_scope->local_tbl); + if (!(ruby_scope->flags & SCOPE_CLONE)) + xfree(ruby_scope->local_tbl); } ruby_scope->local_tbl = local_tbl(); }