summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2018-08-18 02:56:32 +0000
committernagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2018-08-18 02:56:32 +0000
commit6baeed5b4fe22acf1761db879a1e9865cdd2d181 (patch)
tree738c5cc57b212a393fa95a73add3988186b9ccbf
parent7e0cafcd2b904c4bb3e86617d9628ee1296f12c0 (diff)
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.c12
-rw-r--r--test/ruby/test_exception.rb10
-rw-r--r--version.h2
3 files changed, 18 insertions, 6 deletions
diff --git a/error.c b/error.c
index 9255019139..079f01d6a6 100644
--- a/error.c
+++ b/error.c
@@ -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
diff --git a/version.h b/version.h
index 3e58f5f73f..f2cb8aac30 100644
--- a/version.h
+++ b/version.h
@@ -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