]> Dogcows Code - chaz/tar/blobdiff - src/list.c
(read_and): Invoke apply_nonancestor_delayed_set_stat on file names
[chaz/tar] / src / list.c
index 2109d8fe36e370e36633e1ff69d868175f6b9db8..cb3862bc1def4fba9600ccbf66c5b26811d8ebdf 100644 (file)
    with this program; if not, write to the Free Software Foundation, Inc.,
    59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
-/* Define to non-zero for forcing old ctime() instead of isotime().  */
+/* Define to non-zero for forcing old ctime format instead of ISO format.  */
 #undef USE_OLD_CTIME
 
 #include "system.h"
 #include <quotearg.h>
 
 #include <time.h>
+#ifndef time
+time_t time ();
+#endif
 
 #include "common.h"
 
+#define max(a, b) ((a) < (b) ? (b) : (a))
+
 union block *current_header;   /* points to current archive header */
 struct stat current_stat;      /* stat struct corresponding */
 enum archive_format current_format; /* recognized format */
 
-static uintmax_t from_chars PARAMS ((const char *, size_t, const char *,
-                                    uintmax_t, uintmax_t));
+static uintmax_t from_header PARAMS ((const char *, size_t, const char *,
+                                     uintmax_t, uintmax_t));
+
+/* Base 64 digits; see Internet RFC 2045 Table 1.  */
+static char const base_64_digits[64] =
+{
+  'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M',
+  'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z',
+  'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm',
+  'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z',
+  '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '+', '/'
+};
 
-/* Table of base 64 digit values indexed by unsigned chars.
-   The value is 64 for unsigned chars that are not base 64 digits.  */
-static char base64_map[1 + (unsigned char) -1];
+/* Table of base-64 digit values indexed by unsigned chars.
+   The value is 64 for unsigned chars that are not base-64 digits.  */
+static char base64_map[UCHAR_MAX + 1];
 
 static void
 base64_init (void)
@@ -74,41 +89,45 @@ read_and (void (*do_something) ())
          /* Valid header.  We should decode next field (mode) first.
             Ensure incoming names are null terminated.  */
 
-         /* FIXME: This is a quick kludge before 1.12 goes out.  */
-         current_stat.st_mtime
-           = TIME_FROM_CHARS (current_header->header.mtime);
-
-         if (!name_match (current_file_name)
-             || current_stat.st_mtime < newer_mtime_option
+         if (! name_match (current_file_name)
+             || (newer_mtime_option != TYPE_MINIMUM (time_t)
+                 /* FIXME: We get mtime now, and again later; this causes
+                    duplicate diagnostics if header.mtime is bogus.  */
+                 && ((current_stat.st_mtime
+                      = TIME_FROM_HEADER (current_header->header.mtime))
+                     < newer_mtime_option))
              || excluded_name (current_file_name))
            {
-             char save_typeflag;
-
-             if (current_header->header.typeflag == GNUTYPE_VOLHDR
-                 || current_header->header.typeflag == GNUTYPE_MULTIVOL
-                 || current_header->header.typeflag == GNUTYPE_NAMES)
+             char save_typeflag = current_header->header.typeflag;
+             switch (save_typeflag)
                {
-                 (*do_something) ();
-                 continue;
-               }
-             if (show_omitted_dirs_option
-                 && current_header->header.typeflag == DIRTYPE)
-               WARN ((0, 0, _("Omitting %s"), current_file_name));
+               case GNUTYPE_VOLHDR:
+               case GNUTYPE_MULTIVOL:
+               case GNUTYPE_NAMES:
+                 break;
+               
+               case DIRTYPE:
+                 if (show_omitted_dirs_option)
+                   WARN ((0, 0, _("%s: Omitting"),
+                          quotearg_colon (current_file_name)));
+                 /* Fall through.  */
+               default:
 
-             /* Skip past it in the archive.  */
+                 /* Skip past it in the archive.  */
 
-             save_typeflag = current_header->header.typeflag;
-             set_next_block_after (current_header);
-             if (current_header->oldgnu_header.isextended)
-               skip_extended_headers ();
+                 set_next_block_after (current_header);
+                 if (current_header->oldgnu_header.isextended)
+                   skip_extended_headers ();
 
-             /* Skip to the next header on the archive.  */
+                 /* Skip to the next header on the archive.  */
 
-             if (save_typeflag != DIRTYPE)
-               skip_file (current_stat.st_size);
-             continue;
-           }
+                 if (save_typeflag != DIRTYPE)
+                   skip_file (current_stat.st_size);
+                 continue;
+               }
+             }
 
+         apply_nonancestor_delayed_set_stat (current_file_name);
          (*do_something) ();
          continue;
 
