(long_time_expected_width, print_long_format): Fix

bug: the initial byte passed to strftime wasn't initialized to
a nonzero value after the buffer was reallocated.

1 files changed, 15 insertions, 7 deletions

diff --git a/src/ls.c b/src/ls.c
index 3713e3c13..93215e36e 100644
--- a/src/ls.c
+++ b/src/ls.c
@@ -2362,9 +2362,14 @@ long_time_expected_width (void)
       size_t bufsize = sizeof initbuf;
       size_t len;
 
-      *buf = '\1';
-      while (! (len = strftime (buf, bufsize, fmt, tm)) && *buf)
-	buf = alloca (bufsize *= 2);
+      for (;;)
+	{
+	  *buf = '\1';
+	  len = strftime (buf, bufsize, fmt, tm);
+	  if (len || ! *buf)
+	    break;
+	  buf = alloca (bufsize *= 2);
+	}
 
       width = mbsnwidth (buf, len, 0);
       if (width < 0)
@@ -2491,12 +2496,15 @@ print_long_format (const struct fileinfo *f)
       int recent = six_months_ago <= when && when <= now;
 
       char const *fmt = long_time_format[recent];
-      *p = '\1';
 
-      while (! (s = strftime (p, buf + bufsize - p - 1, fmt, when_local))
-	     && *p)
+      for (;;)
 	{
-	  char *newbuf = (char *) alloca (bufsize *= 2);
+	  char *newbuf;
+	  *p = '\1';
+	  s = strftime (p, buf + bufsize - p - 1, fmt, when_local);
+	  if (s || ! *p)
+	    break;
+	  newbuf = alloca (bufsize *= 2);
 	  memcpy (newbuf, buf, p - buf);
 	  p = newbuf + (p - buf);
 	  buf = newbuf;