]> Dogcows Code - chaz/tar/blobdiff - src/create.c
Moved system dependencies to system.c
[chaz/tar] / src / create.c
index b04b3a81bd24ce1ad9a6e898e8f5b19bf5d36bf6..b712f93b716aa220eea16ed6b52bb4115ef424b7 100644 (file)
@@ -1,5 +1,8 @@
 /* Create a tar archive.
-   Copyright 1985, 92, 93, 94, 96, 97, 1999 Free Software Foundation, Inc.
+
+   Copyright (C) 1985, 1992, 1993, 1994, 1996, 1997, 1999, 2000, 2001,
+   2003 Free Software Foundation, Inc.
+
    Written by John Gilmore, on 1985-08-25.
 
    This program is free software; you can redistribute it and/or modify it
 
 #include "system.h"
 
-#if !MSDOS
-# include <pwd.h>
-# include <grp.h>
-#endif
-
 #if HAVE_UTIME_H
 # include <utime.h>
 #else
@@ -33,26 +31,18 @@ struct utimbuf
   };
 #endif
 
-#include "common.h"
-
-#ifndef MSDOS
-extern dev_t ar_dev;
-extern ino_t ar_ino;
-#endif
-
-extern struct name *gnu_list_name;
+#include <quotearg.h>
 
-/* This module is the only one that cares about `struct link's.  */
+#include "common.h"
+#include <hash.h>
 
 struct link
   {
-    struct link *next;
     dev_t dev;
     ino_t ino;
+    size_t nlink;
     char name[1];
   };
-
-static struct link *linklist;  /* points to first link in list */
 \f
 /* The maximum uintmax_t value that can be represented with DIGITS digits,
    assuming that each digit is BITS_PER_DIGIT wide.  */
