From 7e4015e16d3d3c5780440a9d15af4bcae8d4e4e1 Mon Sep 17 00:00:00 2001 From: nobu Date: Thu, 26 Aug 2010 00:49:54 +0000 Subject: [PATCH] * array.c (rb_ary_shuffle_bang): check number of argument. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@29100 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 4 ++++ array.c | 3 +++ test/ruby/test_array.rb | 1 + 3 files changed, 8 insertions(+) diff --git a/ChangeLog b/ChangeLog index 4865ea32c7..5381342154 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +Thu Aug 26 09:49:50 2010 Nobuyoshi Nakada + + * array.c (rb_ary_shuffle_bang): check number of argument. + Tue Aug 26 09:11:40 2010 Kenta Murata * ext/bigdecimal/bigdecimal.c (Init_bigdecimal, diff --git a/array.c b/array.c index a21f9ef51e..a2083210d3 100644 --- a/array.c +++ b/array.c @@ -3754,6 +3754,9 @@ rb_ary_shuffle_bang(int argc, VALUE *argv, VALUE ary) if (OPTHASH_GIVEN_P(opts)) { randgen = rb_hash_lookup2(opts, sym_random, randgen); } + if (argc > 0) { + rb_raise(rb_eArgError, "wrong number of arguments (%d for 0)", argc); + } rb_ary_modify(ary); ptr = RARRAY_PTR(ary); while (i) { diff --git a/test/ruby/test_array.rb b/test/ruby/test_array.rb index 4a8b700c77..4c3aba0589 100644 --- a/test/ruby/test_array.rb +++ b/test/ruby/test_array.rb @@ -1893,6 +1893,7 @@ class TestArray < Test::Unit::TestCase end gen = Random.new(0) + assert_raise(ArgumentError) {[1, 2, 3].shuffle(1, random: gen)} srand(0) 100.times do assert_equal([0, 1, 2].shuffle, [0, 1, 2].shuffle(random: gen))