diff options
author | shugo <shugo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-11-05 16:42:27 +0000 |
---|---|---|
committer | shugo <shugo@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-11-05 16:42:27 +0000 |
commit | 0a407acb5636c3ee0884e9c5da49122496c8b5d7 (patch) | |
tree | 47767a2da8bc746b269f4df18d93f1d6814ab000 | |
parent | cb62a5d7f7606d899ca4bcb31cb102171962f9fa (diff) | |
download | ruby-0a407acb5636c3ee0884e9c5da49122496c8b5d7.tar.gz |
* compile.c (tailcallable_p): disable tail call optimization for
toplevel, eval, and load to avoid SEGV when interrupted by SIGINT.
[ruby-core:76327] [Bug #12576]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@56610 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | compile.c | 4 | ||||
-rw-r--r-- | test/ruby/test_optimization.rb | 27 |
3 files changed, 37 insertions, 0 deletions
@@ -1,3 +1,9 @@ +Sun Nov 6 01:39:36 2016 Shugo Maeda <shugo@ruby-lang.org> + + * compile.c (tailcallable_p): disable tail call optimization for + toplevel, eval, and load to avoid SEGV when interrupted by SIGINT. + [ruby-core:76327] [Bug #12576] + Sun Nov 6 01:28:41 2016 Koichi Sasada <ko1@atdot.net> * vm_insnhelper.h: rename REG_(CFP|PC|SP|EP) to VM_REG_.... @@ -2423,6 +2423,10 @@ static inline int tailcallable_p(rb_iseq_t *iseq) { switch (iseq->body->type) { + case ISEQ_TYPE_TOP: + case ISEQ_TYPE_EVAL: + case ISEQ_TYPE_MAIN: + /* not tail callable because cfp will be over popped */ case ISEQ_TYPE_RESCUE: case ISEQ_TYPE_ENSURE: /* rescue block can't tail call because of errinfo */ diff --git a/test/ruby/test_optimization.rb b/test/ruby/test_optimization.rb index 89a9753f0e..c035a112cb 100644 --- a/test/ruby/test_optimization.rb +++ b/test/ruby/test_optimization.rb @@ -316,6 +316,33 @@ class TestRubyOptimization < Test::Unit::TestCase message(bug12565) {disasm(:add_one_and_two)}) end + def test_tailcall_interrupted_by_sigint + bug = 'ruby-core:76327' + script = <<EOS +RubyVM::InstructionSequence.compile_option = { + :tailcall_optimization => true, + :trace_instruction => false +} + +eval <<EOF +def foo + foo +end +foo +EOF +EOS + err = EnvUtil.invoke_ruby([], "", true, true, {}) { + |in_p, out_p, err_p, pid| + in_p.write(script) + in_p.close + sleep(1) + Process.kill(:SIGINT, pid) + Process.wait(pid) + err_p.read + } + assert_match(/Interrupt/, err, bug) + end + class Bug10557 def [](_) block_given? |