[LTP] [PATCH] common_timers: define NSEC_PER_SEC as long long to avoid overflow on 32-bit
From: Thadeu Lima de Souza Cascardo <hidden>
Date: 2021-06-09 14:38:49
Subsystem:
the rest · Maintainer:
Linus Torvalds
From: Thadeu Lima de Souza Cascardo <hidden>
Date: 2021-06-09 14:38:49
Subsystem:
the rest · Maintainer:
Linus Torvalds
When multiplying 30 with NSEC_PER_SEC, the result would overflow on 32-bit platforms, unless there was promotion to long long, which is what tst_ts_from_ns expects. futex_wake04, which uses that, would end up getting EINVAL when calling futex, because timespec_valid would fail, as tv_nsec would have an invalid value. After this change, futex_wake04 passes on i386. Reported-by: Po-Hsu Lin <redacted> Suggested-by: Cyril Hrubis <chrubis@suse.cz> Signed-off-by: Thadeu Lima de Souza Cascardo <redacted> --- include/lapi/common_timers.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/lapi/common_timers.h b/include/lapi/common_timers.h
index 8da3abec24b2..953741d73810 100644
--- a/include/lapi/common_timers.h
+++ b/include/lapi/common_timers.h@@ -12,7 +12,7 @@ #include "lapi/posix_clocks.h" #ifndef NSEC_PER_SEC -#define NSEC_PER_SEC (1000000000L) +#define NSEC_PER_SEC (1000000000LL) #endif static const clock_t clock_list[] = {
--
2.30.2