diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2020-08-27 09:42:26 +0900 |
|---|---|---|
| committer | Yukihiro "Matz" Matsumoto <[email protected]> | 2020-10-12 18:20:05 +0900 |
| commit | 5134031e189e1cdde198e1c09f7b1d22bf2a1ce0 (patch) | |
| tree | c6f9ad2f670c7aa50690f073a6e756f568cbcf00 /mrbgems/mruby-random | |
| parent | befcbd5607a060e5e619cae7333512c8bdde32f6 (diff) | |
| download | mruby-5134031e189e1cdde198e1c09f7b1d22bf2a1ce0.tar.gz mruby-5134031e189e1cdde198e1c09f7b1d22bf2a1ce0.zip | |
Use `mrb_int_value()` instead of `mrb_fixnum_value()`.
Where fixnum overflow can happen.
Diffstat (limited to 'mrbgems/mruby-random')
| -rw-r--r-- | mrbgems/mruby-random/src/random.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/mrbgems/mruby-random/src/random.c b/mrbgems/mruby-random/src/random.c index 37b100084..0a5df6bd0 100644 --- a/mrbgems/mruby-random/src/random.c +++ b/mrbgems/mruby-random/src/random.c @@ -131,7 +131,7 @@ random_rand(mrb_state *mrb, rand_state *t, mrb_value max) #endif } else { - value = mrb_fixnum_value(rand_uint32(t) % mrb_integer(max)); + value = mrb_int_value(mrb, rand_uint32(t) % mrb_integer(max)); } return value; @@ -224,7 +224,7 @@ random_m_srand(mrb_state *mrb, mrb_value self) } old_seed = rand_seed(t, seed); - return mrb_fixnum_value((mrb_int)old_seed); + return mrb_int_value(mrb, (mrb_int)old_seed); } /* @@ -378,7 +378,7 @@ mrb_ary_sample(mrb_state *mrb, mrb_value ary) } break; } - mrb_ary_push(mrb, result, mrb_fixnum_value(r)); + mrb_ary_push(mrb, result, mrb_int_value(mrb, r)); } for (i=0; i<n; i++) { mrb_int idx = mrb_integer(RARRAY_PTR(result)[i]); |