diff options
| -rw-r--r-- | ChangeLog | 5 | ||||
| -rw-r--r-- | eval_jump.c | 8 | ||||
| -rw-r--r-- | test/ruby/test_beginendblock.rb | 18 |
3 files changed, 28 insertions, 3 deletions
@@ -1,3 +1,8 @@ +Fri Apr 22 15:09:27 2016 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * eval_jump.c (exec_end_procs_chain): restore previous error info + for each end procs. [ruby-core:75038] [Bug #12302] + Fri Apr 22 15:04:56 2016 NAKAMURA Usaku <usa@ruby-lang.org> * tool/redmine-backporter.rb: the fullpath of merger.rb is too long to diff --git a/eval_jump.c b/eval_jump.c index 88fc5a2f35..59dae109ce 100644 --- a/eval_jump.c +++ b/eval_jump.c @@ -94,10 +94,11 @@ rb_mark_end_proc(void) } static void -exec_end_procs_chain(struct end_proc_data *volatile *procs) +exec_end_procs_chain(struct end_proc_data *volatile *procs, VALUE *errp) { struct end_proc_data volatile endproc; struct end_proc_data *link; + VALUE errinfo = *errp; while ((link = *procs) != 0) { *procs = link->next; @@ -105,6 +106,7 @@ exec_end_procs_chain(struct end_proc_data *volatile *procs) xfree(link); rb_set_safe_level_force(endproc.safe); (*endproc.func) (endproc.data); + *errp = errinfo; } } @@ -119,8 +121,8 @@ rb_exec_end_proc(void) TH_PUSH_TAG(th); if ((status = EXEC_TAG()) == 0) { again: - exec_end_procs_chain(&ephemeral_end_procs); - exec_end_procs_chain(&end_procs); + exec_end_procs_chain(&ephemeral_end_procs, &th->errinfo); + exec_end_procs_chain(&end_procs, &th->errinfo); } else { VAR_INITIALIZED(th); diff --git a/test/ruby/test_beginendblock.rb b/test/ruby/test_beginendblock.rb index 0c88e9eb23..a09c963b3c 100644 --- a/test/ruby/test_beginendblock.rb +++ b/test/ruby/test_beginendblock.rb @@ -130,4 +130,22 @@ at_exit { callcc {|_c| c = _c } } EOS assert_normal_exit(script, bug9110) end + + def test_errinfo_at_exit + bug12302 = '[ruby-core:75038] [Bug #12302]' + assert_in_out_err([], <<-'end;', %w[2:exit 1:exit], [], bug12302) + at_exit do + puts "1:#{$!}" + end + + at_exit do + puts "2:#{$!}" + raise 'x' rescue nil + end + + at_exit do + exit + end + end; + end end |