diff options
author | usa <usa@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-12-27 11:39:47 +0000 |
---|---|---|
committer | usa <usa@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-12-27 11:39:47 +0000 |
commit | 9d693b181ce3bbd7553be36f604f5b694f8e9fea (patch) | |
tree | f216f077c57d7b13c64ae36b3acff6614c54b015 | |
parent | f82b0a39eaa1ac262db0093f441289678530e351 (diff) |
merge revision(s) 57137: [Backport #13043]
eval.c: fix circular cause
* eval.c (exc_setup_cause): always set cause of cause to get rid
of circular references. [ruby-core:78688] [Bug #13043]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_2@57222 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | eval.c | 3 | ||||
-rw-r--r-- | test/ruby/test_exception.rb | 20 | ||||
-rw-r--r-- | version.h | 2 |
4 files changed, 29 insertions, 1 deletions
@@ -1,3 +1,8 @@ +Tue Dec 27 20:39:28 2016 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * eval.c (exc_setup_cause): always set cause of cause to get rid of + circular references. [Bug #13043] + Tue Dec 27 20:13:12 2016 Kazuki Yamaguchi <k@rhe.jp> * re.c (match_{names,hash,equal}): consider the case of RMatch::regexp @@ -468,6 +468,9 @@ exc_setup_cause(VALUE exc, VALUE cause) #endif if (!NIL_P(cause) && cause != exc) { rb_ivar_set(exc, id_cause, cause); + if (!rb_ivar_defined(cause, id_cause)) { + rb_ivar_set(cause, id_cause, Qnil); + } } return exc; } diff --git a/test/ruby/test_exception.rb b/test/ruby/test_exception.rb index 3268096463..0e0179fd5f 100644 --- a/test/ruby/test_exception.rb +++ b/test/ruby/test_exception.rb @@ -717,6 +717,26 @@ end.join assert_equal({}, e.arg, bug) end + def test_circular_cause + bug13043 = '[ruby-core:78688] [Bug #13043]' + begin + begin + raise "error 1" + ensure + orig_error = $! + begin + raise "error 2" + rescue => err + raise orig_error + end + end + rescue => x + end + assert_equal(orig_error, x) + assert_equal(orig_error, err.cause) + assert_nil(orig_error.cause, bug13043) + end + def test_anonymous_message assert_in_out_err([], "raise Class.new(RuntimeError), 'foo'", [], /foo\n/) end @@ -1,6 +1,6 @@ #define RUBY_VERSION "2.2.7" #define RUBY_RELEASE_DATE "2016-12-27" -#define RUBY_PATCHLEVEL 412 +#define RUBY_PATCHLEVEL 413 #define RUBY_RELEASE_YEAR 2016 #define RUBY_RELEASE_MONTH 12 |