@@ -117,7 +107,7 @@ to_base256 (int negative, uintmax_t value, char *where, size_t size)
 
 static void
 to_chars (int negative, uintmax_t value, size_t valsize,
-         uintmax_t (*substitute) PARAMS ((int *)),
+         uintmax_t (*substitute) (int *),
          char *where, size_t size, const char *type)
 {
   int base256_allowed = (archive_format == GNU_FORMAT
@@ -330,26 +320,28 @@ uintmax_to_chars (uintmax_t v, char *p, size_t s)
 {
   to_chars (0, v, sizeof v, 0, p, s, "uintmax_t");
 }
+
+void
+string_to_chars (char *str, char *p, size_t s)
+{
+  strncpy (p, str, s);
+  p[s-1] = 0;
+}
+
 \f
 /* Writing routines.  */
 
-/*-----------------------------------------------------------------------.
-| Just zeroes out the buffer so we don't confuse ourselves with leftover |
-| data.                                                                         |
-`-----------------------------------------------------------------------*/
-
+/* Zero out the buffer so we don't confuse ourselves with leftover
+   data.  */
 static void
 clear_buffer (char *buffer)
 {
   memset (buffer, 0, BLOCKSIZE);
 }
 
-/*-------------------------------------------------------------------------.
-| Write the EOT block(s).  We zero at least two blocks, through                   |
-| the end of the record.  Old tar, as previous versions of GNU tar, writes |
-| garbage after two zeroed blocks.                                        |
-`-------------------------------------------------------------------------*/
-
+/* Write the EOT block(s).  Zero at least two blocks, through the end
+   of the record.  Old tar, as previous versions of GNU tar, writes
+   garbage after two zeroed blocks.  */
 void
 write_eot (void)
 {
@@ -361,13 +353,11 @@ write_eot (void)
   set_next_block_after (pointer);
 }
 
-/*-----------------------------------------------------.
-| Write a GNUTYPE_LONGLINK or GNUTYPE_LONGNAME block.  |
-`-----------------------------------------------------*/
+/* Write a GNUTYPE_LONGLINK or GNUTYPE_LONGNAME block.  */
 
 /* FIXME: Cross recursion between start_header and write_long!  */
 
-static union block *start_header PARAMS ((const char *, struct stat *));
+static union block *start_header (const char *, struct tar_stat_info *);
 
 static void
 write_long (const char *p, char type)
@@ -375,14 +365,14 @@ write_long (const char *p, char type)
   size_t size = strlen (p) + 1;
   size_t bufsize;
   union block *header;
-  struct stat foo;
+  struct tar_stat_info foo;
 
   memset (&foo, 0, sizeof foo);
-  foo.st_size = size;
+  foo.stat.st_size = size;
 
   header = start_header ("././@LongLink", &foo);
   header->header.typeflag = type;
-  finish_header (header);
+  finish_header (header, -1);
 
   header = find_next_block ();
 
@@ -401,6 +391,71 @@ write_long (const char *p, char type)
   memset (header->buffer + size, 0, bufsize - size);
   set_next_block_after (header + (size - 1) / BLOCKSIZE);
 }
+
+/* Write a long link name, depending on the current archive format */
+static void
+write_long_link (struct tar_stat_info *st)
+{
+  if (archive_format == POSIX_FORMAT)
+    xheader_store ("linkpath", st);
+  else
+    write_long (st->link_name, GNUTYPE_LONGNAME);
+}
+
+/* NOTE: Cross recursion between start_header and write_extended  */
+
+static union block *
+write_extended (union block *old_header, char type)
+{
+  union block *header, hp;
+  struct tar_stat_info foo;
+  size_t size;
+  char *p;
+
+  if (extended_header.buffer || extended_header.stk == NULL)
+    return old_header; /* Prevent recursion */
+  
+  xheader_finish (&extended_header);
+  size = extended_header.size;
+  memset (&foo, 0, sizeof foo);
+  foo.stat.st_mode = S_IFREG|S_IRUSR|S_IWUSR|S_IRGRP|S_IROTH;
+  time (&foo.stat.st_ctime);
+  foo.stat.st_atime = foo.stat.st_ctime;
+  foo.stat.st_mtime = foo.stat.st_ctime;
+  foo.stat.st_size = size;
+
+  memcpy (hp.buffer, old_header, sizeof (hp));
+  
+  header = start_header ("././@PaxHeader", &foo);
+  header->header.typeflag = type;
+
+  finish_header (header, -1);
+
+  p = extended_header.buffer;
+
+  do
+    {
+      size_t len;
+      
+      header = find_next_block ();
+      len = BLOCKSIZE;
+      if (len > size)
+       len = size;
+      memcpy (header->buffer, p, len);
+      if (len < BLOCKSIZE)
+       memset (header->buffer + len, 0, BLOCKSIZE - len);
+      p += len;
+      size -= len;
+      set_next_block_after (header);
+    }
+  while (size > 0);
+
+  xheader_destroy (&extended_header);
+  header = find_next_block ();
+  memcpy (header, &hp.buffer, sizeof (hp.buffer));
+  return header;
+}
+
 \f
 /* Header handling.  */
 
@@ -408,57 +463,38 @@ write_long (const char *p, char type)
    and return its address.  */
 
 static union block *
-start_header (const char *name, struct stat *st)
+start_header (const char *name, struct tar_stat_info *st)
 {
   union block *header;
 
-  if (!absolute_names_option)
+  name = safer_name_suffix (name, 0);
+  assign_string (&st->file_name, name);
+  
+  if (sizeof header->header.name <= strlen (name))
     {
-      size_t prefix_len = FILESYSTEM_PREFIX_LEN (name);
-
-      if (prefix_len)
-       {
-         static int warned_once;
-         if (!warned_once)
-           {
-             warned_once = 1;
-             WARN ((0, 0, _("Removing `%.*s' prefix from archive names"),
-                    (int) prefix_len, name));
-           }
-         name += prefix_len;
-       }
-
-      while (*name == '/')
-       {
-         static int warned_once;
-         if (!warned_once)
-           {
-             warned_once = 1;
-             WARN ((0, 0, _("Removing leading `/' from archive names")));
-           }
-         name++;
-       }
+      if (archive_format == POSIX_FORMAT)
+       xheader_store ("path", st);
+      else
+       write_long (name, GNUTYPE_LONGNAME);
     }
-
-  if (sizeof header->header.name <= strlen (name))
-    write_long (name, GNUTYPE_LONGNAME);
+  
   header = find_next_block ();
   memset (header->buffer, 0, sizeof (union block));
 
-  assign_string (&current_file_name, name);
+  assign_string (&current_stat_info.file_name, name);
 
   strncpy (header->header.name, name, NAME_FIELD_SIZE);
   header->header.name[NAME_FIELD_SIZE - 1] = '\0';
-
+  
   /* Override some stat fields, if requested to do so.  */
 
   if (owner_option != (uid_t) -1)
-    st->st_uid = owner_option;
+    st->stat.st_uid = owner_option;
   if (group_option != (gid_t) -1)
-    st->st_gid = group_option;
+    st->stat.st_gid = group_option;
   if (mode_option)
-    st->st_mode = ((st->st_mode & ~MODE_ALL)
-                  | mode_adjust (st->st_mode, mode_option));
+    st->stat.st_mode = ((st->stat.st_mode & ~MODE_ALL)
+                  | mode_adjust (st->stat.st_mode, mode_option));
 
   /* Paul Eggert tried the trivial test ($WRITER cf a b; $READER tvf a)
      for a few tars and came up with the following interoperability
@@ -484,20 +520,60 @@ start_header (const char *name, struct stat *st)
      acceptor for Paul's test.  */
 
   if (archive_format == V7_FORMAT)
-    MODE_TO_CHARS (st->st_mode & MODE_ALL, header->header.mode);
+    MODE_TO_CHARS (st->stat.st_mode & MODE_ALL, header->header.mode);
   else
-    MODE_TO_CHARS (st->st_mode, header->header.mode);
+    MODE_TO_CHARS (st->stat.st_mode, header->header.mode);
 
-  UID_TO_CHARS (st->st_uid, header->header.uid);
-  GID_TO_CHARS (st->st_gid, header->header.gid);
-  OFF_TO_CHARS (st->st_size, header->header.size);
-  TIME_TO_CHARS (st->st_mtime, header->header.mtime);
+  if (st->stat.st_uid > MAXOCTAL7 && archive_format == POSIX_FORMAT)
+    xheader_store ("uid", st);
+  else
+    UID_TO_CHARS (st->stat.st_uid, header->header.uid);
+  
+  if (st->stat.st_gid > MAXOCTAL7 && archive_format == POSIX_FORMAT)
+    xheader_store ("gid", st);
+  else
+    GID_TO_CHARS (st->stat.st_gid, header->header.gid);
 
-  if (incremental_option)
+  if (st->stat.st_size > MAXOCTAL11 && archive_format == POSIX_FORMAT)
+    xheader_store ("size", st);
+  else
+    OFF_TO_CHARS (st->stat.st_size, header->header.size);
+
+  TIME_TO_CHARS (st->stat.st_mtime, header->header.mtime);
+
+  /* FIXME */
+  if (S_ISCHR (st->stat.st_mode)
+      || S_ISBLK (st->stat.st_mode))
+    {
+      st->devmajor = major (st->stat.st_rdev);
+      st->devminor = minor (st->stat.st_rdev);
+
+      if (st->devmajor > MAXOCTAL7 && archive_format == POSIX_FORMAT)
+       xheader_store ("devmajor", st);
+      else
+       MAJOR_TO_CHARS (st->devmajor, header->header.devmajor);
+
+      if (st->devminor > MAXOCTAL7 && archive_format == POSIX_FORMAT)
+       xheader_store ("devminor", st);
+      else
+       MAJOR_TO_CHARS (st->devminor, header->header.devminor);
+    }
+  else
+    {
+      MAJOR_TO_CHARS (0, header->header.devmajor);
+      MINOR_TO_CHARS (0, header->header.devminor);
+    }
+  
+  if (archive_format == POSIX_FORMAT)
+    {
+      xheader_store ("atime", st);
+      xheader_store ("ctime", st);
+    }
+  else if (incremental_option)
     if (archive_format == OLDGNU_FORMAT)
       {
-       TIME_TO_CHARS (st->st_atime, header->oldgnu_header.atime);
-       TIME_TO_CHARS (st->st_ctime, header->oldgnu_header.ctime);
+       TIME_TO_CHARS (st->stat.st_atime, header->oldgnu_header.atime);
+       TIME_TO_CHARS (st->stat.st_ctime, header->oldgnu_header.ctime);
       }
 
   header->header.typeflag = archive_format == V7_FORMAT ? AREGTYPE : REGTYPE;
@@ -528,25 +604,54 @@ start_header (const char *name, struct stat *st)
     }
   else
     {
-      uid_to_uname (st->st_uid, header->header.uname);
-      gid_to_gname (st->st_gid, header->header.gname);
+      uid_to_uname (st->stat.st_uid, &st->uname);
+      gid_to_gname (st->stat.st_gid, &st->gname);
+      
+      if (archive_format == POSIX_FORMAT
+         && strlen (st->uname) > UNAME_FIELD_SIZE)
+       xheader_store ("uname", st);
+      else
+       UNAME_TO_CHARS (st->uname, header->header.uname);
+
+      if (archive_format == POSIX_FORMAT
+         && strlen (st->gname) > GNAME_FIELD_SIZE)
+       xheader_store ("gname", st);
+      else
+       GNAME_TO_CHARS (st->gname, header->header.gname);
     }
 
   return header;
 }
 
-/*-------------------------------------------------------------------------.
-| Finish off a filled-in header block and write it out.  We also print the |
-| file name and/or full info if verbose is on.                            |
-`-------------------------------------------------------------------------*/
-
+/* Finish off a filled-in header block and write it out.  We also
+   print the file name and/or full info if verbose is on.  If BLOCK_ORDINAL
+   is not negative, is the block ordinal of the first record for this
+   file, which may be a preceding long name or long link record.  */
 void
-finish_header (union block *header)
+finish_header (union block *header, off_t block_ordinal)
 {
   size_t i;
   int sum;
   char *p;
 
+  /* Note: It is important to do this before the call to write_extended(),
+     so that the actual ustar header is printed */
+  if (verbose_option
+      && header->header.typeflag != GNUTYPE_LONGLINK
+      && header->header.typeflag != GNUTYPE_LONGNAME
+      && header->header.typeflag != XHDTYPE
+      && header->header.typeflag != XGLTYPE)
+    {
+      /* These globals are parameters to print_header, sigh.  */
+
+      current_header = header;
+      /* current_stat_info is already set up.  */
+      current_format = archive_format;
+      print_header (block_ordinal);
+    }
+
+  header = write_extended (header, XHDTYPE);
+  
   memcpy (header->header.chksum, CHKBLANKS, sizeof header->header.chksum);
 
   sum = 0;
@@ -567,29 +672,14 @@ finish_header (union block *header)
 
   uintmax_to_chars ((uintmax_t) sum, header->header.chksum, 7);
 
-  if (verbose_option
-      && header->header.typeflag != GNUTYPE_LONGLINK
-      && header->header.typeflag != GNUTYPE_LONGNAME)
-    {
-      /* These globals are parameters to print_header, sigh.  */
-
-      current_header = header;
-      /* current_stat is already set up.  */
-      current_format = archive_format;
-      print_header ();
-    }
-
   set_next_block_after (header);
 }
 \f
 /* Sparse file processing.  */
 
