diff options
author | nagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2018-08-18 02:56:32 +0000 |
---|---|---|
committer | nagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2018-08-18 02:56:32 +0000 |
commit | 6baeed5b4fe22acf1761db879a1e9865cdd2d181 (patch) | |
tree | 738c5cc57b212a393fa95a73add3988186b9ccbf | |
parent | 7e0cafcd2b904c4bb3e86617d9628ee1296f12c0 (diff) | |
download | ruby-6baeed5b4fe22acf1761db879a1e9865cdd2d181.tar.gz |
merge revision(s) 63417: [Backport #14756]
error.c: check redefined backtrace result
* error.c (rb_get_backtrace): check the result of `backtrace` even
if the method is redefined. [ruby-core:87013] [Bug #14756]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_5@64430 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | error.c | 12 | ||||
-rw-r--r-- | test/ruby/test_exception.rb | 10 | ||||
-rw-r--r-- | version.h | 2 |
3 files changed, 18 insertions, 6 deletions
@@ -1136,19 +1136,21 @@ VALUE rb_get_backtrace(VALUE exc) { ID mid = id_backtrace; + VALUE info; if (rb_method_basic_definition_p(CLASS_OF(exc), id_backtrace)) { - VALUE info, klass = rb_eException; + VALUE klass = rb_eException; rb_execution_context_t *ec = GET_EC(); if (NIL_P(exc)) return Qnil; EXEC_EVENT_HOOK(ec, RUBY_EVENT_C_CALL, exc, mid, mid, klass, Qundef); info = exc_backtrace(exc); EXEC_EVENT_HOOK(ec, RUBY_EVENT_C_RETURN, exc, mid, mid, klass, info); - if (NIL_P(info)) - return Qnil; - return rb_check_backtrace(info); } - return rb_funcallv(exc, mid, 0, 0); + else { + info = rb_funcallv(exc, mid, 0, 0); + } + if (NIL_P(info)) return Qnil; + return rb_check_backtrace(info); } /* diff --git a/test/ruby/test_exception.rb b/test/ruby/test_exception.rb index ff1f86b5c0..862d46dc3c 100644 --- a/test/ruby/test_exception.rb +++ b/test/ruby/test_exception.rb @@ -1123,6 +1123,16 @@ $stderr = $stdout; raise "\x82\xa0"') do |outs, errs, status| raise RuntimeError, "hello" } end; + + error_class = Class.new(StandardError) do + def backtrace; :backtrace; end + end + begin + raise error_class + rescue error_class => e + assert_raise(TypeError) {$@} + assert_raise(TypeError) {e.full_message} + end end def test_full_message @@ -1,6 +1,6 @@ #define RUBY_VERSION "2.5.2" #define RUBY_RELEASE_DATE "2018-08-18" -#define RUBY_PATCHLEVEL 73 +#define RUBY_PATCHLEVEL 74 #define RUBY_RELEASE_YEAR 2018 #define RUBY_RELEASE_MONTH 8 |