]> Dogcows Code - chaz/tar/blobdiff - src/compare.c
tar: port -d to longer symlinks
[chaz/tar] / src / compare.c
index b790a7a089e8a9d874f7a99939b5440cc6557cb9..7ed455848791cd40433dd52aea4b17155e970b3b 100644 (file)
@@ -1,44 +1,35 @@
 /* Diff files from a tar archive.
 
-   Copyright (C) 1988, 1992, 1993, 1994, 1996, 1997, 1999, 2000, 2001,
-   2003, 2004 Free Software Foundation, Inc.
+   Copyright 1988, 1992-1994, 1996-1997, 1999-2001, 2003-2007,
+   2009-2010, 2012-2014 Free Software Foundation, Inc.
 
-   Written by John Gilmore, on 1987-04-30.
+   This file is part of GNU tar.
 
-   This program is free software; you can redistribute it and/or modify it
-   under the terms of the GNU General Public License as published by the
-   Free Software Foundation; either version 2, or (at your option) any later
-   version.
+   GNU tar is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
 
-   This program is distributed in the hope that it will be useful, but
-   WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General
-   Public License for more details.
+   GNU tar is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-   You should have received a copy of the GNU General Public License along
-   with this program; if not, write to the Free Software Foundation, Inc.,
-   59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-#include "system.h"
+   Written by John Gilmore, on 1987-04-30.  */
 
-#if HAVE_UTIME_H
-# include <utime.h>
-#else
-struct utimbuf
-  {
-    long actime;
-    long modtime;
-  };
-#endif
+#include <system.h>
+#include <system-ioctl.h>
 
 #if HAVE_LINUX_FD_H
 # include <linux/fd.h>
 #endif
 
-#include <quotearg.h>
-
 #include "common.h"
-#include "rmt.h"
+#include <quotearg.h>
+#include <rmt.h>
 #include <stdarg.h>
 
 /* Nonzero if we are verifying at the moment.  */
@@ -56,27 +47,27 @@ diff_init (void)
 {
   void *ptr;
   diff_buffer = page_aligned_alloc (&ptr, record_size);
+  if (listed_incremental_option)
+    read_directory_file ();
 }
 
 /* Sigh about something that differs by writing a MESSAGE to stdlis,
    given MESSAGE is nonzero.  Also set the exit status if not already.  */
 void
-report_difference (struct tar_stat_info *st __attribute__ ((unused)),
-                  const char *fmt, ...)
+report_difference (struct tar_stat_info *st, const char *fmt, ...)
 {
   if (fmt)
     {
       va_list ap;
 
-      fprintf (stdlis, "%s: ", quotearg_colon (current_stat_info.file_name));
+      fprintf (stdlis, "%s: ", quotearg_colon (st->file_name));
       va_start (ap, fmt);
       vfprintf (stdlis, fmt, ap);
       va_end (ap);
       fprintf (stdlis, "\n");
     }
 
-  if (exit_status == TAREXIT_SUCCESS)
-    exit_status = TAREXIT_DIFFERS;
+  set_exit_status (TAREXIT_DIFFERS);
 }
 
 /* Take a buffer returned by read_and_process and do nothing with it.  */
