diff options
author | usa <usa@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2017-08-09 08:31:28 +0000 |
---|---|---|
committer | usa <usa@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2017-08-09 08:31:28 +0000 |
commit | 2132fb9fd1fb8d32e5f1ae681fafbf295b91eac5 (patch) | |
tree | f3b231a3381a88dc086a19459ae745b6a82deec4 | |
parent | f6d9f6fd2a7ad2f20472f3704ae7d4e810627c54 (diff) |
merge revision(s) 57415,57474: [Backport #13239]
eval.c: copy special exception
* eval.c (setup_exception): make unfrozen copy of special
exception before setting up a cause.
test_io.rb: separate a test
* test/ruby/test_io.rb (test_closed_stream_in_rescue): run in a
separated process.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_3@59538 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | eval.c | 17 | ||||
-rw-r--r-- | internal.h | 2 | ||||
-rw-r--r-- | test/ruby/test_io.rb | 23 | ||||
-rw-r--r-- | version.h | 2 | ||||
-rw-r--r-- | vm_insnhelper.c | 8 |
6 files changed, 48 insertions, 9 deletions
@@ -1,3 +1,8 @@ +Wed Aug 9 17:28:35 2017 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * eval.c (setup_exception): make unfrozen copy of special + exception before setting up a cause. + Wed Aug 9 17:22:29 2017 TAKANO `takano32' Mitsuhiro <tak@no32.tk> a64: fix crash on register stack mark/sweep pass @@ -471,6 +471,17 @@ sysstack_error_p(VALUE exc) return exc == sysstack_error || (!SPECIAL_CONST_P(exc) && RBASIC_CLASS(exc) == rb_eSysStackError); } +static inline int +special_exception_p(rb_thread_t *th, VALUE exc) +{ + enum ruby_special_exceptions i; + const VALUE *exceptions = th->vm->special_exceptions; + for (i = 0; i < ruby_special_error_count; ++i) { + if (exceptions[i] == exc) return TRUE; + } + return FALSE; +} + static void setup_exception(rb_thread_t *th, int tag, volatile VALUE mesg, VALUE cause) { @@ -488,6 +499,9 @@ setup_exception(rb_thread_t *th, int tag, volatile VALUE mesg, VALUE cause) mesg = rb_exc_new(rb_eRuntimeError, 0, 0); nocause = 0; } + else if (special_exception_p(th, mesg)) { + mesg = ruby_vm_special_exception_copy(mesg); + } if (cause != Qundef) { mesg = exc_setup_cause(mesg, cause); } @@ -504,9 +518,6 @@ setup_exception(rb_thread_t *th, int tag, volatile VALUE mesg, VALUE cause) if (sysstack_error_p(mesg)) { if (NIL_P(rb_attr_get(mesg, idBt))) { at = rb_vm_backtrace_object(); - if (mesg == sysstack_error) { - mesg = ruby_vm_sysstack_error_copy(); - } rb_ivar_set(mesg, idBt, at); rb_ivar_set(mesg, idBt_locations, at); } diff --git a/internal.h b/internal.h index 6eb01d8b74..8eadb3d5b1 100644 --- a/internal.h +++ b/internal.h @@ -1239,7 +1239,7 @@ void rb_vm_pop_cfunc_frame(void); int rb_vm_add_root_module(ID id, VALUE module); void rb_vm_check_redefinition_by_prepend(VALUE klass); VALUE rb_yield_refine_block(VALUE refinement, VALUE refinements); -VALUE ruby_vm_sysstack_error_copy(void); +VALUE ruby_vm_special_exception_copy(VALUE); /* vm_dump.c */ void rb_print_backtrace(void); diff --git a/test/ruby/test_io.rb b/test/ruby/test_io.rb index 9a443318b4..4dd515da39 100644 --- a/test/ruby/test_io.rb +++ b/test/ruby/test_io.rb @@ -3357,5 +3357,28 @@ End end assert_equal(true, closed, "#{bug13158}: stream should be closed") end + + def test_closed_stream_in_rescue + assert_separately([], "#{<<-"begin;"}\n#{<<~"end;"}") + begin; + 10.times do + assert_nothing_raised(RuntimeError, /frozen IOError/) do + IO.pipe do |r, w| + th = Thread.start {r.close} + r.gets + rescue IOError + # swallow pending exceptions + begin + sleep 0.001 + rescue IOError + retry + end + ensure + th.kill.join + end + end + end + end; + end end end @@ -1,6 +1,6 @@ #define RUBY_VERSION "2.3.5" #define RUBY_RELEASE_DATE "2017-08-09" -#define RUBY_PATCHLEVEL 348 +#define RUBY_PATCHLEVEL 349 #define RUBY_RELEASE_YEAR 2017 #define RUBY_RELEASE_MONTH 8 diff --git a/vm_insnhelper.c b/vm_insnhelper.c index b90621bd1f..2f786e3c84 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -25,17 +25,17 @@ static rb_control_frame_t *vm_get_ruby_level_caller_cfp(const rb_thread_t *th, const rb_control_frame_t *cfp); VALUE -ruby_vm_sysstack_error_copy(void) +ruby_vm_special_exception_copy(VALUE exc) { - VALUE e = rb_obj_alloc(rb_eSysStackError); - rb_obj_copy_ivar(e, sysstack_error); + VALUE e = rb_obj_alloc(rb_class_real(RBASIC_CLASS(exc))); + rb_obj_copy_ivar(e, exc); return e; } static void vm_stackoverflow(void) { - rb_exc_raise(ruby_vm_sysstack_error_copy()); + rb_exc_raise(ruby_vm_special_exception_copy(sysstack_error)); } #if VM_CHECK_MODE > 0 |