-/*-------------------------------------------------------------------------.
-| Takes a blockful of data and basically cruises through it to see if it's |
-| made *entirely* of zeros, returning a 0 the instant it finds something   |
-| that is a nonzero, i.e., useful data.                                           |
-`-------------------------------------------------------------------------*/
-
+/* Takes a blockful of data and basically cruises through it to see if
+   it's made *entirely* of zeros, returning a 0 the instant it finds
+   something that is a nonzero, i.e., useful data.  */
 static int
 zero_block_p (char *buffer)
 {
@@ -601,49 +691,29 @@ zero_block_p (char *buffer)
   return 1;
 }
 
-/*---.
-| ?  |
-`---*/
-
-static void
+void
 init_sparsearray (void)
 {
-  int counter;
-
-  sp_array_size = 10;
-
-  /* Make room for our scratch space -- initially is 10 elts long.  */
-
-  sparsearray = xmalloc (sp_array_size * sizeof (struct sp_array));
-  for (counter = 0; counter < sp_array_size; counter++)
-    {
-      sparsearray[counter].offset = 0;
-      sparsearray[counter].numbytes = 0;
-    }
+  if (! sp_array_size)
+    sp_array_size = SPARSES_IN_OLDGNU_HEADER;
+  sparsearray = xmalloc (sp_array_size * sizeof *sparsearray);
 }
 
-/*---.
-| ?  |
-`---*/
-
-static void
-find_new_file_size (off_t *filesize, int highest_index)
+static off_t
+find_new_file_size (int sparses)
 {
-  int counter;
-
-  *filesize = 0;
-  for (counter = 0;
-       sparsearray[counter].numbytes && counter <= highest_index;
-       counter++)
-    *filesize += sparsearray[counter].numbytes;
+  int i;
+  off_t s = 0;
+  for (i = 0; i < sparses; i++)
+    s += sparsearray[i].numbytes;
+  return s;
 }
 
-/*-----------------------------------------------------------------------.
-| Make one pass over the file NAME, studying where any non-zero data is, |
-| that is, how far into the file each instance of data is, and how many  |
-| bytes are there.  Save this information in the sparsearray, which will |
-| later be translated into header information.                           |
-`-----------------------------------------------------------------------*/
+/* Make one pass over the file NAME, studying where any non-zero data
+   is, that is, how far into the file each instance of data is, and
+   how many bytes are there.  Save this information in the
+   sparsearray, which will later be translated into header
+   information.  */
 
 /* There is little point in trimming small amounts of null data at the head
    and tail of blocks, only avoid dumping full null blocks.  */
@@ -657,7 +727,7 @@ deal_with_sparse (char *name, union block *header)
   size_t numbytes = 0;
   off_t offset = 0;
   int file;
-  int sparse_index = 0;
+  int sparses = 0;
   ssize_t count;
   char buffer[BLOCKSIZE];
 
@@ -671,18 +741,22 @@ deal_with_sparse (char *name, union block *header)
   init_sparsearray ();
   clear_buffer (buffer);
 
-  while (count = safe_read (file, buffer, sizeof buffer), count != 0)
+  for (;;)
     {
       /* Realloc the scratch area as necessary.  FIXME: should reallocate
         only at beginning of a new instance of non-zero data.  */
 
-      if (sparse_index > sp_array_size - 1)
+      if (sp_array_size <= sparses)
        {
          sparsearray =
            xrealloc (sparsearray,
                      2 * sp_array_size * sizeof (struct sp_array));
          sp_array_size *= 2;
        }
+      
+      count = safe_read (file, buffer, sizeof buffer);
+      if (count <= 0)
+       break;
 
       /* Process one block.  */
 
@@ -692,14 +766,14 @@ deal_with_sparse (char *name, union block *header)
          {
            if (numbytes)
              {
-               sparsearray[sparse_index++].numbytes = numbytes;
+               sparsearray[sparses++].numbytes = numbytes;
                numbytes = 0;
              }
          }
        else
          {
            if (!numbytes)
-             sparsearray[sparse_index].offset = offset;
+             sparsearray[sparses].offset = offset;
            numbytes += count;
          }
 
@@ -710,7 +784,7 @@ deal_with_sparse (char *name, union block *header)
        if (!zero_block_p (buffer))
          {
            if (!numbytes)
-             sparsearray[sparse_index].offset = offset;
+             sparsearray[sparses].offset = offset;
            numbytes += count;
          }
        else
@@ -731,53 +805,36 @@ deal_with_sparse (char *name, union block *header)
     }
 
   if (numbytes)
-    sparsearray[sparse_index++].numbytes = numbytes;
+    sparsearray[sparses++].numbytes = numbytes;
   else
     {
-      sparsearray[sparse_index].offset = offset - 1;
-      sparsearray[sparse_index++].numbytes = 1;
+      sparsearray[sparses].offset = offset - 1;
+      sparsearray[sparses++].numbytes = 1;
     }
 
-  close (file);
-  return sparse_index - 1;
+  return close (file) == 0 && 0 <= count ? sparses : 0;
 }
 