@@ -90,7 +81,7 @@ process_noop (size_t size __attribute__ ((unused)),
 static int
 process_rawdata (size_t bytes, char *buffer)
 {
-  size_t status = safe_read (diff_handle, diff_buffer, bytes);
+  size_t status = blocking_read (diff_handle, diff_buffer, bytes);
 
   if (status != bytes)
     {
@@ -111,45 +102,28 @@ process_rawdata (size_t bytes, char *buffer)
     }
 
   if (memcmp (buffer, diff_buffer, bytes))
-    {
-      report_difference (&current_stat_info,
-                        _("Contents differ"));
-      return 0;
-    }
-
-  return 1;
-}
-
-/* Directory contents, only for GNUTYPE_DUMPDIR.  */
-
-static char *dumpdir_cursor;
-
-static int
-process_dumpdir (size_t bytes, char *buffer)
-{
-  if (memcmp (buffer, dumpdir_cursor, bytes))
     {
       report_difference (&current_stat_info, _("Contents differ"));
       return 0;
     }
 
-  dumpdir_cursor += bytes;
   return 1;
 }
 
 /* Some other routine wants SIZE bytes in the archive.  For each chunk
    of the archive, call PROCESSOR with the size of the chunk, and the
    address of the chunk it can work with.  The PROCESSOR should return
-   nonzero for success.  It it return error once, continue skipping
+   nonzero for success.  Once it returns error, continue skipping
    without calling PROCESSOR anymore.  */
+
 static void
-read_and_process (off_t size, int (*processor) (size_t, char *))
+read_and_process (struct tar_stat_info *st, int (*processor) (size_t, char *))
 {
   union block *data_block;
   size_t data_size;
+  off_t size = st->stat.st_size;
 
-  if (multi_volume_option)
-    save_sizeleft = size;
+  mv_begin_read (st);
   while (size)
     {
       data_block = find_next_block ();
@@ -167,9 +141,9 @@ read_and_process (off_t size, int (*processor) (size_t, char *))
       set_next_block_after ((union block *)
                            (data_block->buffer + data_size - 1));
       size -= data_size;
-      if (multi_volume_option)
-       save_sizeleft -= data_size;
+      mv_size_left (size);
     }
+  mv_end ();
 }
 
 /* Call either stat or lstat over STAT_DATA, depending on
@@ -178,7 +152,7 @@ read_and_process (off_t size, int (*processor) (size_t, char *))
 static int
 get_stat_data (char const *file_name, struct stat *stat_data)
 {
-  int status = deref_stat (dereference_option, file_name, stat_data);
+  int status = deref_stat (file_name, stat_data);
 
   if (status != 0)
     {
@@ -193,57 +167,37 @@ get_stat_data (char const *file_name, struct stat *stat_data)
   return 1;
 }
 
-/* Diff a file against the archive.  */
-void
-diff_archive (void)
+\f
+static void
+diff_dir (void)
 {
   struct stat stat_data;
-  int status;
-  struct utimbuf restore_times;
 
-  set_next_block_after (current_header);
-  decode_header (current_header, &current_stat_info, &current_format, 1);
+  if (!get_stat_data (current_stat_info.file_name, &stat_data))
+    return;
 
-  /* Print the block from current_header and current_stat_info.  */
+  if (!S_ISDIR (stat_data.st_mode))
+    report_difference (&current_stat_info, _("File type differs"));
+  else if ((current_stat_info.stat.st_mode & MODE_ALL) !=
+          (stat_data.st_mode & MODE_ALL))
+    report_difference (&current_stat_info, _("Mode differs"));
+}
 
-  if (verbose_option)
+static void
+diff_file (void)
+{
+  char const *file_name = current_stat_info.file_name;
+  struct stat stat_data;
+
+  if (!get_stat_data (file_name, &stat_data))
+    skip_member ();
+  else if (!S_ISREG (stat_data.st_mode))
     {
-      if (now_verifying)
-       fprintf (stdlis, _("Verify "));
-      print_header (&current_stat_info, -1);
+      report_difference (&current_stat_info, _("File type differs"));
+      skip_member ();
     }
-
-  switch (current_header->header.typeflag)
+  else
     {
-    default:
-      ERROR ((0, 0, _("%s: Unknown file type '%c', diffed as normal file"),
-             quotearg_colon (current_stat_info.file_name),
-             current_header->header.typeflag));
-      /* Fall through.  */
-
-    case AREGTYPE:
-    case REGTYPE:
-    case GNUTYPE_SPARSE:
-    case CONTTYPE:
-
-      /* Appears to be a file.  See if it's really a directory.  */
-
-      if (current_stat_info.had_trailing_slash)
-       goto really_dir;
-
-      if (!get_stat_data (current_stat_info.file_name, &stat_data))
-       {
-         skip_member ();
-         goto quit;
-       }
-
-      if (!S_ISREG (stat_data.st_mode))
-       {
-         report_difference (&current_stat_info, _("File type differs"));
-         skip_member ();
-         goto quit;
-       }
-
       if ((current_stat_info.stat.st_mode & MODE_ALL) !=
          (stat_data.st_mode & MODE_ALL))
        report_difference (&current_stat_info, _("Mode differs"));
@@ -253,254 +207,343 @@ diff_archive (void)
       if (!sys_compare_gid (&stat_data, &current_stat_info.stat))
        report_difference (&current_stat_info, _("Gid differs"));
 
-      if (stat_data.st_mtime != current_stat_info.stat.st_mtime)
+      if (tar_timespec_cmp (get_stat_mtime (&stat_data),
+                            current_stat_info.mtime))
        report_difference (&current_stat_info, _("Mod time differs"));
-      if (current_header->header.typeflag != GNUTYPE_SPARSE &&
-         stat_data.st_size != current_stat_info.stat.st_size)
+      if (current_header->header.typeflag != GNUTYPE_SPARSE
+         && stat_data.st_size != current_stat_info.stat.st_size)
        {
          report_difference (&current_stat_info, _("Size differs"));
          skip_member ();
-         goto quit;
        }
-
-      diff_handle = open (current_stat_info.file_name, O_RDONLY | O_BINARY);
-
-      if (diff_handle < 0)
+      else
        {
-         open_error (current_stat_info.file_name);
-         skip_member ();
-         report_difference (&current_stat_info, NULL);
-         goto quit;
+         diff_handle = openat (chdir_fd, file_name, open_read_flags);
+
+         if (diff_handle < 0)
+           {
+             open_error (file_name);
+             skip_member ();
+             report_difference (&current_stat_info, NULL);
+           }
+         else
+           {
+             int status;
+
+             if (current_stat_info.is_sparse)
+               sparse_diff_file (diff_handle, &current_stat_info);
+             else
+               read_and_process (&current_stat_info, process_rawdata);
+
+             if (atime_preserve_option == replace_atime_preserve
+                 && stat_data.st_size != 0)
+               {
+                 struct timespec atime = get_stat_atime (&stat_data);
+                 if (set_file_atime (diff_handle, chdir_fd, file_name, atime)
+                     != 0)
+                   utime_error (file_name);
+               }
+
+             status = close (diff_handle);
+             if (status != 0)
+               close_error (file_name);
+           }
        }
+    }
+}
+
+static void
+diff_link (void)
+{
+  struct stat file_data;
+  struct stat link_data;
+
+  if (get_stat_data (current_stat_info.file_name, &file_data)
+      && get_stat_data (current_stat_info.link_name, &link_data)
+      && !sys_compare_links (&file_data, &link_data))
+    report_difference (&current_stat_info,
+                      _("Not linked to %s"),
+                      quote (current_stat_info.link_name));
+}
 
-      restore_times.actime = stat_data.st_atime;
-      restore_times.modtime = stat_data.st_mtime;
+#ifdef HAVE_READLINK
+static void
+diff_symlink (void)
+{
+  char buf[1024];
+  size_t len = strlen (current_stat_info.link_name);
+  char *linkbuf = len < sizeof buf ? buf : xmalloc (len + 1);
 
-      /* Need to treat sparse files completely differently here.  */
+  ssize_t status = readlinkat (chdir_fd, current_stat_info.file_name,
+                              linkbuf, len + 1);
 
-      if (current_stat_info.is_sparse)
-       sparse_diff_file (diff_handle, &current_stat_info);
+  if (status < 0)
+    {
+      if (errno == ENOENT)
+       readlink_warn (current_stat_info.file_name);
       else
-       {
-         if (multi_volume_option)
-           {
-             assign_string (&save_name, current_stat_info.file_name);
-             save_totsize = current_stat_info.stat.st_size;
-             /* save_sizeleft is set in read_and_process.  */
-           }
-
-         read_and_process (current_stat_info.stat.st_size, process_rawdata);
+       readlink_error (current_stat_info.file_name);
+      report_difference (&current_stat_info, NULL);
+    }
+  else if (status != len
+          || memcmp (current_stat_info.link_name, linkbuf, len) != 0)
+    report_difference (&current_stat_info, _("Symlink differs"));
 
-         if (multi_volume_option)
-           assign_string (&save_name, 0);
-       }
+  if (linkbuf != buf)
+    free (linkbuf);
+}
+#endif
 
-      status = close (diff_handle);
-      if (status != 0)
-       close_error (current_stat_info.file_name);
+static void
+diff_special (void)
+{
+  struct stat stat_data;
 
-      if (atime_preserve_option)
-       utime (current_stat_info.file_name, &restore_times);
+  /* FIXME: deal with umask.  */
 
-    quit:
-      break;
+  if (!get_stat_data (current_stat_info.file_name, &stat_data))
+    return;
 
-    case LNKTYPE:
-      {
-       struct stat file_data;
-       struct stat link_data;
-
-       if (!get_stat_data (current_stat_info.file_name, &file_data))
-         break;
-       if (!get_stat_data (current_stat_info.link_name, &link_data))
-         break;
-       if (!sys_compare_links (&file_data, &link_data))
-         report_difference (&current_stat_info,
-                            _("Not linked to %s"),
-                            quote (current_stat_info.link_name));
-      }
-      break;
+  if (current_header->header.typeflag == CHRTYPE
+      ? !S_ISCHR (stat_data.st_mode)
+      : current_header->header.typeflag == BLKTYPE
+      ? !S_ISBLK (stat_data.st_mode)
+      : /* current_header->header.typeflag == FIFOTYPE */
+      !S_ISFIFO (stat_data.st_mode))
+    {
+      report_difference (&current_stat_info, _("File type differs"));
+      return;
+    }
 
-#ifdef HAVE_READLINK
-    case SYMTYPE:
-      {
-       size_t len = strlen (current_stat_info.link_name);
-       char *linkbuf = alloca (len + 1);
+  if ((current_header->header.typeflag == CHRTYPE
+       || current_header->header.typeflag == BLKTYPE)
+      && current_stat_info.stat.st_rdev != stat_data.st_rdev)
+    {
+      report_difference (&current_stat_info, _("Device number differs"));
+      return;
+    }
 
-       status = readlink (current_stat_info.file_name, linkbuf, len + 1);
+  if ((current_stat_info.stat.st_mode & MODE_ALL) !=
+      (stat_data.st_mode & MODE_ALL))
+    report_difference (&current_stat_info, _("Mode differs"));
+}
 
-       if (status < 0)
-         {
-           if (errno == ENOENT)
-             readlink_warn (current_stat_info.file_name);
-           else
-             readlink_error (current_stat_info.file_name);
-           report_difference (&current_stat_info, NULL);
-         }
-       else if (status != len
-                || strncmp (current_stat_info.link_name, linkbuf, len) != 0)
-         report_difference (&current_stat_info, _("Symlink differs"));
+static int
+dumpdir_cmp (const char *a, const char *b)
+{
+  size_t len;
 
+  while (*a)
+    switch (*a)
+      {
+      case 'Y':
+      case 'N':
+       if (!strchr ("YN", *b))
+         return 1;
+       if (strcmp(a + 1, b + 1))
+         return 1;
+       len = strlen (a) + 1;
+       a += len;
+       b += len;
        break;
-      }
-#endif
 
-    case CHRTYPE:
-    case BLKTYPE:
-    case FIFOTYPE:
+      case 'D':
+       if (strcmp(a, b))
+         return 1;
+       len = strlen (a) + 1;
+       a += len;
+       b += len;
+       break;
 
-      /* FIXME: deal with umask.  */
+      case 'R':
+      case 'T':
+      case 'X':
+       return *b;
+      }
+  return *b;
+}
 
-      if (!get_stat_data (current_stat_info.file_name, &stat_data))
-       break;
+static void
+diff_dumpdir (struct tar_stat_info *dir)
+{
+  const char *dumpdir_buffer;
 
-      if (current_header->header.typeflag == CHRTYPE
-         ? !S_ISCHR (stat_data.st_mode)
-         : current_header->header.typeflag == BLKTYPE
-         ? !S_ISBLK (stat_data.st_mode)
-         : /* current_header->header.typeflag == FIFOTYPE */
-         !S_ISFIFO (stat_data.st_mode))
+  if (dir->fd == 0)
+    {
+      void (*diag) (char const *) = NULL;
+      int fd = subfile_open (dir->parent, dir->orig_file_name, open_read_flags);
+      if (fd < 0)
+       diag = open_diag;
+      else if (fstat (fd, &dir->stat))
+        {
+         diag = stat_diag;
+          close (fd);
+        }
+      else
+       dir->fd = fd;
+      if (diag)
        {
-         report_difference (&current_stat_info, _("File type differs"));
-         break;
+         file_removed_diag (dir->orig_file_name, false, diag);
+         return;
        }
+    }
+  dumpdir_buffer = directory_contents (scan_directory (dir));
 
-      if ((current_header->header.typeflag == CHRTYPE
-          || current_header->header.typeflag == BLKTYPE)
-         && current_stat_info.stat.st_rdev != stat_data.st_rdev)
-       {
-         report_difference (&current_stat_info, _("Device number differs"));
-         break;
-       }
+  if (dumpdir_buffer)
+    {
+      if (dumpdir_cmp (dir->dumpdir, dumpdir_buffer))
+       report_difference (dir, _("Contents differ"));
+    }
+  else
+    read_and_process (dir, process_noop);
+}
 
-      if ((current_stat_info.stat.st_mode & MODE_ALL) != (stat_data.st_mode & MODE_ALL))
-       {
-         report_difference (&current_stat_info, _("Mode differs"));
-         break;
-       }
+static void
+diff_multivol (void)
+{
+  struct stat stat_data;
+  int fd, status;
+  off_t offset;
 
-      break;
+  if (current_stat_info.had_trailing_slash)
+    {
+      diff_dir ();
+      return;
+    }
 
-    case GNUTYPE_DUMPDIR:
-      {
-       char *dumpdir_buffer = get_directory_contents (current_stat_info.file_name, 0);
+  if (!get_stat_data (current_stat_info.file_name, &stat_data))
+    return;
 
-       if (multi_volume_option)
-         {
-           assign_string (&save_name, current_stat_info.file_name);
-           save_totsize = current_stat_info.stat.st_size;
-           /* save_sizeleft is set in read_and_process.  */
-         }
+  if (!S_ISREG (stat_data.st_mode))
+    {
+      report_difference (&current_stat_info, _("File type differs"));
+      skip_member ();
+      return;
+    }
 
-       if (dumpdir_buffer)
-         {
-           dumpdir_cursor = dumpdir_buffer;
-           read_and_process (current_stat_info.stat.st_size, process_dumpdir);
-           free (dumpdir_buffer);
-         }
-       else
-         read_and_process (current_stat_info.stat.st_size, process_noop);
+  offset = OFF_FROM_HEADER (current_header->oldgnu_header.offset);
+  if (offset < 0
+      || INT_ADD_OVERFLOW (current_stat_info.stat.st_size, offset)
+      || stat_data.st_size != current_stat_info.stat.st_size + offset)
+    {
+      report_difference (&current_stat_info, _("Size differs"));
+      skip_member ();
+      return;
+    }
 
-       if (multi_volume_option)
-         assign_string (&save_name, 0);
-       /* Fall through.  */
-      }
 
-    case DIRTYPE:
-    really_dir:
-      if (!get_stat_data (current_stat_info.file_name, &stat_data))
-       break;
+  fd = openat (chdir_fd, current_stat_info.file_name, open_read_flags);
 
-      if (!S_ISDIR (stat_data.st_mode))
-       {
-         report_difference (&current_stat_info, _("File type differs"));
-         break;
-       }
+  if (fd < 0)
+    {
+      open_error (current_stat_info.file_name);
+      report_difference (&current_stat_info, NULL);
+      skip_member ();
+      return;
+    }
 
-      if ((current_stat_info.stat.st_mode & MODE_ALL) != (stat_data.st_mode & MODE_ALL))
-       {
-         report_difference (&current_stat_info, _("Mode differs"));
-         break;
-       }
+  if (lseek (fd, offset, SEEK_SET) < 0)
+    {
+      seek_error_details (current_stat_info.file_name, offset);
+      report_difference (&current_stat_info, NULL);
+    }
+  else
+    read_and_process (&current_stat_info, process_rawdata);
 
-      break;
+  status = close (fd);
+  if (status != 0)
+    close_error (current_stat_info.file_name);
+}
 
-    case GNUTYPE_VOLHDR:
-      break;
+/* Diff a file against the archive.  */
+void
+diff_archive (void)
+{
 
-    case GNUTYPE_MULTIVOL:
-      {
-       off_t offset;
+  set_next_block_after (current_header);
 
-       if (current_stat_info.had_trailing_slash)
-         goto really_dir;
+  /* Print the block from current_header and current_stat_info.  */
 
-       if (!get_stat_data (current_stat_info.file_name, &stat_data))
-         break;
+  if (verbose_option)
+    {
+      if (now_verifying)
+       fprintf (stdlis, _("Verify "));
+      print_header (&current_stat_info, current_header, -1);
+    }
 
-       if (!S_ISREG (stat_data.st_mode))
-         {
-           report_difference (&current_stat_info, _("File type differs"));
-           skip_member ();
-           break;
-         }
+  switch (current_header->header.typeflag)
+    {
+    default:
+      ERROR ((0, 0, _("%s: Unknown file type '%c', diffed as normal file"),
+             quotearg_colon (current_stat_info.file_name),
+             current_header->header.typeflag));
+      /* Fall through.  */
 
-       offset = OFF_FROM_HEADER (current_header->oldgnu_header.offset);
-       if (stat_data.st_size != current_stat_info.stat.st_size + offset)
-         {
-           report_difference (&current_stat_info, _("Size differs"));
-           skip_member ();
-           break;
-         }
+    case AREGTYPE:
+    case REGTYPE:
+    case GNUTYPE_SPARSE:
+    case CONTTYPE:
 
-       diff_handle = open (current_stat_info.file_name, O_RDONLY | O_BINARY);
+      /* Appears to be a file.  See if it's really a directory.  */
 
-       if (diff_handle < 0)
-         {
-           open_error (current_stat_info.file_name);
-           report_difference (&current_stat_info, NULL);
-           skip_member ();
-           break;
-         }
+      if (current_stat_info.had_trailing_slash)
+       diff_dir ();
+      else
+       diff_file ();
+      break;
 
-       if (lseek (diff_handle, offset, SEEK_SET) < 0)
-         {
-           seek_error_details (current_stat_info.file_name, offset);
-           report_difference (&current_stat_info, NULL);
-           break;
-         }
+    case LNKTYPE:
+      diff_link ();
+      break;
 
-       if (multi_volume_option)
-         {
-           assign_string (&save_name, current_stat_info.file_name);
-           save_totsize = stat_data.st_size;
-           /* save_sizeleft is set in read_and_process.  */
-         }
+#ifdef HAVE_READLINK
+    case SYMTYPE:
+      diff_symlink ();
+      break;
+#endif
 
-       read_and_process (current_stat_info.stat.st_size, process_rawdata);
+    case CHRTYPE:
+    case BLKTYPE:
+    case FIFOTYPE:
+      diff_special ();
+      break;
 
-       if (multi_volume_option)
-         assign_string (&save_name, 0);
+    case GNUTYPE_DUMPDIR:
+    case DIRTYPE:
+      if (is_dumpdir (&current_stat_info))
+       diff_dumpdir (&current_stat_info);
+      diff_dir ();
+      break;
 
-       status = close (diff_handle);
-       if (status != 0)
-         close_error (current_stat_info.file_name);
+    case GNUTYPE_VOLHDR:
+      break;
 
-       break;
-      }
+    case GNUTYPE_MULTIVOL:
+      diff_multivol ();
     }
 }
 
 void
 verify_volume (void)
 {
+  int may_fail = 0;
   if (removed_prefixes_p ())
     {
       WARN((0, 0,
            _("Archive contains file names with leading prefixes removed.")));
+      may_fail = 1;
+    }
+  if (transform_program_p ())
+    {
       WARN((0, 0,
-           _("Verification may fail to locate original files.")));
+           _("Archive contains transformed file names.")));
+      may_fail = 1;
     }
