diff options
| author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-07-18 07:57:45 +0000 |
|---|---|---|
| committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-07-18 07:57:45 +0000 |
| commit | a074cd84921032ad30635f3c4d9263565879df5b (patch) | |
| tree | 35d582eebab3963033b6e2698feab1aa7c124269 | |
| parent | e873e9269be48d08e1c42ba46240e949006db5ac (diff) | |
| download | ruby-a074cd84921032ad30635f3c4d9263565879df5b.tar.gz ruby-a074cd84921032ad30635f3c4d9263565879df5b.tar.xz ruby-a074cd84921032ad30635f3c4d9263565879df5b.zip | |
* bignum.c (rb_big_pow): refine overflow check. [ruby-dev:31242]
git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@12813 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
| -rw-r--r-- | ChangeLog | 4 | ||||
| -rw-r--r-- | bignum.c | 4 |
2 files changed, 7 insertions, 1 deletions
@@ -1,3 +1,7 @@ +Wed Jul 18 16:57:41 2007 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * bignum.c (rb_big_pow): refine overflow check. [ruby-dev:31242] + Wed Jul 18 09:19:07 2007 Nobuyoshi Nakada <nobu@ruby-lang.org> * parse.y (rb_parser_append_print, rb_parser_while_loop): moved check @@ -1675,8 +1675,10 @@ rb_big_pow(VALUE x, VALUE y) if (yy > 0) { VALUE z = 0; SIGNED_VALUE mask; + const long BIGLEN_LIMIT = 1024*1024 / SIZEOF_BDIGITS; - if (RBIGNUM(x)->len * SIZEOF_BDIGITS * yy > 1024*1024) { + if ((RBIGNUM(x)->len > BIGLEN_LIMIT) || + (RBIGNUM(x)->len > BIGLEN_LIMIT / yy)) { rb_warn("in a**b, b may be too big"); d = (double)yy; break; |