diff options
| -rw-r--r-- | ChangeLog | 5 | ||||
| -rw-r--r-- | test/ruby/test_module.rb | 11 | ||||
| -rw-r--r-- | vm_method.c | 1 |
3 files changed, 16 insertions, 1 deletions
@@ -1,3 +1,8 @@ +Sun Nov 15 18:28:43 2015 Kenichi Kamiya <kachick1@gmail.com> + + * vm_method.c (set_method_visibility): should fail if the receiver + is frozen. [ruby-core:71489] [Bug #11687] + Sat Nov 14 22:15:07 2015 Tanaka Akira <akr@fsij.org> * ext/socket/lib/socket.rb: Specify frozen_string_literal: true. diff --git a/test/ruby/test_module.rb b/test/ruby/test_module.rb index 19cf418cf1..dbbeca850d 100644 --- a/test/ruby/test_module.rb +++ b/test/ruby/test_module.rb @@ -624,13 +624,22 @@ class TestModule < Test::Unit::TestCase end def test_freeze - m = Module.new + m = Module.new do + def self.baz; end + def bar; end + end m.freeze assert_raise(RuntimeError) do m.module_eval do def foo; end end end + assert_raise(RuntimeError) do + m.__send__ :private, :bar + end + assert_raise(RuntimeError) do + m.private_class_method :baz + end end def test_attr_obsoleted_flag diff --git a/vm_method.c b/vm_method.c index 1bc499f1b5..c88167434b 100644 --- a/vm_method.c +++ b/vm_method.c @@ -1552,6 +1552,7 @@ set_method_visibility(VALUE self, int argc, const VALUE *argv, rb_method_visibil { int i; + rb_check_frozen(self); if (argc == 0) { rb_warning("%"PRIsVALUE" with no argument is just ignored", QUOTE_ID(rb_frame_callee())); |