diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-12-21 11:12:39 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-12-21 11:12:39 +0000 |
commit | 3603999e50b0300ffc3aa78d888a78da4c289529 (patch) | |
tree | b39f1d7b2287caaddff7f1c11dddd9372a38c54b | |
parent | c0c1f874f0d3009611c7374ba6387896bf3a2d0f (diff) | |
download | ruby-3603999e50b0300ffc3aa78d888a78da4c289529.tar.gz |
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/trunk@57137 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | eval.c | 3 | ||||
-rw-r--r-- | test/ruby/test_exception.rb | 20 |
2 files changed, 23 insertions, 0 deletions
@@ -457,6 +457,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 b397d76d3e..a6b707bcca 100644 --- a/test/ruby/test_exception.rb +++ b/test/ruby/test_exception.rb @@ -777,6 +777,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 |