@@ -142,12 +161,12 @@ read_and (void (*do_something) ())
          switch (prev_status)
            {
            case HEADER_STILL_UNREAD:
-             WARN ((0, 0, _("Hmm, this doesn't look like a tar archive")));
+             ERROR ((0, 0, _("This does not look like a tar archive")));
              /* Fall through.  */
 
            case HEADER_ZERO_BLOCK:
            case HEADER_SUCCESS:
-             WARN ((0, 0, _("Skipping to next file header")));
+             ERROR ((0, 0, _("Skipping to next header")));
              break;
 
            case HEADER_END_OF_FILE:
@@ -200,27 +219,25 @@ list_archive (void)
          data_block = find_next_block ();
          if (!data_block)
            {
-             ERROR ((0, 0, _("EOF in archive file")));
+             ERROR ((0, 0, _("Unexpected EOF in archive")));
              break;            /* FIXME: What happens, then?  */
            }
          written = available_space_after (data_block);
          if (written > size)
            written = size;
-         errno = 0;            /* FIXME: errno should be read-only */
+         errno = 0;
          check = fwrite (data_block->buffer, sizeof (char), written, stdlis);
          set_next_block_after ((union block *)
                                (data_block->buffer + written - 1));
          if (check != written)
            {
-             ERROR ((0, errno, _("Only wrote %lu of %lu bytes to file %s"),
-                     (unsigned long) check,
-                     (unsigned long) written, current_file_name));
+             write_error_details (current_file_name, check, written);
              skip_file (size - written);
              break;
            }
        }
       if (multi_volume_option)
-       assign_string (&save_name, NULL);
+       assign_string (&save_name, 0);
       fputc ('\n', stdlis);
       fflush (stdlis);
       return;
@@ -241,7 +258,7 @@ list_archive (void)
   skip_file (current_stat.st_size);
 
   if (multi_volume_option)
-    assign_string (&save_name, NULL);
+    assign_string (&save_name, 0);
 }
 
 /*-----------------------------------------------------------------------.
@@ -260,7 +277,7 @@ list_archive (void)
    bytes in the header as type char.  I think the type char was unsigned
    on the PDP-11, but it's signed on the Next and Sun.  It looks like the
    sources to BSD tar were never changed to compute the checksum
-   currectly, so both the Sun and Next add the bytes of the header as
+   correctly, so both the Sun and Next add the bytes of the header as
    signed chars.  This doesn't cause a problem until you get a file with
    a name containing characters with the high bit set.  So read_header
    computes two checksums -- signed and unsigned.  */
@@ -291,7 +308,7 @@ read_header (void)
       unsigned_sum = 0;
       signed_sum = 0;
       p = header->buffer;
