diff options
author | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-10-02 12:41:08 +0000 |
---|---|---|
committer | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-10-02 12:41:08 +0000 |
commit | e7f484d469a9cb880501f47b0651378dbf4444f0 (patch) | |
tree | 703d254d7a25ebf38df57e7854a7a09f6a819b95 | |
parent | 8c0033a7b625ebf8d3f809791a90512464efdbed (diff) | |
download | ruby-e7f484d469a9cb880501f47b0651378dbf4444f0.tar.gz |
* random.c (make_seed_value): a local array declaration was accessed
out of scope. Coverity Scan found this bug.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@43120 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | random.c | 2 |
2 files changed, 6 insertions, 1 deletions
@@ -1,3 +1,8 @@ +Wed Oct 2 21:37:04 2013 Yusuke Endoh <mame@tsg.ne.jp> + + * random.c (make_seed_value): a local array declaration was accessed + out of scope. Coverity Scan found this bug. + Wed Oct 2 18:52:40 2013 Koichi Sasada <ko1@atdot.net> * gc.c: relax GC condition due to malloc_limit. @@ -495,10 +495,10 @@ make_seed_value(const uint32_t *ptr) { VALUE seed; size_t len; + uint32_t buf[DEFAULT_SEED_CNT+1]; if (ptr[DEFAULT_SEED_CNT-1] <= 1) { /* set leading-zero-guard */ - uint32_t buf[DEFAULT_SEED_CNT+1]; MEMCPY(buf, ptr, uint32_t, DEFAULT_SEED_CNT); buf[DEFAULT_SEED_CNT] = 1; ptr = buf; |