summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorEli Zaretskii <eliz@gnu.org>2016-10-17 10:25:58 +0300
committerEli Zaretskii <eliz@gnu.org>2016-10-17 10:25:58 +0300
commitd70ed8aaeae8529813f591ce548631e424d853a2 (patch)
tree067357c6f24079ea2d4a0544830fcb863f65a543 /src
parent99892eeec8990884ef38601f14038ec6dc227741 (diff)
downloademacs-d70ed8aaeae8529813f591ce548631e424d853a2.tar.gz
Fix time-related data types in 2 editfns.c functions
* src/editfns.c (format_time_string, Fcurrent_time_zone): Pass a pointer to time_t value to emacs_localtime_rz and gmtime_r, instead of relying on struct timespec's tv_sec member to be of compatible type.
Diffstat (limited to 'src')
-rw-r--r--src/editfns.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/src/editfns.c b/src/editfns.c
index 4e90dad568f..403569f1fcd 100644
--- a/src/editfns.c
+++ b/src/editfns.c
@@ -2054,7 +2054,11 @@ format_time_string (char const *format, ptrdiff_t formatlen,
USE_SAFE_ALLOCA;
timezone_t tz = tzlookup (zone, false);
- tmp = emacs_localtime_rz (tz, &t.tv_sec, tmp);
+ /* On some systems, like 32-bit MinGW, tv_sec of struct timespec is
+ a 64-bit type, but time_t is a 32-bit type. emacs_localtime_rz
+ expects a pointer to time_t value. */
+ time_t tsec = t.tv_sec;
+ tmp = emacs_localtime_rz (tz, &tsec, tmp);
if (! tmp)
{
xtzfree (tz);
@@ -2313,7 +2317,10 @@ the data it can't find. */)
zone_name = format_time_string ("%Z", sizeof "%Z" - 1, value,
zone, &local_tm);
- if (HAVE_TM_GMTOFF || gmtime_r (&value.tv_sec, &gmt_tm))
+ /* gmtime_r expects a pointer to time_t, but tv_sec of struct
+ timespec on some systems (MinGW) is a 64-bit field. */
+ time_t tsec = value.tv_sec;
+ if (HAVE_TM_GMTOFF || gmtime_r (&tsec, &gmt_tm))
{
long int offset = (HAVE_TM_GMTOFF
? tm_gmtoff (&local_tm)