diff options
author | ktsj <ktsj@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-08-20 04:26:20 +0000 |
---|---|---|
committer | ktsj <ktsj@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-08-20 04:26:20 +0000 |
commit | 5cb16a55f56220eb8a0e0bfa69c5427023a12975 (patch) | |
tree | 9c048c7059288824ab553f4125e3a50a67adcdca | |
parent | 4536f22a69d719d04eb7e597a10c48732e80607d (diff) | |
download | ruby-5cb16a55f56220eb8a0e0bfa69c5427023a12975.tar.gz |
* iseq.c (iseq_s_disasm): fix a bug that may cause SEGV.
* test/ruby/test_method.rb (test_body): add a test for the above change.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@33013 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | iseq.c | 6 | ||||
-rw-r--r-- | test/ruby/test_method.rb | 1 |
3 files changed, 10 insertions, 3 deletions
@@ -1,3 +1,9 @@ +Sat Aug 20 13:19:52 2011 Kazuki Tsujimoto <kazuki@callcc.net> + + * iseq.c (iseq_s_disasm): fix a bug that may cause SEGV. + + * test/ruby/test_method.rb (test_body): add a test for the above change. + Sat Aug 20 10:43:24 2011 Nobuyoshi Nakada <nobu@ruby-lang.org> * ext/stringio/stringio.c (strio_read): return new string if nil @@ -1030,9 +1030,9 @@ iseq_s_disasm(VALUE klass, VALUE body) rb_proc_t *proc; VALUE iseqval; GetProcPtr(body, proc); - iseqval = proc->block.iseq->self; - if (RUBY_VM_NORMAL_ISEQ_P(iseqval)) { - ret = rb_iseq_disasm(iseqval); + iseq = proc->block.iseq; + if (RUBY_VM_NORMAL_ISEQ_P(iseq)) { + ret = rb_iseq_disasm(iseq->self); } } else if ((iseq = rb_method_get_iseq(body)) != 0) { diff --git a/test/ruby/test_method.rb b/test/ruby/test_method.rb index 1da32791f7..2cc0de5409 100644 --- a/test/ruby/test_method.rb +++ b/test/ruby/test_method.rb @@ -123,6 +123,7 @@ class TestMethod < Test::Unit::TestCase def o.foo; end assert_nothing_raised { RubyVM::InstructionSequence.disasm(o.method(:foo)) } assert_nothing_raised { RubyVM::InstructionSequence.disasm("x".method(:upcase)) } + assert_nothing_raised { RubyVM::InstructionSequence.disasm(method(:to_s).to_proc) } end def test_new |