tv_scale(): Avoid integer overflow.
authorAndre Noll <maan@systemlinux.org>
Sat, 19 Sep 2009 08:11:19 +0000 (10:11 +0200)
committerAndre Noll <maan@systemlinux.org>
Sat, 19 Sep 2009 08:11:19 +0000 (10:11 +0200)
Just use an uint64_t and compute everything in microseconds which even
simplifies the code a bit.

time.c

diff --git a/time.c b/time.c
index 6e1d603..0cb9bab 100644 (file)
--- a/time.c
+++ b/time.c
@@ -106,9 +106,10 @@ void tv_add(const struct timeval *a, const struct timeval *b,
 void tv_scale(const unsigned long mult, const struct timeval *tv,
        struct timeval *result)
 {
-       result->tv_sec = mult * tv->tv_sec;
-       result->tv_sec += tv->tv_usec * mult / 1000 / 1000;
-       result->tv_usec = tv->tv_usec * mult % (1000 * 1000);
+       uint64_t x = ((uint64_t)tv->tv_sec * 1000 * 1000 + tv->tv_usec) * mult;
+
+       result->tv_sec = x / 1000 / 1000;
+       result->tv_usec = x % (1000 * 1000);
 }
 
 /**