Ensure that the integer type used in expressions involving
multiplication with NSEC_PER_SEC is large enough to avoid overflows on
32-bit systems. In the expressions fixed by this patch a 64-bit type
(long long) is required.

Signed-off-by: Alexandros Frantzis <alexandros.frant...@collabora.com>
---
 tests/timespec-test.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/timespec-test.c b/tests/timespec-test.c
index f127bcee..31a6f146 100644
--- a/tests/timespec-test.c
+++ b/tests/timespec-test.c
@@ -160,7 +160,7 @@ ZUC_TEST(timespec_test, timespec_sub_to_nsec)
        a.tv_nsec = 1;
        b.tv_sec = 1;
        b.tv_nsec = 2;
-       ZUC_ASSERT_EQ((999L * NSEC_PER_SEC) - 1, timespec_sub_to_nsec(&a, &b));
+       ZUC_ASSERT_EQ((999LL * NSEC_PER_SEC) - 1, timespec_sub_to_nsec(&a, &b));
 }
 
 ZUC_TEST(timespec_test, timespec_sub_to_msec)
@@ -190,7 +190,7 @@ ZUC_TEST(timespec_test, timespec_from_nsec)
        ZUC_ASSERT_EQ(1, a.tv_sec);
        ZUC_ASSERT_EQ(0, a.tv_nsec);
 
-       timespec_from_nsec(&a, (5L * NSEC_PER_SEC) + 1);
+       timespec_from_nsec(&a, (5LL * NSEC_PER_SEC) + 1);
        ZUC_ASSERT_EQ(5, a.tv_sec);
        ZUC_ASSERT_EQ(1, a.tv_nsec);
 }
-- 
2.14.1

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to