-/*---.
-| ?  |
-`---*/
-
 static int
 finish_sparse_file (int file, off_t *sizeleft, off_t fullsize, char *name)
 {
   union block *start;
   size_t bufsize;
-  int sparse_index = 0;
+  int sparses = 0;
   ssize_t count;
 
   while (*sizeleft > 0)
     {
       start = find_next_block ();
       memset (start->buffer, 0, BLOCKSIZE);
-      bufsize = sparsearray[sparse_index].numbytes;
-      if (!bufsize)
-       {
-         /* We blew it, maybe.  */
-         char buf1[UINTMAX_STRSIZE_BOUND];
-         char buf2[UINTMAX_STRSIZE_BOUND];
-
-         ERROR ((0, 0, _("Wrote %s of %s bytes to file %s"),
-                 STRINGIFY_BIGINT (fullsize - *sizeleft, buf1),
-                 STRINGIFY_BIGINT (fullsize, buf2),
-                 name));
-         break;
-       }
+      bufsize = sparsearray[sparses].numbytes;
+      if (! bufsize)
+       abort ();
 
-      if (lseek (file, sparsearray[sparse_index++].offset, SEEK_SET) < 0)
+      if (lseek (file, sparsearray[sparses++].offset, SEEK_SET) < 0)
        {
-         char buf[UINTMAX_STRSIZE_BOUND];
-         ERROR ((0, errno, _("lseek error at byte %s in file %s"),
-                 STRINGIFY_BIGINT (sparsearray[sparse_index - 1].offset, buf),
-                 name));
+         (ignore_failed_read_option ? seek_warn_details : seek_error_details)
+           (name, sparsearray[sparses - 1].offset);
          break;
        }
 
@@ -786,28 +843,13 @@ finish_sparse_file (int file, off_t *sizeleft, off_t fullsize, char *name)
 
       while (bufsize > BLOCKSIZE)
        {
-#if 0
-         if (amount_read)
-           {
-             count = safe_read (file, start->buffer + amount_read,
-                                BLOCKSIZE - amount_read);
-             bufsize -= BLOCKSIZE - amount_read;
-             amount_read = 0;
-             set_next_block_after (start);
-             start = find_next_block ();
-             memset (start->buffer, 0, BLOCKSIZE);
-           }
-#endif
-         /* Store the data.  */
-
          count = safe_read (file, start->buffer, BLOCKSIZE);
          if (count < 0)
            {
-             char buf[UINTMAX_STRSIZE_BOUND];
-             ERROR ((0, errno,
-                     _("Read error at byte %s, reading %lu bytes, in file %s"),
-                     STRINGIFY_BIGINT (fullsize - *sizeleft, buf),
-                     (unsigned long) bufsize, name));
+             (ignore_failed_read_option
+              ? read_warn_details
+              : read_error_details)
+               (name, fullsize - *sizeleft, bufsize);
              return 1;
            }
          bufsize -= count;
@@ -827,34 +869,15 @@ finish_sparse_file (int file, off_t *sizeleft, off_t fullsize, char *name)
 
       if (count < 0)
        {
-         char buf[UINTMAX_STRSIZE_BOUND];
-         
-         ERROR ((0, errno,
-                 _("Read error at byte %s, reading %lu bytes, in file %s"),
-                 STRINGIFY_BIGINT (fullsize - *sizeleft, buf),
-                 (unsigned long) bufsize, name));
+         (ignore_failed_read_option
+          ? read_warn_details
+          : read_error_details)
+           (name, fullsize - *sizeleft, bufsize);
          return 1;
        }
-#if 0
-      if (amount_read >= BLOCKSIZE)
-       {
-         amount_read = 0;
-         set_next_block_after (start + (count - 1) / BLOCKSIZE);
-         if (count != bufsize)
-           {
-             ERROR ((0, 0,
-                     _("File %s shrunk, padding with zeros"),
-                     name));
-             return 1;
-           }
-         start = find_next_block ();
-       }
-      else
-       amount_read += bufsize;
-#endif
+
       *sizeleft -= count;
       set_next_block_after (start);
-
     }
   free (sparsearray);
 #if 0
@@ -865,10 +888,6 @@ finish_sparse_file (int file, off_t *sizeleft, off_t fullsize, char *name)
 \f
 /* Main functions of this module.  */
 
-/*---.
-| ?  |
-`---*/
-
 void
 create_archive (void)
 {
@@ -878,9 +897,9 @@ create_archive (void)
 
   if (incremental_option)
     {
-      char *buffer = xmalloc (PATH_MAX);
+      size_t buffer_size = 1000;
+      char *buffer = xmalloc (buffer_size);
       const char *q;
-      char *bufp;
 
       collect_and_sort_names ();
 
@@ -892,18 +911,34 @@ create_archive (void)
       while (p = name_from_list (), p)
        if (!excluded_name (p))
          {
-           strcpy (buffer, p);
-           if (p[strlen (p) - 1] != '/')
-             strcat (buffer, "/");
-           bufp = buffer + strlen (buffer);
+           size_t plen = strlen (p);
+           if (buffer_size <= plen)
+             {
+               while ((buffer_size *= 2) <= plen)
+                 continue;
+               buffer = xrealloc (buffer, buffer_size);
+             }
+           memcpy (buffer, p, plen);
+           if (! ISSLASH (buffer[plen - 1]))
+             buffer[plen++] = '/';
            q = gnu_list_name->dir_contents;
            if (q)
-             for (; *q; q += strlen (q) + 1)
-               if (*q == 'Y')
-                 {
-                   strcpy (bufp, q + 1);
-                   dump_file (buffer, -1, (dev_t) 0);
-                 }
+             while (*q)
+               {
+                 size_t qlen = strlen (q);
+                 if (*q == 'Y')
+                   {
+                     if (buffer_size < plen + qlen)
+                       {
+                         while ((buffer_size *=2 ) < plen + qlen)
+                           continue;
+                         buffer = xrealloc (buffer, buffer_size);
+                       }
+                     strcpy (buffer + plen, q + 1);
+                     dump_file (buffer, -1, (dev_t) 0);
+                   }
+                 q += qlen + 1;
+               }
          }
       free (buffer);
     }
@@ -921,13 +956,36 @@ create_archive (void)
     write_directory_file ();
 }
 
+
+/* Calculate the hash of a link.  */
+static unsigned
+hash_link (void const *entry, unsigned n_buckets)
+{
+  struct link const *link = entry;
+  return (uintmax_t) (link->dev ^ link->ino) % n_buckets;
+}
+
+/* Compare two links for equality.  */
+static bool
+compare_links (void const *entry1, void const *entry2)
+{
+  struct link const *link1 = entry1;
+  struct link const *link2 = entry2;
+  return ((link1->dev ^ link2->dev) | (link1->ino ^ link2->ino)) == 0;
+}
+
+/* Table of all non-directories that we've written so far.  Any time
+   we see another, we check the table and avoid dumping the data
+   again if we've done it once already.  */
+static Hash_table *link_table;
+
 /* Dump a single file, recursing on directories.  P is the file name
    to dump.  TOP_LEVEL tells whether this is a top-level call; zero
-   means no, positive means yes, and negative means an incremental
-   dump where it's irrelevant.  PARENT_DEVICE is the device of P's
+   means no, positive means yes, and negative means the top level
+   of an incremental dump.  PARENT_DEVICE is the device of P's
    parent directory; it is examined only if TOP_LEVEL is zero.
 
-   Set global CURRENT_STAT to stat output for this file.  */
+   Set global CURRENT_STAT_INFO to stat output for this file.  */
 
 /* FIXME: One should make sure that for *every* path leading to setting
    exit_status to failure, a clear diagnostic has been issued.  */
@@ -939,29 +997,31 @@ dump_file (char *p, int top_level, dev_t parent_device)
   char type;
   union block *exhdr;
   char save_typeflag;
+  time_t original_ctime;
   struct utimbuf restore_times;
-  off_t restore_size;
+  off_t block_ordinal = -1;
 
-  /* FIXME: `header' and `upperbound' might be used uninitialized in this
+  /* FIXME: `header' might be used uninitialized in this
      function.  Reported by Bruno Haible.  */
 
   if (interactive_option && !confirm ("add", p))
     return;
 
-  if (deref_stat (dereference_option, p, &current_stat) != 0)
+  if (deref_stat (dereference_option, p, &current_stat_info.stat) != 0)
     {
-      WARN ((0, errno, _("Cannot add file %s"), p));
-      if (!ignore_failed_read_option)
-       exit_status = TAREXIT_FAILURE;
+      if (ignore_failed_read_option)
+       stat_warn (p);
+      else
+       stat_error (p);
       return;
     }
 
-  restore_times.actime = current_stat.st_atime;
-  restore_times.modtime = current_stat.st_mtime;
-  restore_size = current_stat.st_size;
+  original_ctime = current_stat_info.stat.st_ctime;
+  restore_times.actime = current_stat_info.stat.st_atime;
+  restore_times.modtime = current_stat_info.stat.st_mtime;
 
 #ifdef S_ISHIDDEN
-  if (S_ISHIDDEN (current_stat.st_mode))
+  if (S_ISHIDDEN (current_stat_info.stat.st_mode))
     {
       char *new = (char *) alloca (strlen (p) + 2);
       if (new)
@@ -977,47 +1037,44 @@ dump_file (char *p, int top_level, dev_t parent_device)
      put in the archive.  */
 
   if ((0 < top_level || !incremental_option)
-      && !S_ISDIR (current_stat.st_mode)
-      && current_stat.st_mtime < newer_mtime_option
-      && (!after_date_option || current_stat.st_ctime < newer_ctime_option))
+      && !S_ISDIR (current_stat_info.stat.st_mode)
+      && current_stat_info.stat.st_mtime < newer_mtime_option
+      && (!after_date_option || current_stat_info.stat.st_ctime < newer_ctime_option))
     {
       if (0 < top_level)
-       WARN ((0, 0, _("%s: is unchanged; not dumped"), p));
+       WARN ((0, 0, _("%s: file is unchanged; not dumped"),
+              quotearg_colon (p)));
       /* FIXME: recheck this return.  */
       return;
     }
 
-#if !MSDOS
   /* See if we are trying to dump the archive.  */
-
-  if (ar_dev && current_stat.st_dev == ar_dev && current_stat.st_ino == ar_ino)
+  if (sys_file_is_archive (&current_stat_info))
     {
-      WARN ((0, 0, _("%s is the archive; not dumped"), p));
+      WARN ((0, 0, _("%s: file is the archive; not dumped"),
+            quotearg_colon (p)));
       return;
     }
-#endif
 
-  if (S_ISDIR (current_stat.st_mode))
+  if (S_ISDIR (current_stat_info.stat.st_mode))
     {
-      DIR *directory;
-      struct dirent *entry;
+      char *directory;
+      char const *entry;
+      size_t entrylen;
       char *namebuf;
       size_t buflen;
       size_t len;
-      dev_t our_device = current_stat.st_dev;
+      dev_t our_device = current_stat_info.stat.st_dev;
 
-      /* If this tar program is installed suid root, like for Amanda, the
-        access might look like denied, while it is not really.
+      errno = 0;
 
-        FIXME: I have the feeling this test is done too early.  Couldn't it
-        just be bundled in later actions?  I guess that the proper support
-        of --ignore-failed-read is the key of the current writing.  */
-
-      if (access (p, R_OK) == -1 && geteuid () != 0)
+      directory = savedir (p);
+      if (! directory)
        {
-         WARN ((0, errno, _("Cannot add directory %s"), p));
-         if (!ignore_failed_read_option)
-           exit_status = TAREXIT_FAILURE;
+         if (ignore_failed_read_option)
+           savedir_warn (p);
+         else
+           savedir_error (p);
          return;
        }
 
@@ -1026,8 +1083,8 @@ dump_file (char *p, int top_level, dev_t parent_device)
       len = strlen (p);
       buflen = len + NAME_FIELD_SIZE;
       namebuf = xmalloc (buflen + 1);
-      strncpy (namebuf, p, buflen);
-      while (len >= 1 && namebuf[len - 1] == '/')
+      memcpy (namebuf, p, len);
+      while (len >= 1 && ISSLASH (namebuf[len - 1]))
        len--;
       namebuf[len++] = '/';
       namebuf[len] = '\0';
@@ -1044,18 +1101,19 @@ dump_file (char *p, int top_level, dev_t parent_device)
             reasons either, so until these are reported (anew?), just allow
             directory blocks to be written even with old archives.  */
 
-         current_stat.st_size = 0;     /* force 0 size on dir */
+         block_ordinal = current_block_ordinal ();
+         current_stat_info.stat.st_size = 0;   /* force 0 size on dir */
 
          /* FIXME: If people could really read standard archives, this
             should be:
 
             header
-              = start_header (standard_option ? p : namebuf, &current_stat);
+              = start_header (standard_option ? p : namebuf, &current_stat_info);
 
             but since they'd interpret DIRTYPE blocks as regular
             files, we'd better put the / on the name.  */
 
-         header = start_header (namebuf, &current_stat);
+         header = start_header (namebuf, &current_stat_info);
 
          if (incremental_option)
            header->header.typeflag = GNUTYPE_DUMPDIR;
@@ -1065,7 +1123,7 @@ dump_file (char *p, int top_level, dev_t parent_device)
          /* If we're gnudumping, we aren't done yet so don't close it.  */
 
          if (!incremental_option)
-           finish_header (header);     /* done with directory header */
+           finish_header (header, block_ordinal);
        }
 
       if (incremental_option && gnu_list_name->dir_contents)
@@ -1079,17 +1137,16 @@ dump_file (char *p, int top_level, dev_t parent_device)
 
          buffer = gnu_list_name->dir_contents; /* FOO */
          totsize = 0;
-         for (p_buffer = buffer; p_buffer && *p_buffer;)
-           {
-             size_t tmp;
-
-             tmp = strlen (p_buffer) + 1;
-             totsize += tmp;
-             p_buffer += tmp;
-           }
+         if (buffer)
+           for (p_buffer = buffer; *p_buffer; )
+             {
+               size_t size = strlen (p_buffer) + 1;
+               totsize += size;
+               p_buffer += size;
+             }
          totsize++;
          OFF_TO_CHARS (totsize, header->header.size);
-         finish_header (header);
+         finish_header (header, block_ordinal);
          p_buffer = buffer;
          sizeleft = totsize;
          while (sizeleft > 0)
@@ -1116,59 +1173,49 @@ dump_file (char *p, int top_level, dev_t parent_device)
            }
          if (multi_volume_option)
            assign_string (&save_name, 0);
-         if (atime_preserve_option)
-           utime (p, &restore_times);
-         return;
+         goto finish_dir;
        }
 
       /* See if we are about to recurse into a directory, and avoid doing
         so if the user wants that we do not descend into directories.  */
 
