summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorLuis Pires <luis.pires@eldorado.org.br>2021-09-10 08:26:03 -0300
committerDavid Gibson <david@gibson.dropbear.id.au>2021-09-29 19:37:38 +1000
commit06c0259a086f0f4ddd57a14ba811bba0b9e45130 (patch)
tree35bd44452d7511b0f06fa3152ad67b35738a4216
parent6b54a31bf7b403672a798b6443b1930ae6c74dea (diff)
downloadfocaccia-qemu-06c0259a086f0f4ddd57a14ba811bba0b9e45130.tar.gz
focaccia-qemu-06c0259a086f0f4ddd57a14ba811bba0b9e45130.zip
host-utils: Fix overflow detection in divu128()
The previous code didn't detect overflows if the high 64-bit
of the dividend were equal to the 64-bit divisor. In that case,
64 bits wouldn't be enough to hold the quotient.

Signed-off-by: Luis Pires <luis.pires@eldorado.org.br>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20210910112624.72748-2-luis.pires@eldorado.org.br>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
-rw-r--r--util/host-utils.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/util/host-utils.c b/util/host-utils.c
index 7b9322071d..a789a11b46 100644
--- a/util/host-utils.c
+++ b/util/host-utils.c
@@ -102,7 +102,7 @@ int divu128(uint64_t *plow, uint64_t *phigh, uint64_t divisor)
         *plow  = dlo / divisor;
         *phigh = dlo % divisor;
         return 0;
-    } else if (dhi > divisor) {
+    } else if (dhi >= divisor) {
         return 1;
     } else {