]> Savannah Git Hosting - gnulib.git/commitdiff
readutmp: work around glibc utmpx bug
authorPaul Eggert <eggert@cs.ucla.edu>
Sun, 30 Jul 2023 00:06:42 +0000 (17:06 -0700)
committerBruno Haible <bruno@clisp.org>
Wed, 16 Aug 2023 16:49:49 +0000 (18:49 +0200)
When compiled with _TIME_BITS=64, glibc <utmpx.h> does not work,
because the files use 32-bit time_t and the code passes this to
the user unmodified, but <utmpx.h> defines a struct with 64-bit
time_t.  Work around this compatibility bug.  Problem reported
by Jakub Wilk via Sven Joachim <https://bugs.gnu.org/64937>.
* lib/readutmp.c (copy_utmp_entry): New function.
(read_utmp): Use it.

ChangeLog
doc/glibc-functions/getutmp.texi
doc/glibc-functions/getutmpx.texi
doc/glibc-headers/utmp.texi
doc/posix-headers/utmpx.texi
doc/year2038.texi
lib/readutmp.c

index a325f24f49e34203df10c0c2b59fae703f63fce1..ddb24f6284b986aa9704f591d76a0ad876fcee1a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2023-07-29  Paul Eggert  <eggert@cs.ucla.edu>
+
+       readutmp: work around glibc utmpx bug
+       When compiled with _TIME_BITS=64, glibc <utmpx.h> does not work,
+       because the files use 32-bit time_t and the code passes this to
+       the user unmodified, but <utmpx.h> defines a struct with 64-bit
+       time_t.  Work around this compatibility bug.
+       * lib/readutmp.c (copy_utmp_entry): New function.
+       (read_utmp): Use it.
+
 2023-07-29  Bruno Haible  <bruno@clisp.org>
 
        time-h: Obey GNULIB_POSIXCHECK, not GNULIB_PORTCHECK.
index 2628441e1366782fe80afb31571dfaf72e18dbc0..b53369a5975f2a843146b9a4b7b81ba449ea74a6 100644 (file)
@@ -26,4 +26,17 @@ Portability problems not fixed by Gnulib:
 @item
 This function is missing on some platforms:
 FreeBSD 13.0, OpenBSD 6.7, Minix 3.1.8, AIX 5.1, HP-UX 11, Cygwin 2.9, mingw, MSVC 14, Android 9.0.
+@item
+On some platforms, this function does not support timestamps past the
+year 2038:
+glibc 2.38 on 32-bit platforms like x86 and ARM where @code{time_t}
+was historically 32 bits.
+@item
+On some platforms, this function misbehaves if the @code{year2038} or
+@code{year2038-recommended} modules are used and the program is
+configured without the @option{--disable-year2038} option.
+The @code{readutmp} module works around this problem:
+glibc 2.38 on 32-bit platforms like x86 and ARM where @code{time_t}
+was historically 32 bits.
+@xref{Avoiding the year 2038 problem}.
 @end itemize
index 0ebc01978bacca0b6eeee2a191dfc9aa069a3703..bee96fc239674d5153ababb553958154712ae220 100644 (file)
@@ -26,4 +26,17 @@ Portability problems not fixed by Gnulib:
 @item
 This function is missing on some platforms:
 FreeBSD 13.0, OpenBSD 6.7, Minix 3.1.8, AIX 5.1, HP-UX 11, Cygwin 2.9, mingw, MSVC 14, Android 9.0.
+@item
+On some platforms, this function does not support timestamps past the
+year 2038:
+glibc 2.38 on 32-bit platforms like x86 and ARM where @code{time_t}
+was historically 32 bits.
+@item
+On some platforms, this function misbehaves if the @code{year2038} or
+@code{year2038-recommended} modules are used and the program is
+configured without the @option{--disable-year2038} option.
+The @code{readutmp} module works around this problem:
+glibc 2.38 on 32-bit platforms like x86 and ARM where @code{time_t}
+was historically 32 bits.
+@xref{Avoiding the year 2038 problem}.
 @end itemize
index d7d1091e144fa1c98e9e4f52b88c066636258f90..1e9d3757cb66479c0421ce11b44feb19d5f56e4a 100644 (file)
@@ -31,4 +31,17 @@ FreeBSD 8.0, OpenBSD 7.2.
 
 Portability problems not fixed by Gnulib:
 @itemize
+@item
+On some platforms, this API does not support timestamps past the
+year 2038:
+glibc 2.38 on 32-bit platforms like x86 and ARM where @code{time_t}
+was historically 32 bits.
+@item
+On some platforms, this header misbehaves if the @code{year2038} or
+@code{year2038-recommended} modules are used and the program is
+configured without the @option{--disable-year2038} option.
+The @code{readutmp} module works around this problem:
+glibc 2.38 on 32-bit platforms like x86 and ARM where @code{time_t}
+was historically 32 bits.
+@xref{Avoiding the year 2038 problem}.
 @end itemize