-      if (no_recurse_option)
-       return;
+      if (! recursion_option)
+       goto finish_dir;
 
       /* See if we are crossing from one file system to another, and
         avoid doing so if the user only wants to dump one file system.  */
 
       if (one_file_system_option && !top_level
-         && parent_device != current_stat.st_dev)
+         && parent_device != current_stat_info.stat.st_dev)
        {
          if (verbose_option)
-           WARN ((0, 0, _("%s: On a different filesystem; not dumped"), p));
-         return;
+           WARN ((0, 0,
+                  _("%s: file is on a different filesystem; not dumped"),
+                  quotearg_colon (p)));
+         goto finish_dir;
        }
 
       /* Now output all the files in the directory.  */
 
-      errno = 0;               /* FIXME: errno should be read-only */
-
-      directory = opendir (p);
-      if (!directory)
-       {
-         ERROR ((0, errno, _("Cannot open directory %s"), p));
-         return;
-       }
-
       /* FIXME: Should speed this up by cd-ing into the dir.  */
 
-      while (entry = readdir (directory), entry)
+      for (entry = directory;
+          (entrylen = strlen (entry)) != 0;
+          entry += entrylen + 1)
        {
-         /* Skip `.', `..', and excluded file names.  */
-
-         if (is_dot_or_dotdot (entry->d_name))
-           continue;
-
-         if ((int) NAMLEN (entry) + len >= buflen)
+         if (buflen < len + entrylen)
            {
-             buflen = len + NAMLEN (entry);
+             buflen = len + entrylen;
              namebuf = xrealloc (namebuf, buflen + 1);
            }
-         strcpy (namebuf + len, entry->d_name);
+         strcpy (namebuf + len, entry);
          if (!excluded_name (namebuf))
            dump_file (namebuf, 0, our_device);
        }
 
