From 75e7f2ff2d23c8a148558c71c88157635ba3fda3 Mon Sep 17 00:00:00 2001 From: nagachika Date: Sun, 12 Jan 2014 14:23:53 +0000 Subject: merge revision(s) 44260: [Backport #9168] patch inspired from r44260 on trunk. [ruby-core:58652] [Bug #9168] * gc.c (run_finalizer): clear th->errinfo before invoke finalizer and restore afterward. * test/ruby/test_gc.rb (test_exception_in_finalizer): add test for above. * vm_trace.c (rb_threadptr_exec_event_hooks_orig): clear th->errinfo before invoke hook. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_0_0@44576 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 13 +++++++++++++ gc.c | 3 +++ test/ruby/test_gc.rb | 13 +++++++++++++ version.h | 2 +- vm_trace.c | 1 + 5 files changed, 31 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 12afcc0f1a..d484a5a253 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,16 @@ +Sun Jan 12 22:46:49 2014 CHIKANAGA Tomoyuki + + patch inspired from r44260 on trunk. [ruby-core:58652] [Bug #9168] + + * gc.c (run_finalizer): clear th->errinfo before invoke finalizer and + restore afterward. + + * test/ruby/test_gc.rb (test_exception_in_finalizer): add test for + above. + + * vm_trace.c (rb_threadptr_exec_event_hooks_orig): clear th->errinfo + before invoke hook. + Sun Jan 12 16:07:52 2014 vo.x (Vit Ondruch) * tool/rbinstall.rb (Gem::Specification#collect): make stable diff --git a/gc.c b/gc.c index a654e2ec5c..71fd2ced8f 100644 --- a/gc.c +++ b/gc.c @@ -1375,6 +1375,7 @@ run_finalizer(rb_objspace_t *objspace, VALUE obj, VALUE table) int status; VALUE args[3]; VALUE objid = nonspecial_obj_id(obj); + VALUE saved_errinfo = rb_errinfo(); if (RARRAY_LEN(table) > 0) { args[1] = rb_obj_freeze(rb_ary_new3(1, objid)); @@ -1384,6 +1385,7 @@ run_finalizer(rb_objspace_t *objspace, VALUE obj, VALUE table) } args[2] = (VALUE)rb_safe_level(); + rb_set_errinfo(Qnil); for (i=0; ierrinfo = saved_errinfo; } static void diff --git a/test/ruby/test_gc.rb b/test/ruby/test_gc.rb index c678c3c6c6..b7de62b195 100644 --- a/test/ruby/test_gc.rb +++ b/test/ruby/test_gc.rb @@ -174,4 +174,17 @@ class TestGc < Test::Unit::TestCase end; end end + + def test_exception_in_finalizer + bug9168 = '[ruby-core:58652] [Bug #9168]' + assert_normal_exit(<<-'end;', bug9168) + raise_proc = proc {raise} + 10000.times do + ObjectSpace.define_finalizer(Object.new, raise_proc) + Thread.handle_interrupt(RuntimeError => :immediate) {break} + Thread.handle_interrupt(RuntimeError => :on_blocking) {break} + Thread.handle_interrupt(RuntimeError => :never) {break} + end + end; + end end diff --git a/version.h b/version.h index a3fb76ca08..9d655c55c9 100644 --- a/version.h +++ b/version.h @@ -1,6 +1,6 @@ #define RUBY_VERSION "2.0.0" #define RUBY_RELEASE_DATE "2014-01-12" -#define RUBY_PATCHLEVEL 383 +#define RUBY_PATCHLEVEL 384 #define RUBY_RELEASE_YEAR 2014 #define RUBY_RELEASE_MONTH 1 diff --git a/vm_trace.c b/vm_trace.c index 0d6e43c786..2b21426e27 100644 --- a/vm_trace.c +++ b/vm_trace.c @@ -290,6 +290,7 @@ rb_threadptr_exec_event_hooks_orig(rb_trace_arg_t *trace_arg, int pop_p) const int outer_state = th->state; int state = 0; th->state = 0; + th->errinfo = Qnil; th->vm->trace_running++; th->trace_arg = trace_arg; -- cgit v1.2.3