+  if (may_fail)
+    WARN((0, 0,
+         _("Verification may fail to locate original files.")));
+
+  clear_directory_table ();
 
   if (!diff_buffer)
     diff_init ();
@@ -554,7 +597,9 @@ verify_volume (void)
   flush_read ();
   while (1)
     {
-      enum read_header status = read_header (false);
+      enum read_header status = read_header (&current_header,
+                                             &current_stat_info,
+                                             read_header_auto);
 
       if (status == HEADER_FAILURE)
        {
@@ -564,7 +609,8 @@ verify_volume (void)
            {
              counter++;
              set_next_block_after (current_header);
-             status = read_header (false);
+             status = read_header (&current_header, &current_stat_info,
+                                   read_header_auto);
            }
          while (status == HEADER_FAILURE);
 
@@ -573,12 +619,29 @@ verify_volume (void)
                            "VERIFY FAILURE: %d invalid headers detected",
                            counter), counter));
        }
-      if (status == HEADER_ZERO_BLOCK || status == HEADER_END_OF_FILE)
+      if (status == HEADER_END_OF_FILE)
        break;
+      if (status == HEADER_ZERO_BLOCK)
+       {
+         set_next_block_after (current_header);
+          if (!ignore_zeros_option)
+            {
+             char buf[UINTMAX_STRSIZE_BOUND];
+
+             status = read_header (&current_header, &current_stat_info,
+                                   read_header_auto);
+             if (status == HEADER_ZERO_BLOCK)
+               break;
+             WARNOPT (WARN_ALONE_ZERO_BLOCK,
+                      (0, 0, _("A lone zero block at %s"),
+                       STRINGIFY_BIGINT (current_block_ordinal (), buf)));
+            }
+         continue;
+       }
 
+      decode_header (current_header, &current_stat_info, &current_format, 1);
       diff_archive ();
       tar_stat_destroy (&current_stat_info);
-      xheader_destroy (&extended_header);
     }
 
   access_mode = ACCESS_WRITE;
This page took 0.036266 seconds and 4 git commands to generate.