-      closedir (directory);
+    finish_dir:
+
+      free (directory);
       free (namebuf);
       if (atime_preserve_option)
        utime (p, &restore_times);
@@ -1178,77 +1225,52 @@ dump_file (char *p, int top_level, dev_t parent_device)
     return;
   else
     {
-      /* Check for multiple links.
-
-        We maintain a list of all such files that we've written so far.  Any
-        time we see another, we check the list and avoid dumping the data
-        again if we've done it once already.  */
+      /* Check for multiple links.  */
 
-      if (1 < current_stat.st_nlink)
+      if (1 < current_stat_info.stat.st_nlink && link_table)
        {
-         struct link *lp;
-
-         /* FIXME: First quick and dirty.  Hashing, etc later.  */
+         struct link lp;
+         struct link *dup;
+         lp.ino = current_stat_info.stat.st_ino;
+         lp.dev = current_stat_info.stat.st_dev;
 
-         for (lp = linklist; lp; lp = lp->next)
-           if (lp->ino == current_stat.st_ino
-               && lp->dev == current_stat.st_dev)
-             {
-               char *link_name = lp->name;
-
-               /* We found a link.  */
-
-               while (!absolute_names_option && *link_name == '/')
-                 {
-                   static int warned_once;
-                   if (!warned_once)
-                     {
-                       warned_once = 1;
-                       WARN ((0, 0,
-                              _("Removing leading `/' from link names")));
-                     }
-                   link_name++;
-                 }
-               if (strlen (link_name) >= NAME_FIELD_SIZE)
-                 write_long (link_name, GNUTYPE_LONGLINK);
-               assign_string (&current_link_name, link_name);
-
-               current_stat.st_size = 0;
-               header = start_header (p, &current_stat);
-               strncpy (header->header.linkname,
-                        link_name, NAME_FIELD_SIZE);
+         if ((dup = hash_lookup (link_table, &lp)))
+           {
+             /* We found a link.  */
+             char const *link_name = safer_name_suffix (dup->name, 1);
 
-               /* Force null truncated.  */
+             dup->nlink--;
+             
+             block_ordinal = current_block_ordinal ();
+             assign_string (&current_stat_info.link_name, link_name);
+             if (NAME_FIELD_SIZE <= strlen (link_name))
+               write_long_link (&current_stat_info);
 
-               header->header.linkname[NAME_FIELD_SIZE - 1] = 0;
+             current_stat_info.stat.st_size = 0;
+             header = start_header (p, &current_stat_info);
+             strncpy (header->header.linkname, link_name, NAME_FIELD_SIZE);
 
-               header->header.typeflag = LNKTYPE;
-               finish_header (header);
+             /* Force null termination.  */
+             header->header.linkname[NAME_FIELD_SIZE - 1] = 0;
 
-               /* FIXME: Maybe remove from list after all links found?  */
+             header->header.typeflag = LNKTYPE;
+             finish_header (header, block_ordinal);
 
-               if (remove_files_option)
-                 if (unlink (p) == -1)
-                   ERROR ((0, errno, _("Cannot remove %s"), p));
+             /* FIXME: Maybe remove from table after all links found?  */
 
-               /* We dumped it.  */
-               return;
-             }
+             if (remove_files_option && unlink (p) != 0)
+               unlink_error (p);
 
-         /* Not found.  Add it to the list of possible links.  */
-
-         lp = xmalloc (sizeof (struct link) + strlen (p));
-         lp->ino = current_stat.st_ino;
-         lp->dev = current_stat.st_dev;
-         strcpy (lp->name, p);
-         lp->next = linklist;
-         linklist = lp;
+             /* We dumped it, and we don't need to put it in the
+                 table again.  */
+             return;
+           }
        }
 
       /* This is not a link to a previously dumped file, so dump it.  */
 
-      if (S_ISREG (current_stat.st_mode)
-         || S_ISCTG (current_stat.st_mode))
+      if (S_ISREG (current_stat_info.stat.st_mode)
+         || S_ISCTG (current_stat_info.stat.st_mode))
        {
          int f;                        /* file descriptor */
          size_t bufsize;
@@ -1257,7 +1279,7 @@ dump_file (char *p, int top_level, dev_t parent_device)
          union block *start;
          int header_moved;
          char isextended = 0;
-         int upperbound;
+         int sparses = 0;
 
          header_moved = 0;
 
@@ -1265,7 +1287,7 @@ dump_file (char *p, int top_level, dev_t parent_device)
            {
              /* Check the size of the file against the number of blocks
                 allocated for it, counting both data and indirect blocks.
-                If there is a smaller number of blocks that would be
+                If there is a smaller number of blocks than would be
                 necessary to accommodate a file of this size, this is safe
                 to say that we have a sparse file: at least one of those
                 blocks in the file is just a useless hole.  For sparse
@@ -1294,27 +1316,27 @@ dump_file (char *p, int top_level, dev_t parent_device)
                 st_blocks, so `du' and `ls -s' give wrong results.  So, the
                 --sparse option would not work on a minix filesystem.  */
 
-             if (ST_NBLOCKS (current_stat)
-                 < (current_stat.st_size / ST_NBLOCKSIZE
-                    + (current_stat.st_size % ST_NBLOCKSIZE != 0)))
+             if (ST_NBLOCKS (current_stat_info.stat)
+                 < (current_stat_info.stat.st_size / ST_NBLOCKSIZE
+                    + (current_stat_info.stat.st_size % ST_NBLOCKSIZE != 0)))
                {
-                 off_t filesize = current_stat.st_size;
                  int counter;
 
-                 header = start_header (p, &current_stat);
+                 block_ordinal = current_block_ordinal ();
+                 header = start_header (p, &current_stat_info);
                  header->header.typeflag = GNUTYPE_SPARSE;
                  header_moved = 1;
 
                  /* Call the routine that figures out the layout of the
-                    sparse file in question.  UPPERBOUND is the index of the
-                    last element of the "sparsearray," i.e., the number of
-                    elements it needed to describe the file.  */
+                    sparse file in question.  SPARSES is the index of the
+                    first unused element of the "sparsearray," i.e.,
+                    the number of elements it needed to describe the file.  */
 
-                 upperbound = deal_with_sparse (p, header);
+                 sparses = deal_with_sparse (p, header);
 
                  /* See if we'll need an extended header later.  */
 
-                 if (upperbound > SPARSES_IN_OLDGNU_HEADER - 1)
+                 if (SPARSES_IN_OLDGNU_HEADER < sparses)
                    header->oldgnu_header.isextended = 1;
 
                  /* We store the "real" file size so we can show that in
@@ -1322,50 +1344,47 @@ dump_file (char *p, int top_level, dev_t parent_device)
                     <file>.  It might be kind of disconcerting if the
                     shrunken file size was the one that showed up.  */
 
-                 OFF_TO_CHARS (current_stat.st_size,
+                 OFF_TO_CHARS (current_stat_info.stat.st_size,
                                header->oldgnu_header.realsize);
 
                  /* This will be the new "size" of the file, i.e., the size
                     of the file minus the blocks of holes that we're
                     skipping over.  */
 
-                 find_new_file_size (&filesize, upperbound);
-                 current_stat.st_size = filesize;
-                 OFF_TO_CHARS (filesize, header->header.size);
+                 current_stat_info.stat.st_size = find_new_file_size (sparses);
+                 OFF_TO_CHARS (current_stat_info.stat.st_size, header->header.size);
 
-                 for (counter = 0; counter < SPARSES_IN_OLDGNU_HEADER; counter++)
+                 for (counter = 0;
+                      counter < sparses && counter < SPARSES_IN_OLDGNU_HEADER;
+                      counter++)
                    {
-                     if (!sparsearray[counter].numbytes)
-                       break;
-
                      OFF_TO_CHARS (sparsearray[counter].offset,
                                    header->oldgnu_header.sp[counter].offset);
                      SIZE_TO_CHARS (sparsearray[counter].numbytes,
                                     header->oldgnu_header.sp[counter].numbytes);
                    }
-
                }
            }
-         else
-           upperbound = SPARSES_IN_OLDGNU_HEADER - 1;
 
-         sizeleft = current_stat.st_size;
+         sizeleft = current_stat_info.stat.st_size;
 
          /* Don't bother opening empty, world readable files.  Also do not open
             files when archive is meant for /dev/null.  */
 
          if (dev_null_output
              || (sizeleft == 0
-                 && MODE_R == (MODE_R & current_stat.st_mode)))
+                 && MODE_R == (MODE_R & current_stat_info.stat.st_mode)))
            f = -1;
          else
            {
              f = open (p, O_RDONLY | O_BINARY);
              if (f < 0)
                {
-                 WARN ((0, errno, _("Cannot add file %s"), p));
-                 if (!ignore_failed_read_option)
-                   exit_status = TAREXIT_FAILURE;
+                 if (! top_level && errno == ENOENT)
+                   WARN ((0, 0, _("%s: File removed before we read it"),
+                          quotearg_colon (p)));
+                 else
+                   (ignore_failed_read_option ? open_warn : open_error) (p);
                  return;
                }
            }
@@ -1373,47 +1392,50 @@ dump_file (char *p, int top_level, dev_t parent_device)
          /* If the file is sparse, we've already taken care of this.  */
 
          if (!header_moved)
-           header = start_header (p, &current_stat);
+           {
+             block_ordinal = current_block_ordinal ();
+             header = start_header (p, &current_stat_info);
+           }
 
          /* Mark contiguous files, if we support them.  */
 
-         if (archive_format != V7_FORMAT && S_ISCTG (current_stat.st_mode))
+         if (archive_format != V7_FORMAT && S_ISCTG (current_stat_info.stat.st_mode))
            header->header.typeflag = CONTTYPE;
 
          isextended = header->oldgnu_header.isextended;
          save_typeflag = header->header.typeflag;
-         finish_header (header);
+         finish_header (header, block_ordinal);
          if (isextended)
            {
-             int counter;
-             /* static */ int index_offset = SPARSES_IN_OLDGNU_HEADER;
+             int sparses_emitted = SPARSES_IN_OLDGNU_HEADER;
 
-           extend:
-             exhdr = find_next_block ();
-             memset (exhdr->buffer, 0, BLOCKSIZE);
-             for (counter = 0; counter < SPARSES_IN_SPARSE_HEADER; counter++)
+             for (;;)
                {
-                 if (counter + index_offset > upperbound)
+                 int i;
+                 exhdr = find_next_block ();
+                 memset (exhdr->buffer, 0, BLOCKSIZE);
+                 for (i = 0;
+                      (i < SPARSES_IN_SPARSE_HEADER
+                       && sparses_emitted + i < sparses);
+                      i++)
+                   {
+                     SIZE_TO_CHARS (sparsearray[sparses_emitted + i].numbytes,
+                                    exhdr->sparse_header.sp[i].numbytes);
+                     OFF_TO_CHARS (sparsearray[sparses_emitted + i].offset,
+                                   exhdr->sparse_header.sp[i].offset);
+                   }
+                 set_next_block_after (exhdr);
+                 sparses_emitted += i;
+                 if (sparses == sparses_emitted)
                    break;
-
-                 SIZE_TO_CHARS (sparsearray[counter + index_offset].numbytes,
-                                exhdr->sparse_header.sp[counter].numbytes);
-                 OFF_TO_CHARS (sparsearray[counter + index_offset].offset,
-                               exhdr->sparse_header.sp[counter].offset);
-               }
-             set_next_block_after (exhdr);
-             if (index_offset + counter <= upperbound)
-               {
-                 index_offset += counter;
                  exhdr->sparse_header.isextended = 1;
-                 goto extend;
                }
-
            }
          if (save_typeflag == GNUTYPE_SPARSE)
            {
              if (f < 0
-                 || finish_sparse_file (f, &sizeleft, current_stat.st_size, p))
+                 || finish_sparse_file (f, &sizeleft,
+                                        current_stat_info.stat.st_size, p))
                goto padit;
            }
          else
