about summary refs log tree commit diff
path: root/include
diff options
context:
space:
mode:
authorYunQiang Su <yunqiang.su@cipunited.com>2022-11-08 12:49:46 +0800
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>2022-11-17 14:35:13 -0300
commita9acb7b39ed21386142b963aeecc35e0b468c0de (patch)
tree0c77b1e8f3dcc46a1a0e13d8938bac571e636be2 /include
parent94628de77888c3292fc103840731ff85f283368e (diff)
downloadglibc-a9acb7b39ed21386142b963aeecc35e0b468c0de.tar.gz
glibc-a9acb7b39ed21386142b963aeecc35e0b468c0de.tar.xz
glibc-a9acb7b39ed21386142b963aeecc35e0b468c0de.zip
Define in_int32_t_range to check if the 64 bit time_t syscall should be used
Currently glibc uses in_time_t_range to detects time_t overflow,
and if it occurs fallbacks to 64 bit syscall version.

The function name is confusing because internally time_t might be
either 32 bits or 64 bits (depending on __TIMESIZE).

This patch refactors the in_time_t_range by replacing it with
in_int32_t_range for the case to check if the 64 bit time_t syscall
should be used.

The in_time_t range is used to detect overflow of the
syscall return value.

Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>
Diffstat (limited to 'include')
-rw-r--r--include/time.h10
1 files changed, 9 insertions, 1 deletions
diff --git a/include/time.h b/include/time.h
index 20abea69d4..f599eeed4e 100644
--- a/include/time.h
+++ b/include/time.h
@@ -347,12 +347,20 @@ libc_hidden_proto (__time64)
 /* Check whether T fits in int32_t, assume all usages are for
    sizeof(time_t) == 32.  */
 static inline bool
-in_time_t_range (__time64_t t)
+in_int32_t_range (__time64_t t)
 {
   int32_t s = t;
   return s == t;
 }
 
+/* Check whether T fits in time_t.  */
+static inline bool
+in_time_t_range (__time64_t t)
+{
+  time_t s = t;
+  return s == t;
+}
+
 /* Convert a known valid struct timeval into a struct __timespec64.  */
 static inline struct __timespec64
 valid_timeval_to_timespec64 (const struct timeval tv)