summaryrefslogtreecommitdiffhomepage
path: root/include
diff options
context:
space:
mode:
authorYukihiro "Matz" Matsumoto <[email protected]>2020-09-07 16:22:01 +0900
committerYukihiro "Matz" Matsumoto <[email protected]>2020-10-12 18:20:22 +0900
commit3ff2757bf705082e40013796ef2bac1277499de8 (patch)
tree35765477da5408a63198c8d1fa9b651b0b1e8bfd /include
parent4dcc1819c9a5d2ad8c786dd4c7443c048c639575 (diff)
downloadmruby-3ff2757bf705082e40013796ef2bac1277499de8.tar.gz
mruby-3ff2757bf705082e40013796ef2bac1277499de8.zip
Fix `mrb_int_mul_overflow()` to check either operand being zero.
Diffstat (limited to 'include')
-rw-r--r--include/mruby/numeric.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/include/mruby/numeric.h b/include/mruby/numeric.h
index f88868796..7e70b9dee 100644
--- a/include/mruby/numeric.h
+++ b/include/mruby/numeric.h
@@ -136,15 +136,15 @@ mrb_int_mul_overflow(mrb_int multiplier, mrb_int multiplicand, mrb_int *product)
if (multiplicand > 0) {
if (multiplier > MRB_INT_MAX / multiplicand) return TRUE;
}
- else {
+ else if (multiplicand < 0) {
if (multiplicand < MRB_INT_MAX / multiplier) return TRUE;
}
}
- else {
+ else if (multiplier < 0) {
if (multiplicand > 0) {
if (multiplier < MRB_INT_MAX / multiplicand) return TRUE;
}
- else {
+ else if (multiplicand < 0) {
if (multiplier != 0 && multiplicand < MRB_INT_MAX / multiplier) return TRUE;
}
}