index 812544ad708f805f8f32f6fc3c2a0665aa10a586..fd3d4d6ab7f610ad682b80c0abd6ba08a4785f4b 100644 (file)
@@ -14,4 +14,17 @@ Portability problems not fixed by Gnulib:
 @item
 This header file is missing on some platforms:
 FreeBSD 6.0, OpenBSD 6.7, Minix 3.1.8, mingw, MSVC 14, Android 9.0.
+@item
+On some platforms, this API does not support timestamps past the
+year 2038:
+glibc 2.38 on 32-bit platforms like x86 and ARM where @code{time_t}
+was historically 32 bits.
+@item
+On some platforms, this header misbehaves if the @code{year2038} or
+@code{year2038-recommended} modules are used and the program is
+configured without the @option{--disable-year2038} option.
+The @code{readutmp} module works around this problem:
+glibc 2.38 on 32-bit platforms like x86 and ARM where @code{time_t}
+was historically 32 bits.
+@xref{Avoiding the year 2038 problem}.
 @end itemize
index 59646018313cbf3f52cbd13b81b553346ab42ef2..4d56f323f1ae7905d0023734a1a87649574ab999 100644 (file)
@@ -103,3 +103,11 @@ Cygwin 3.3.6 (2022) and earlier on x86,
 @item
 Haiku/x86.
 @end itemize
+
+If you use the @samp{year2038} or @samp{year2038-recommended} modules,
+and configure to support timestamps after the year 2038,
+your code should not include @samp{<utmp.h>} or @samp{<utmpx.h>}
+directly, because these include files do not work with 64-bit timestamps
+if the platform's @code{time_t} was traditionally 32 bits.
+Your code can instead use the @samp{readutmp} module,
+which works around this problem.
index d282254cdce1715c8a2156ea78da4410b9f23757..cfc8b69c038b9163c9f6f033597f6ab74bc52cbb 100644 (file)
@@ -86,6 +86,51 @@ desirable_utmp_entry (STRUCT_UTMP const *u, int options)
 
 #ifdef UTMP_NAME_FUNCTION
 
+static void
+copy_utmp_entry (STRUCT_UTMP *dst, STRUCT_UTMP *src)
+{
+#if __GLIBC__ && _TIME_BITS == 64
+  /* Convert from external form in SRC to internal form in DST.
+     It is OK to convert now, rather than earlier, before
+     desirable_utmp_entry was invoked, because desirable_utmp_entry
+     inspects only the leading prefix of the entry, which is the
+     same in both external and internal forms.  */
+
+  /* This is a near-copy of glibc's struct utmpx, which stops working
+     after the year 2038.  Unlike the glibc version, struct utmpx32
+     describes the file format even if time_t is 64 bits.  */
+  struct utmpx32
+  {
+    short int ut_type;                 /* Type of login.  */
+    pid_t ut_pid;                      /* Process ID of login process.  */
+    char ut_line[sizeof src->ut_line]; /* Devicename.  */
+    char ut_id[sizeof src->ut_id];     /* Inittab ID.  */
+    char ut_user[sizeof src->ut_user];  /* Username.  */
+    char ut_host[sizeof src->ut_host]; /* Hostname for remote login.  */
+    struct __exit_status ut_exit;      /* Exit status of a process marked
+                                           as DEAD_PROCESS.  */
+    /* The fields ut_session and ut_tv must be the same size when compiled
+       32- and 64-bit.  This allows files and shared memory to be shared
+       between 32- and 64-bit applications.  */
+    int ut_session;                    /* Session ID, used for windowing.  */
+    struct
+    {
+      int tv_sec;                      /* Seconds.  */
+      int tv_usec;                     /* Microseconds.  */
+    } ut_tv;                           /* Time entry was made.  */
+    int ut_addr_v6[4];                 /* Internet address of remote host.  */
+    char ut_reserved[20];              /* Reserved for future use.  */
+  } *s = (struct utmpx32 *) src;
+  memcpy (dst, s, offsetof (struct utmpx32, ut_session));
+  dst->ut_session = s->ut_session;
+  dst->ut_tv.tv_sec = s->ut_tv.tv_sec;
+  dst->ut_tv.tv_usec = s->ut_tv.tv_usec;
+  memcpy (&dst->ut_addr_v6, s->ut_addr_v6, sizeof dst->ut_addr_v6);
+#else
+  *dst = *src;
+#endif
+}
+
 int
 read_utmp (char const *file, size_t *n_entries, STRUCT_UTMP **utmp_buf,
            int options)
@@ -109,7 +154,7 @@ read_utmp (char const *file, size_t *n_entries, STRUCT_UTMP **utmp_buf,
         if (n_read == n_alloc)
           utmp = xpalloc (utmp, &n_alloc, 1, -1, sizeof *utmp);
 
-        utmp[n_read++] = *u;
+        copy_utmp_entry (&utmp[n_read++], u);
       }
 
   END_UTMP_ENT ();