@@ -1423,7 +1445,7 @@ dump_file (char *p, int top_level, dev_t parent_device)
                  {
                    assign_string (&save_name, p);
                    save_sizeleft = sizeleft;
-                   save_totsize = current_stat.st_size;
+                   save_totsize = current_stat_info.stat.st_size;
                  }
                start = find_next_block ();
 
@@ -1444,28 +1466,31 @@ dump_file (char *p, int top_level, dev_t parent_device)
                  count = safe_read (f, start->buffer, bufsize);
                if (count < 0)
                  {
-                   char buf[UINTMAX_STRSIZE_BOUND];
-                   ERROR ((0, errno,
-                           _("Read error at byte %s, reading %lu bytes, in file %s"),
-                           STRINGIFY_BIGINT (current_stat.st_size - sizeleft,
-                                             buf),
-                           (unsigned long) bufsize, p));
+                   (ignore_failed_read_option
+                    ? read_warn_details
+                    : read_error_details)
+                     (p, current_stat_info.stat.st_size - sizeleft, bufsize);
                    goto padit;
                  }
                sizeleft -= count;
 
                /* This is nonportable (the type of set_next_block_after's arg).  */
 
-               set_next_block_after (start + (count - 1) / BLOCKSIZE);
+               set_next_block_after (start + (bufsize - 1) / BLOCKSIZE);
 
-               if (count == bufsize)
-                 continue;
-               else
+
+               if (count != bufsize)
                  {
                    char buf[UINTMAX_STRSIZE_BOUND];
-                   ERROR ((0, 0,
-                           _("File %s shrunk by %s bytes, padding with zeros"),
-                           p, STRINGIFY_BIGINT (sizeleft, buf)));
+                   memset (start->buffer + count, 0, bufsize - count);
+                   WARN ((0, 0,
+                          ngettext ("%s: File shrank by %s byte; padding with zeros",
+                                    "%s: File shrank by %s bytes; padding with zeros",
+                                    sizeleft),
+                          quotearg_colon (p),
+                          STRINGIFY_BIGINT (sizeleft, buf)));
+                   if (! ignore_failed_read_option)
+                     exit_status = TAREXIT_FAILURE;
                    goto padit;         /* short read */
                  }
              }
