From 607d93b92e374d4416ba189dd0d73ad51cdb924e Mon Sep 17 00:00:00 2001 From: usa Date: Tue, 31 Jul 2018 14:01:54 +0000 Subject: [PATCH] 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_4@64147 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- error.c | 12 +++++++----- test/ruby/test_exception.rb | 9 +++++++++ version.h | 2 +- 3 files changed, 17 insertions(+), 6 deletions(-) diff --git a/error.c b/error.c index ee805d4cc1..605d6621de 100644 --- a/error.c +++ b/error.c @@ -969,19 +969,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_thread_t *th = GET_THREAD(); if (NIL_P(exc)) return Qnil; EXEC_EVENT_HOOK(th, RUBY_EVENT_C_CALL, exc, mid, mid, klass, Qundef); info = exc_backtrace(exc); EXEC_EVENT_HOOK(th, RUBY_EVENT_C_RETURN, exc, mid, mid, klass, info); - if (NIL_P(info)) - return Qnil; - return rb_check_backtrace(info); } - return rb_funcall(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 8eb92faa08..b41ef87629 100644 --- a/test/ruby/test_exception.rb +++ b/test/ruby/test_exception.rb @@ -1066,5 +1066,14 @@ $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) {$@} + end end end diff --git a/version.h b/version.h index 19c4bf2d90..a13c7a9094 100644 --- a/version.h +++ b/version.h @@ -1,6 +1,6 @@ #define RUBY_VERSION "2.4.5" #define RUBY_RELEASE_DATE "2018-07-31" -#define RUBY_PATCHLEVEL 312 +#define RUBY_PATCHLEVEL 313 #define RUBY_RELEASE_YEAR 2018 #define RUBY_RELEASE_MONTH 7