From beb39c211b4ff52c90106cc69666427f4e992c5e Mon Sep 17 00:00:00 2001 From: Michael Jeanson Date: Tue, 9 Feb 2021 11:04:25 -0500 Subject: [PATCH] fix: cast LTTNG_KERNEL_VERSION/LTTNG_LINUX_VERSION_CODE to uint64_t Cast our version macros to an unsigned 64bits value to prevent overflowing when we append distro specific version information. Signed-off-by: Michael Jeanson Signed-off-by: Mathieu Desnoyers Change-Id: Idd1ba502b02b03e52341ca1f3685de889fa06b3e --- lttng-kernel-version.h | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/lttng-kernel-version.h b/lttng-kernel-version.h index ec2f084d..2044203a 100644 --- a/lttng-kernel-version.h +++ b/lttng-kernel-version.h @@ -39,19 +39,22 @@ * of LINUX_VERSION_CODE from the kernel headers and allocate 16bits. * Otherwise, keep using the version code from the headers to minimise the * behavior change and avoid regressions. + * + * Cast the result to uint64_t to prevent overflowing when we append distro + * specific version information. */ #if (LTTNG_LINUX_PATCH >= 256) #define LTTNG_KERNEL_VERSION(a, b, c) \ - (((a) << 24) + ((b) << 16) + (c)) + ((((a) << 24) + ((b) << 16) + (c)) * 1ULL) #define LTTNG_LINUX_VERSION_CODE \ LTTNG_KERNEL_VERSION(LTTNG_LINUX_MAJOR, LTTNG_LINUX_MINOR, LTTNG_LINUX_PATCH) #else -#define LTTNG_KERNEL_VERSION(a, b, c) KERNEL_VERSION(a, b, c) -#define LTTNG_LINUX_VERSION_CODE LINUX_VERSION_CODE +#define LTTNG_KERNEL_VERSION(a, b, c) (KERNEL_VERSION(a, b, c) * 1ULL) +#define LTTNG_LINUX_VERSION_CODE (LINUX_VERSION_CODE * 1ULL) #endif -- 2.34.1