-      for (i = sizeof (*header); i-- != 0;)
+      for (i = sizeof *header; i-- != 0;)
        {
          unsigned_sum += (unsigned char) *p;
          signed_sum += signed_char (*p++);
@@ -302,7 +319,7 @@ read_header (void)
 
       /* Adjust checksum to count the "chksum" field as blanks.  */
 
-      for (i = sizeof (header->header.chksum); i-- != 0;)
+      for (i = sizeof header->header.chksum; i-- != 0;)
        {
          unsigned_sum -= (unsigned char) header->header.chksum[i];
          signed_sum -= signed_char (header->header.chksum[i]);
@@ -310,10 +327,10 @@ read_header (void)
       unsigned_sum += ' ' * sizeof header->header.chksum;
       signed_sum += ' ' * sizeof header->header.chksum;
 
-      parsed_sum = from_chars (header->header.chksum,
-                              sizeof header->header.chksum,
-                              (char *) 0, (uintmax_t) 0,
-                              (uintmax_t) TYPE_MAXIMUM (int));
+      parsed_sum = from_header (header->header.chksum,
+                               sizeof header->header.chksum, 0,
+                               (uintmax_t) 0,
+                               (uintmax_t) TYPE_MAXIMUM (int));
       if (parsed_sum == (uintmax_t) -1)
        return HEADER_FAILURE;
 
@@ -327,7 +344,7 @@ read_header (void)
       if (header->header.typeflag == LNKTYPE)
        current_stat.st_size = 0;       /* links 0 size on tape */
       else
-       current_stat.st_size = OFF_FROM_CHARS (header->header.size);
+       current_stat.st_size = OFF_FROM_HEADER (header->header.size);
 
       if (header->header.typeflag == GNUTYPE_LONGNAME
          || header->header.typeflag == GNUTYPE_LONGLINK)
@@ -341,15 +358,15 @@ read_header (void)
            free (*longp);
          size = current_stat.st_size;
          if (size != current_stat.st_size)
-           FATAL_ERROR ((0, 0, _("Memory exhausted")));
-         bp = *longp = (char *) xmalloc (size);
+           xalloc_die ();
+         bp = *longp = xmalloc (size);
 
          for (; size > 0; size -= written)
            {
              data_block = find_next_block ();
-             if (data_block == NULL)
+             if (! data_block)
                {
-                 ERROR ((0, 0, _("Unexpected EOF on archive file")));
+                 ERROR ((0, 0, _("Unexpected EOF in archive")));
                  break;
                }
              written = available_space_after (data_block);
@@ -446,19 +463,19 @@ decode_header (union block *header, struct stat *stat_info,
     format = V7_FORMAT;
   *format_pointer = format;
 
-  stat_info->st_mode = MODE_FROM_CHARS (header->header.mode);
-  stat_info->st_mtime = TIME_FROM_CHARS (header->header.mtime);
+  stat_info->st_mode = MODE_FROM_HEADER (header->header.mode);
+  stat_info->st_mtime = TIME_FROM_HEADER (header->header.mtime);
 
   if (format == OLDGNU_FORMAT && incremental_option)
     {
-      stat_info->st_atime = TIME_FROM_CHARS (header->oldgnu_header.atime);
-      stat_info->st_ctime = TIME_FROM_CHARS (header->oldgnu_header.ctime);
+      stat_info->st_atime = TIME_FROM_HEADER (header->oldgnu_header.atime);
+      stat_info->st_ctime = TIME_FROM_HEADER (header->oldgnu_header.ctime);
     }
 
   if (format == V7_FORMAT)
     {
-      stat_info->st_uid = UID_FROM_CHARS (header->header.uid);
-      stat_info->st_gid = GID_FROM_CHARS (header->header.gid);
+      stat_info->st_uid = UID_FROM_HEADER (header->header.uid);
+      stat_info->st_gid = GID_FROM_HEADER (header->header.gid);
       stat_info->st_rdev = 0;
     }
   else
@@ -470,25 +487,25 @@ decode_header (union block *header, struct stat *stat_info,
          if (numeric_owner_option
              || !*header->header.uname
              || !uname_to_uid (header->header.uname, &stat_info->st_uid))
-           stat_info->st_uid = UID_FROM_CHARS (header->header.uid);
+           stat_info->st_uid = UID_FROM_HEADER (header->header.uid);
 
          if (numeric_owner_option
              || !*header->header.gname
              || !gname_to_gid (header->header.gname, &stat_info->st_gid))
-           stat_info->st_gid = GID_FROM_CHARS (header->header.gid);
+           stat_info->st_gid = GID_FROM_HEADER (header->header.gid);
        }
       switch (header->header.typeflag)
        {
        case BLKTYPE:
          stat_info->st_rdev
-           = makedev (MAJOR_FROM_CHARS (header->header.devmajor),
-                      MINOR_FROM_CHARS (header->header.devminor));
+           = makedev (MAJOR_FROM_HEADER (header->header.devmajor),
+                      MINOR_FROM_HEADER (header->header.devminor));
          break;
 
        case CHRTYPE:
          stat_info->st_rdev
-           = makedev (MAJOR_FROM_CHARS (header->header.devmajor),
-                      MINOR_FROM_CHARS (header->header.devminor));
+           = makedev (MAJOR_FROM_HEADER (header->header.devmajor),
+                      MINOR_FROM_HEADER (header->header.devminor));
          break;
 
        default:
@@ -500,31 +517,34 @@ decode_header (union block *header, struct stat *stat_info,
 /*------------------------------------------------------------------------.
 | Convert buffer at WHERE0 of size DIGS from external format to uintmax_t.|
 | The data is of type TYPE.  The buffer must represent a value in the     |
-| range -MINUS_MINVAL through MAXVAL.                                    |
+| range -MINUS_MINVAL through MAXVAL.  DIGS must be positive.            |
 `------------------------------------------------------------------------*/
 
 static uintmax_t
-from_chars (char const *where0, size_t digs, char const *type,
-           uintmax_t minus_minval, uintmax_t maxval)
+from_header (char const *where0, size_t digs, char const *type,
+            uintmax_t minus_minval, uintmax_t maxval)
 {
   uintmax_t value;
   char const *where = where0;
   char const *lim = where + digs;
   int negative = 0;
 
-  /* Accommodate older tars, which output leading spaces, and at least one
-     buggy tar, which outputs leading NUL if the previous field overflows.  */
+  /* Accommodate buggy tar of unknown vintage, which outputs leading
+     NUL if the previous field overflows.  */
+  where += !*where;
+
+  /* Accommodate older tars, which output leading spaces.  */
   for (;;)
     {
       if (where == lim)
        {
          if (type)
            ERROR ((0, 0,
-                   _("Empty header where numeric %s value expected"),
+                   _("Blanks in header where numeric %s value expected"),
                    type));
          return -1;
        }
-      if (!ISSPACE ((unsigned char) *where) && *where)
+      if (!ISSPACE ((unsigned char) *where))
        break;
       where++;
     }
@@ -532,35 +552,124 @@ from_chars (char const *where0, size_t digs, char const *type,
   value = 0;
   if (ISODIGIT (*where))
     {
-      do
+      char const *where1 = where;
+      uintmax_t overflow = 0;
+
+      for (;;)
        {
-         if (value << LG_8 >> LG_8 != value)
-           goto out_of_range;
-         value = (value << LG_8) | (*where++ - '0');
+         value += *where++ - '0';
+         if (where == lim || ! ISODIGIT (*where))
+           break;
+         overflow |= value ^ (value << LG_8 >> LG_8);
+         value <<= LG_8;
        }
-      while (where != lim && ISODIGIT (*where));
 
-      /* Parse the output of older tars, which output negative values
-        in two's complement octal.  This method works only if the
-        type has the same number of bits as it did on the host that
-        created the tar file, but that's the best we can do.  */
-      if (maxval < value && value - maxval <= minus_minval)
+      /* Parse the output of older, unportable tars, which generate
+         negative values in two's complement octal.  If the leading
+         nonzero digit is 1, we can't recover the original value
+         reliably; so do this only if the digit is 2 or more.  This
+         catches the common case of 32-bit negative time stamps.  */
+      if ((overflow || maxval < value) && '2' <= *where1)
        {
-         value = minus_minval - (value - maxval);
-         negative = 1;
+         /* Compute the negative of the input value, assuming two's
+            complement.  */
+         int digit = (*where1 - '0') | 4;
+         overflow = 0;
+         value = 0;
+         where = where1;
+         for (;;)
+           {
+             value += 7 - digit;
+             where++;
+             if (where == lim || ! ISODIGIT (*where))
+               break;
+             digit = *where - '0';
+             overflow |= value ^ (value << LG_8 >> LG_8);
+             value <<= LG_8;
+           }
+         value++;
+         overflow |= !value;
+
+         if (!overflow && value <= minus_minval)
+           {
+             WARN ((0, 0,
+                    _("Archive octal value %.*s is out of %s range; assuming two's complement"),
+                    (int) (where - where1), where1, type));
+             negative = 1;
+           }
+       }
+
+      if (overflow)
+       {
+         ERROR ((0, 0,
+                 _("Archive octal value %.*s is out of %s range"),
+                 (int) (where - where1), where1, type));
+         return -1;
        }
     }
-  else if (*where == '-' || *where == '+')
+  else if (type)
     {
-      int dig;
-      negative = *where++ == '-';
-      while (where != lim
-            && (dig = base64_map[(unsigned char) *where]) < 64)
+      /* The following forms cannot appear as checksums, so we don't
+        check for them if TYPE is null.  */
+
+      if (*where == '-' || *where == '+')
+       {
+         /* Parse base-64 output produced only by tar test versions
+            1.13.6 (1999-08-11) through 1.13.11 (1999-08-23).
+            Support for this will be withdrawn in future releases.  */
+         int dig;
+         static int warned_once;
+         if (! warned_once)
+           {
+             warned_once = 1;
+             WARN ((0, 0,
+                    _("Archive contains obsolescent base-64 headers")));
+           }
+         negative = *where++ == '-';
+         while (where != lim
+                && (dig = base64_map[(unsigned char) *where]) < 64)
+           {
+             if (value << LG_64 >> LG_64 != value)
+               {
+                 ERROR ((0, 0,
+                         _("Archive signed base-64 string `%.*s' is out of %s range"),
+                         (int) digs, where0, type));
+                 return -1;
+               }
+             value = (value << LG_64) | dig;
+             where++;
+           }
+       }
+      else if (*where == '\200' /* positive base-256 */
+              || *where == '\377' /* negative base-256 */)
        {
-         if (value << LG_64 >> LG_64 != value)
-           goto out_of_range;
-         value = (value << LG_64) | dig;
-         where++;
+         /* Parse base-256 output.  A nonnegative number N is
+            represented as (256**DIGS)/2 + N; a negative number -N is
+            represented as (256**DIGS) - N, i.e. as two's complement.
+            The representation guarantees that the leading bit is
+            always on, so that we don't confuse this format with the
+            others (assuming ASCII bytes of 8 bits or more).  */
+         int signbit = *where & (1 << (LG_256 - 2));
+         uintmax_t topbits = (((uintmax_t) - signbit)
+                              << (CHAR_BIT * sizeof (uintmax_t)
+                                  - LG_256 - (LG_256 - 2)));
+         value = (*where++ & ((1 << (LG_256 - 2)) - 1)) - signbit;
+         for (;;)
+           {
+             value = (value << LG_256) + (unsigned char) *where++;
+             if (where == lim)
+               break;
+             if (((value << LG_256 >> LG_256) | topbits) != value)
+               {
+                 ERROR ((0, 0,
+                         _("Archive base-256 value is out of %s range"),
+                         type));
+                 return -1;
+               }
+           }
+         negative = signbit;
+         if (negative)
+           value = -value;
        }
     }
 
@@ -573,15 +682,15 @@ from_chars (char const *where0, size_t digs, char const *type,
 
          if (!o)
            {
-             o = clone_quoting_options ((struct quoting_options *) 0);
-             set_quoting_style (o, c_quoting_style);
+             o = clone_quoting_options (0);
+             set_quoting_style (o, locale_quoting_style);
            }
 
          while (where0 != lim && ! lim[-1])
            lim--;
          quotearg_buffer (buf, sizeof buf, where0, lim - where, o);
          ERROR ((0, 0,
-                 _("Header contains `%.*s' where numeric %s value expected"),
+                 _("Archive contains %.*s where numeric %s value expected"),
                  (int) sizeof buf, buf, type));
        }
 
@@ -591,44 +700,56 @@ from_chars (char const *where0, size_t digs, char const *type,
   if (value <= (negative ? minus_minval : maxval))
     return negative ? -value : value;
 
- out_of_range:
   if (type)
-    ERROR ((0, 0, _("Numeric value `%.*s' is out of range for %s"),
-           (int) digs, where0, type));
+    {
+      char minval_buf[UINTMAX_STRSIZE_BOUND + 1];
+      char maxval_buf[UINTMAX_STRSIZE_BOUND];
+      char value_buf[UINTMAX_STRSIZE_BOUND + 1];
+      char *minval_string = STRINGIFY_BIGINT (minus_minval, minval_buf + 1);
+      char *value_string = STRINGIFY_BIGINT (value, value_buf + 1);
+      if (negative)
+       *--value_string = '-';
+      if (minus_minval)
+       *--minval_string = '-';
+      ERROR ((0, 0, _("Archive value %s is out of %s range %s..%s"),
+             value_string, type,
+             minval_string, STRINGIFY_BIGINT (maxval, maxval_buf)));
+    }
+
   return -1;
 }
 
 gid_t
-gid_from_chars (const char *p, size_t s)
+gid_from_header (const char *p, size_t s)
 {
-  return from_chars (p, s, "gid_t",
-                    - (uintmax_t) TYPE_MINIMUM (gid_t),
-                    (uintmax_t) TYPE_MAXIMUM (gid_t));
+  return from_header (p, s, "gid_t",
+                     - (uintmax_t) TYPE_MINIMUM (gid_t),
+                     (uintmax_t) TYPE_MAXIMUM (gid_t));
 }
 
 major_t
-major_from_chars (const char *p, size_t s)
+major_from_header (const char *p, size_t s)
 {
-  return from_chars (p, s, "major_t",
-                    - (uintmax_t) TYPE_MINIMUM (major_t),
-                    (uintmax_t) TYPE_MAXIMUM (major_t));
+  return from_header (p, s, "major_t",
+                     - (uintmax_t) TYPE_MINIMUM (major_t),
+                     (uintmax_t) TYPE_MAXIMUM (major_t));
 }
 
 minor_t
-minor_from_chars (const char *p, size_t s)
+minor_from_header (const char *p, size_t s)
 {
-  return from_chars (p, s, "minor_t",
-                    - (uintmax_t) TYPE_MINIMUM (minor_t),
-                    (uintmax_t) TYPE_MAXIMUM (minor_t));
+  return from_header (p, s, "minor_t",
+                     - (uintmax_t) TYPE_MINIMUM (minor_t),
+                     (uintmax_t) TYPE_MAXIMUM (minor_t));
 }
 
 mode_t
-mode_from_chars (const char *p, size_t s)
+mode_from_header (const char *p, size_t s)
 {
   /* Do not complain about unrecognized mode bits.  */
-  unsigned u = from_chars (p, s, "mode_t",
-                          - (uintmax_t) TYPE_MINIMUM (mode_t),
-                          TYPE_MAXIMUM (uintmax_t));
+  unsigned u = from_header (p, s, "mode_t",
+                           - (uintmax_t) TYPE_MINIMUM (mode_t),
+                           TYPE_MAXIMUM (uintmax_t));
   return ((u & TSUID ? S_ISUID : 0)
          | (u & TSGID ? S_ISGID : 0)
          | (u & TSVTX ? S_ISVTX : 0)
@@ -644,40 +765,45 @@ mode_from_chars (const char *p, size_t s)
 }
 
 off_t
-off_from_chars (const char *p, size_t s)
+off_from_header (const char *p, size_t s)
 {
-  return from_chars (p, s, "off_t",
-                    - (uintmax_t) TYPE_MINIMUM (off_t),
-                    (uintmax_t) TYPE_MAXIMUM (off_t));
+  /* Negative offsets are not allowed in tar files, so invoke
+     from_header with minimum value 0, not TYPE_MINIMUM (off_t).  */
+  return from_header (p, s, "off_t", (uintmax_t) 0,
+                     (uintmax_t) TYPE_MAXIMUM (off_t));
 }
 
 size_t
-size_from_chars (const char *p, size_t s)
+size_from_header (const char *p, size_t s)
 {
-  return from_chars (p, s, "size_t", (uintmax_t) 0, 
-                    (uintmax_t) TYPE_MAXIMUM (size_t));
+  return from_header (p, s, "size_t", (uintmax_t) 0, 
+                     (uintmax_t) TYPE_MAXIMUM (size_t));
 }
 
 time_t
-time_from_chars (const char *p, size_t s)
+time_from_header (const char *p, size_t s)
 {
-  return from_chars (p, s, "time_t",
-                    - (uintmax_t) TYPE_MINIMUM (time_t),
-                    (uintmax_t) TYPE_MAXIMUM (time_t));
+  time_t t = from_header (p, s, "time_t",
+                         - (uintmax_t) TYPE_MINIMUM (time_t),
+                         (uintmax_t) TYPE_MAXIMUM (time_t));
+  if (start_time < t && time (0) < t)
+    WARN ((0, 0, _("Archive contains future timestamp %s"), tartime (t)));
+  return t;
 }
 
 uid_t
-uid_from_chars (const char *p, size_t s)
+uid_from_header (const char *p, size_t s)
 {
-  return from_chars (p, s, "uid_t", (uintmax_t) 0,
-                    (uintmax_t) TYPE_MAXIMUM (uid_t));
+  return from_header (p, s, "uid_t",
+                     - (uintmax_t) TYPE_MINIMUM (uid_t),
+                     (uintmax_t) TYPE_MAXIMUM (uid_t));
 }
 
 uintmax_t
-uintmax_from_chars (const char *p, size_t s)
+uintmax_from_header (const char *p, size_t s)
 {
-  return from_chars (p, s, "uintmax_t", (uintmax_t) 0,
-                    TYPE_MAXIMUM (uintmax_t));
+  return from_header (p, s, "uintmax_t", (uintmax_t) 0,
+                     TYPE_MAXIMUM (uintmax_t));
 }
 
 
@@ -695,56 +821,50 @@ stringify_uintmax_t_backwards (uintmax_t o, char *buf)
   return buf;
 }
 
-#if !USE_OLD_CTIME
-
-/*-------------------------------------------.
-| Return the time formatted along ISO 8601.  |
-`-------------------------------------------*/
-
-/* Also, see http://www.ft.uni-erlangen.de/~mskuhn/iso-time.html.  */
-
-static char *
-isotime (const time_t *time)
+/* Return a printable representation of T.  The result points to
+   static storage that can be reused in the next call to this
+   function, to ctime, or to asctime.  */
+char const *
+tartime (time_t t)
 {
-  static char buffer[INT_STRLEN_BOUND (int) + 16];
-  struct tm *tm = localtime (time);
-  if (tm)
-    sprintf (buffer, "%04d-%02d-%02d %02d:%02d:%02d",
-            tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday,
-            tm->tm_hour, tm->tm_min, tm->tm_sec);
-  else
-    /* Interpose %s between ?? and - to avoid ANSI C trigraph brain damage.  */
-    sprintf (buffer, "????%s-??%s-?? ??:??:??", "", "");
-
-  return buffer;
-}
-
-#endif /* not USE_OLD_CTIME */
+  static char buffer[max (UINTMAX_STRSIZE_BOUND + 1,
+                         INT_STRLEN_BOUND (int) + 16)];
+  char *p;
 
-/*-------------------------------------------------------------------------.
-| Decode MODE from its binary form in a stat structure, and encode it into |
-| a 9 characters string STRING, terminated with a NUL.                     |
-`-------------------------------------------------------------------------*/
+#if USE_OLD_CTIME
+  p = ctime (&t);
+  if (p)
+    {
+      char const *time_stamp = p + 4;
+      for (p += 16; p[4] != '\n'; p++)
+       p[0] = p[4];
+      p[0] = '\0';
+      return time_stamp;
+    }
+#else
+  /* Use ISO 8610 format.  See:
+     http://www.cl.cam.ac.uk/~mgk25/iso-time.html  */
+  struct tm *tm = localtime (&t);
+  if (tm)
+    {
+      sprintf (buffer, "%04d-%02d-%02d %02d:%02d:%02d",
+              tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday,
+              tm->tm_hour, tm->tm_min, tm->tm_sec);
+      return buffer;
+    }
+#endif
 
-static void
-decode_mode (mode_t mode, char *string)
-{
-  *string++ = mode & S_IRUSR ? 'r' : '-';
-  *string++ = mode & S_IWUSR ? 'w' : '-';
-  *string++ = (mode & S_ISUID
-              ? (mode & S_IXUSR ? 's' : 'S')
-              : (mode & S_IXUSR ? 'x' : '-'));
-  *string++ = mode & S_IRGRP ? 'r' : '-';
-  *string++ = mode & S_IWGRP ? 'w' : '-';
-  *string++ = (mode & S_ISGID
-              ? (mode & S_IXGRP ? 's' : 'S')
-              : (mode & S_IXGRP ? 'x' : '-'));
-  *string++ = mode & S_IROTH ? 'r' : '-';
-  *string++ = mode & S_IWOTH ? 'w' : '-';
-  *string++ = (mode & S_ISVTX
-              ? (mode & S_IXOTH ? 't' : 'T')
-              : (mode & S_IXOTH ? 'x' : '-'));
-  *string = '\0';
+  /* The time stamp cannot be broken down, most likely because it
+     is out of range.  Convert it as an integer,
+     right-adjusted in a field with the same width as the usual
+     19-byte 4-year ISO time format.  */
+  p = stringify_uintmax_t_backwards (t < 0 ? - (uintmax_t) t : (uintmax_t) t,
+                                    buffer + sizeof buffer);
+  if (t < 0)
+    *--p = '-';
+  while (buffer + sizeof buffer - 19 - 1 < p)
+    *--p = ' ';
+  return p;
 }
 
 /*-------------------------------------------------------------------------.
@@ -777,16 +897,14 @@ void
 print_header (void)
 {
   char modes[11];
-  char const *timestamp;
+  char const *time_stamp;
   /* These hold formatted ints.  */
   char uform[UINTMAX_STRSIZE_BOUND], gform[UINTMAX_STRSIZE_BOUND];
   char *user, *group;
   char size[2 * UINTMAX_STRSIZE_BOUND];
                                /* holds formatted size or major,minor */
   char uintbuf[UINTMAX_STRSIZE_BOUND];
-  time_t longie;               /* to make ctime() call portable */
   int pad;
-  char *name;
 
   if (block_number_option)
     {
@@ -798,16 +916,7 @@ print_header (void)
   if (verbose_option <= 1)
     {
       /* Just the fax, mam.  */
-
-      char *quoted_name = quote_copy_string (current_file_name);
-
-      if (quoted_name)
-       {
-         fprintf (stdlis, "%s\n", quoted_name);
-         free (quoted_name);
-       }
-      else
-       fprintf (stdlis, "%s\n", current_file_name);
+      fprintf (stdlis, "%s\n", quotearg (current_file_name));
     }
   else
     {
@@ -866,25 +975,9 @@ print_header (void)
 
       decode_mode (current_stat.st_mode, modes + 1);
 
-      /* Timestamp.  */
+      /* Time stamp.  */
 
-      longie = current_stat.st_mtime;
-#if USE_OLD_CTIME
-      {
-       char *ct = ctime (&longie);
-       if (ct)
-         {
-           timestamp = ct + 4;
-           for (ct += 16; ct[4] != '\n'; ct++)
-             ct[0] = ct[4];
-           ct[0] = '\0';
-         }
-       else
-         timestamp = "??? ?? ??:?? ????";
-      }
-#else
-      timestamp = isotime (&longie);
-#endif
+      time_stamp = tartime (current_stat.st_mtime);
 
       /* User and group names.  */
 
@@ -892,7 +985,7 @@ print_header (void)
          && !numeric_owner_option)
        user = current_header->header.uname;
       else
-       user = STRINGIFY_BIGINT (UINTMAX_FROM_CHARS
+       user = STRINGIFY_BIGINT (UINTMAX_FROM_HEADER
                                 (current_header->header.uid),
                                 uform);
 
@@ -900,7 +993,7 @@ print_header (void)
          && !numeric_owner_option)
        group = current_header->header.gname;
       else
-       group = STRINGIFY_BIGINT (UINTMAX_FROM_CHARS
+       group = STRINGIFY_BIGINT (UINTMAX_FROM_HEADER
                                  (current_header->header.gid),
                                  gform);
 
@@ -910,14 +1003,17 @@ print_header (void)
        {
        case CHRTYPE:
        case BLKTYPE:
-         sprintf (size, "%lu,%lu",
-                  (unsigned long) major (current_stat.st_rdev),
-                  (unsigned long) minor (current_stat.st_rdev));
+         strcpy (size,
+                 STRINGIFY_BIGINT (major (current_stat.st_rdev), uintbuf));
+         strcat (size, ",");
+         strcat (size,
+                 STRINGIFY_BIGINT (minor (current_stat.st_rdev), uintbuf));
          break;
        case GNUTYPE_SPARSE:
          strcpy (size,
                  STRINGIFY_BIGINT
-                 (UINTMAX_FROM_CHARS (current_header->oldgnu_header.realsize),
+                 (UINTMAX_FROM_HEADER (current_header
+                                       ->oldgnu_header.realsize),
                   uintbuf));
          break;
        default:
@@ -932,39 +1028,18 @@ print_header (void)
        ugswidth = pad;
 
       fprintf (stdlis, "%s %s/%s %*s%s %s",
-              modes, user, group, ugswidth - pad, "", size, timestamp);
+              modes, user, group, ugswidth - pad, "", size, time_stamp);
 
-      name = quote_copy_string (current_file_name);
-      if (name)
-       {
-         fprintf (stdlis, " %s", name);
-         free (name);
-       }
-      else
-       fprintf (stdlis, " %s", current_file_name);
+      fprintf (stdlis, " %s", quotearg (current_file_name));
 
       switch (current_header->header.typeflag)
        {
        case SYMTYPE:
-         name = quote_copy_string (current_link_name);
-         if (name)
-           {
-             fprintf (stdlis, " -> %s\n", name);
-             free (name);
-           }
-         else
-           fprintf (stdlis, " -> %s\n", current_link_name);
+         fprintf (stdlis, " -> %s\n", quotearg (current_link_name));
          break;
 
        case LNKTYPE:
-         name = quote_copy_string (current_link_name);
-         if (name)
-           {
-             fprintf (stdlis, _(" link to %s\n"), name);
-             free (name);
-           }
-         else
-           fprintf (stdlis, _(" link to %s\n"), current_link_name);
+         fprintf (stdlis, _(" link to %s\n"), quotearg (current_link_name));
          break;
 
        default:
@@ -991,7 +1066,7 @@ print_header (void)
        case GNUTYPE_MULTIVOL:
          strcpy (size,
                  STRINGIFY_BIGINT
-                 (UINTMAX_FROM_CHARS (current_header->oldgnu_header.offset),
+                 (UINTMAX_FROM_HEADER (current_header->oldgnu_header.offset),
                   uintbuf));
          fprintf (stdlis, _("--Continued at byte %s--\n"), size);
          break;
@@ -1012,7 +1087,6 @@ void
 print_for_mkdir (char *pathname, int length, mode_t mode)
 {
   char modes[11];
-  char *name;
 
   if (verbose_option > 1)
     {
@@ -1027,16 +1101,9 @@ print_for_mkdir (char *pathname, int length, mode_t mode)
          fprintf (stdlis, _("block %s: "),
                   STRINGIFY_BIGINT (current_block_ordinal (), buf));
        }
-      name = quote_copy_string (pathname);
-      if (name)
-       {
-         fprintf (stdlis, "%s %*s %.*s\n", modes, ugswidth + DATEWIDTH,
-                  _("Creating directory:"), length, name);
-         free (name);
-       }
-      else
-       fprintf (stdlis, "%s %*s %.*s\n", modes, ugswidth + DATEWIDTH,
-                _("Creating directory:"), length, pathname);
+
+      fprintf (stdlis, "%s %*s %.*s\n", modes, ugswidth + DATEWIDTH,
+              _("Creating directory:"), length, quotearg (pathname));
     }
 }
 
@@ -1058,8 +1125,8 @@ skip_file (off_t size)
   while (size > 0)
     {
       x = find_next_block ();
-      if (x == NULL)
-       FATAL_ERROR ((0, 0, _("Unexpected EOF on archive file")));
+      if (! x)
+       FATAL_ERROR ((0, 0, _("Unexpected EOF in archive")));
 
       set_next_block_after (x);
       size -= BLOCKSIZE;
@@ -1081,7 +1148,7 @@ skip_extended_headers (void)
     {
       exhdr = find_next_block ();
       if (!exhdr)
-       FATAL_ERROR ((0, 0, _("Unexpected EOF on archive file")));
+       FATAL_ERROR ((0, 0, _("Unexpected EOF in archive")));
       set_next_block_after (exhdr);
     }
   while (exhdr->sparse_header.isextended);
This page took 0.054488 seconds and 4 git commands to generate.