diff options
author | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-07-21 14:51:17 +0000 |
---|---|---|
committer | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-07-21 14:51:17 +0000 |
commit | 4516f623b8628e8b424197cf29c786172f23e1ad (patch) | |
tree | 9e8835e2e2a3334b5e56072a35f6eddf3de732b6 | |
parent | 7203e59f275dfe6f0b6e6f301673400565c5e0dc (diff) | |
download | ruby-4516f623b8628e8b424197cf29c786172f23e1ad.tar.gz |
* random.c (rand_init): array length of random seed was broken, which
causes memory error with srand(2**1000000-1).
* test/ruby/test_rand.c: test for above.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@24228 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | random.c | 6 | ||||
-rw-r--r-- | test/ruby/test_rand.rb | 5 |
3 files changed, 14 insertions, 4 deletions
@@ -1,3 +1,10 @@ +Tue Jul 21 23:47:38 2009 Yusuke Endoh <mame@tsg.ne.jp> + + * random.c (rand_init): array length of random seed was broken, which + causes memory error with srand(2**1000000-1). + + * test/ruby/test_rand.c: test for above. + Tue Jul 21 21:37:19 2009 Keiju Ishitsuka <keiju@emperor2.pendome> * lib/irb/cmd/help.rb: fixed irb's "help" command. [ruby-core:22310]. @@ -304,11 +304,9 @@ rand_init(struct MT *mt, VALUE vseed) if (blen == 0) { len = 1; } - else if (blen > MT_MAX_STATE * SIZEOF_INT32 / SIZEOF_BDIGITS) { - blen = (len = MT_MAX_STATE) * SIZEOF_INT32 / SIZEOF_BDIGITS; - len = roomof(len, SIZEOF_INT32); - } else { + if (blen > MT_MAX_STATE * SIZEOF_INT32 / SIZEOF_BDIGITS) + blen = (len = MT_MAX_STATE) * SIZEOF_INT32 / SIZEOF_BDIGITS; len = roomof((int)blen * SIZEOF_BDIGITS, SIZEOF_INT32); } /* allocate ints for init_by_array */ diff --git a/test/ruby/test_rand.rb b/test/ruby/test_rand.rb index 7bfd96f355..140851d5ba 100644 --- a/test/ruby/test_rand.rb +++ b/test/ruby/test_rand.rb @@ -164,4 +164,9 @@ class TestRand < Test::Unit::TestCase srand(0) assert_equal([1,4,2,5,3], [1,2,3,4,5].shuffle) end + + def test_big_seed + srand(2**1000000-1) + assert_equal(1143843490, rand(0x100000000)) + end end |