diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2016-11-25 09:19:22 +0900 |
|---|---|---|
| committer | GitHub <[email protected]> | 2016-11-25 09:19:22 +0900 |
| commit | 6905597f5b750e75e3320a5c9a74e4c270f155eb (patch) | |
| tree | 8c55faa3cff3501aa85d2f636905a6d9c80aef09 /test | |
| parent | a28fa35ea9de58e586441cbfdbfb0b7ba86aea99 (diff) | |
| parent | 22f550405449faf5bb424f04a52835376add2ae2 (diff) | |
| download | mruby-6905597f5b750e75e3320a5c9a74e4c270f155eb.tar.gz mruby-6905597f5b750e75e3320a5c9a74e4c270f155eb.zip | |
Merge pull request #3284 from bouk/remove-method-segfault
Fix segfault on remove_method with invalid argument
Diffstat (limited to 'test')
| -rw-r--r-- | test/t/class.rb | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/test/t/class.rb b/test/t/class.rb index 7bcaaf90d..597999d3e 100644 --- a/test/t/class.rb +++ b/test/t/class.rb @@ -401,3 +401,12 @@ assert('class with non-class/module outer raises TypeError') do assert_raise(TypeError) { class 0::C1; end } assert_raise(TypeError) { class []::C2; end } end + +assert("remove_method doesn't segfault if the passed in argument isn't a symbol") do + klass = Class.new + assert_raise(TypeError) { klass.remove_method nil } + assert_raise(TypeError) { klass.remove_method 123 } + assert_raise(TypeError) { klass.remove_method 1.23 } + assert_raise(NameError) { klass.remove_method "hello" } + assert_raise(TypeError) { klass.remove_method Class.new } +end |