From: Jim Meyering <jim@meyering.net>
Date: Sun, 20 Jun 2004 06:53:05 +0000 (+0000)
Subject: Don't dump core if ctime returns NULL; this is possible on
X-Git-Tag: v5.3.0~1305
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f2d779e87c20bc37c3e60194c3a5c1f0f3f7c1f4;p=thirdparty%2Fcoreutils.git

Don't dump core if ctime returns NULL; this is possible on
hosts with 64-bit time_t and 32-bit int.

Include "inttostr.h".
(time_string): If ctime fails, print the raw time as an integer
instead of dumping core.
---

diff --git a/src/who.c b/src/who.c
index 554ae16569..6dd1203e6c 100644
--- a/src/who.c
+++ b/src/who.c
@@ -33,6 +33,7 @@
 
 #include "readutmp.h"
 #include "error.h"
+#include "inttostr.h"
 #include "vasprintf.h"
 
 /* The official name of this program (e.g., no `g' prefix).  */
@@ -233,8 +234,17 @@ time_string (const STRUCT_UTMP *utmp_ent)
      the tv_sec member of a struct timeval value.''  */
   time_t tm = UT_TIME_MEMBER (utmp_ent);
 
-  char *ptr = ctime (&tm) + 4;
-  ptr[12] = '\0';
+  char *ptr = ctime (&tm);
+  if (ptr)
+    {
+      ptr += 4;
+      ptr[12] = '\0';
+    }
+  else
+    {
+      static char buf[INT_BUFSIZE_BOUND (intmax_t)];
+      ptr = (TYPE_SIGNED (time_t) ? imaxtostr (tm, buf) : umaxtostr (tm, buf));
+    }
   return ptr;
 }