From d620431f661375d3564b60f110d1f69575ac78d7 Mon Sep 17 00:00:00 2001 From: Jeremie Courreges-Anglas Date: Thu, 5 Oct 2017 01:43:33 +0200 Subject: [PATCH] Cast time_t to long double in order to print it. The underlying type of time_t can be anything from unsigned 32 bits to signed 64 bits to float. To reliably print it, better cast it to "long long", which is at least 64 bits wide and can represent values beyond 2038. Printing as a "long" could cause problems on ILP32 systems using a 64 bits time_t (eg OpenBSD/armv7). Signed-off-by: Jeremie Courreges-Anglas --- src/openvpn/error.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/openvpn/error.c b/src/openvpn/error.c index 04bf0da5..7b46c5ec 100644 --- a/src/openvpn/error.c +++ b/src/openvpn/error.c @@ -342,8 +342,8 @@ x_msg_va(const unsigned int flags, const char *format, va_list arglist) struct timeval tv; gettimeofday(&tv, NULL); - fprintf(fp, "%lu.%06lu %x %s%s%s%s", - tv.tv_sec, + fprintf(fp, "%lld.%06lu %x %s%s%s%s", + (long long)tv.tv_sec, (unsigned long)tv.tv_usec, flags, prefix, -- 2.14.2