@@ -1477,21 +1502,33 @@ dump_file (char *p, int top_level, dev_t parent_device)
            {
              struct stat final_stat;
              if (fstat (f, &final_stat) != 0)
-               ERROR ((0, errno, "%s: fstat", p));
-             else if (final_stat.st_mtime != restore_times.modtime
-                      || final_stat.st_size != restore_size)
-               ERROR ((0, errno, _("%s: file changed as we read it"), p));
+               {
+                 if (ignore_failed_read_option)
+                   stat_warn (p);
+                 else
+                   stat_error (p);
+               }
+             else if (final_stat.st_ctime != original_ctime)
+               {
+                 char const *qp = quotearg_colon (p);
+                 WARN ((0, 0, _("%s: file changed as we read it"), qp));
+               }
              if (close (f) != 0)
-               ERROR ((0, errno, _("%s: close"), p));
+               {
+                 if (ignore_failed_read_option)
+                   close_warn (p);
+                 else
+                   close_error (p);
+               }
              if (atime_preserve_option)
                utime (p, &restore_times);
            }
          if (remove_files_option)
            {
              if (unlink (p) == -1)
-               ERROR ((0, errno, _("Cannot remove %s"), p));
+               unlink_error (p);
            }
-         return;
+         goto file_was_dumped;
 
          /* File shrunk or gave error, pad out tape to match the size we
             specified in the header.  */
@@ -1513,55 +1550,64 @@ dump_file (char *p, int top_level, dev_t parent_device)
              if (atime_preserve_option)
                utime (p, &restore_times);
            }
-         return;
+         goto file_was_dumped;
        }
 #ifdef HAVE_READLINK
-      else if (S_ISLNK (current_stat.st_mode))
+      else if (S_ISLNK (current_stat_info.stat.st_mode))
        {
+         char *buffer;
          int size;
-         char *buffer = (char *) alloca (PATH_MAX + 1);
-
-         size = readlink (p, buffer, PATH_MAX + 1);
+         size_t linklen = current_stat_info.stat.st_size;
+         if (linklen != current_stat_info.stat.st_size || linklen + 1 == 0)
+           xalloc_die ();
+         buffer = (char *) alloca (linklen + 1);
+         size = readlink (p, buffer, linklen + 1);
          if (size < 0)
            {
-             WARN ((0, errno, _("Cannot add file %s"), p));
-             if (!ignore_failed_read_option)
-               exit_status = TAREXIT_FAILURE;
+             if (ignore_failed_read_option)
+               readlink_warn (p);
+             else
+               readlink_error (p);
              return;
            }
          buffer[size] = '\0';
+         assign_string (&current_stat_info.link_name, buffer);
          if (size >= NAME_FIELD_SIZE)
-           write_long (buffer, GNUTYPE_LONGLINK);
-         assign_string (&current_link_name, buffer);
+           write_long_link (&current_stat_info);
 
-         current_stat.st_size = 0;     /* force 0 size on symlink */
-         header = start_header (p, &current_stat);
+         block_ordinal = current_block_ordinal ();
+         current_stat_info.stat.st_size = 0;   /* force 0 size on symlink */
+         header = start_header (p, &current_stat_info);
          strncpy (header->header.linkname, buffer, NAME_FIELD_SIZE);
          header->header.linkname[NAME_FIELD_SIZE - 1] = '\0';
          header->header.typeflag = SYMTYPE;
-         finish_header (header);       /* nothing more to do to it */
+         finish_header (header, block_ordinal);
+         /* nothing more to do to it */
+
          if (remove_files_option)
            {
              if (unlink (p) == -1)
-               ERROR ((0, errno, _("Cannot remove %s"), p));
+               unlink_error (p);
            }
-         return;
+         goto file_was_dumped;
        }
 #endif
-      else if (S_ISCHR (current_stat.st_mode))
+      else if (S_ISCHR (current_stat_info.stat.st_mode))
        type = CHRTYPE;
-      else if (S_ISBLK (current_stat.st_mode))
+      else if (S_ISBLK (current_stat_info.stat.st_mode))
        type = BLKTYPE;
-      else if (S_ISFIFO (current_stat.st_mode)
-              || S_ISSOCK (current_stat.st_mode))
+      else if (S_ISFIFO (current_stat_info.stat.st_mode))
        type = FIFOTYPE;
-#ifdef S_ISDOOR
-      else if (S_ISDOOR (current_stat.st_mode))
+      else if (S_ISSOCK (current_stat_info.stat.st_mode))
        {
-         WARN ((0, 0, _("%s: door ignored"), p));
+         WARN ((0, 0, _("%s: socket ignored"), quotearg_colon (p)));
+         return;
+       }
+      else if (S_ISDOOR (current_stat_info.stat.st_mode))
+       {
+         WARN ((0, 0, _("%s: door ignored"), quotearg_colon (p)));
          return;
        }
-#endif
       else
        goto unknown;
     }
@@ -1569,24 +1615,72 @@ dump_file (char *p, int top_level, dev_t parent_device)
   if (archive_format == V7_FORMAT)
     goto unknown;
 
-  current_stat.st_size = 0;    /* force 0 size */
-  header = start_header (p, &current_stat);
+  block_ordinal = current_block_ordinal ();
+  current_stat_info.stat.st_size = 0;  /* force 0 size */
+  header = start_header (p, &current_stat_info);
   header->header.typeflag = type;
 
   if (type != FIFOTYPE)
     {
-      MAJOR_TO_CHARS (major (current_stat.st_rdev), header->header.devmajor);
-      MINOR_TO_CHARS (minor (current_stat.st_rdev), header->header.devminor);
+      MAJOR_TO_CHARS (major (current_stat_info.stat.st_rdev), header->header.devmajor);
+      MINOR_TO_CHARS (minor (current_stat_info.stat.st_rdev), header->header.devminor);
     }
 
-  finish_header (header);
+  finish_header (header, block_ordinal);
   if (remove_files_option)
     {
       if (unlink (p) == -1)
-       ERROR ((0, errno, _("Cannot remove %s"), p));
+       unlink_error (p);
     }
-  return;
+  goto file_was_dumped;
 
 unknown:
-  ERROR ((0, 0, _("%s: Unknown file type; file ignored"), p));
+  WARN ((0, 0, _("%s: Unknown file type; file ignored"),
+        quotearg_colon (p)));
+  if (! ignore_failed_read_option)
+    exit_status = TAREXIT_FAILURE;
+  return;
+
+file_was_dumped:
+  if (1 < current_stat_info.stat.st_nlink)
+    {
+      struct link *dup;
+      struct link *lp = xmalloc (offsetof (struct link, name)
+                                + strlen (p) + 1);
+      lp->ino = current_stat_info.stat.st_ino;
+      lp->dev = current_stat_info.stat.st_dev;
+      lp->nlink = current_stat_info.stat.st_nlink;
+      strcpy (lp->name, p);
+
+      if (! ((link_table
+             || (link_table = hash_initialize (0, 0, hash_link,
+                                               compare_links, 0)))
+            && (dup = hash_insert (link_table, lp))))
+       xalloc_die ();
+
+      if (dup != lp)
+       abort ();
+      lp->nlink--;
+    }
+
+}
+
+/* For each dumped file, check if all its links were dumped. Emit
+   warnings if it is not so. */
+void
+check_links ()
+{
+  struct link *lp;
+
+  if (!link_table)
+    return;
+
+  for (lp = hash_get_first (link_table); lp;
+       lp = hash_get_next (link_table, lp))
+    {
+      if (lp->nlink)
+       {
+         WARN ((0, 0, _("Missing links to '%s'.\n"), lp->name));
+       }
+    }
 }
This page took 0.059975 seconds and 4 git commands to generate.