6b8c4d7
diff --git a/THANKS b/THANKS
6b8c4d7
index a734fb6..2da6100 100644
6b8c4d7
--- a/THANKS
6b8c4d7
+++ b/THANKS
6b8c4d7
@@ -173,6 +173,7 @@ Fabio d'Alessi		cars@civ.bio.unipd.it
6b8c4d7
 Frank Heckenbach	frank@g-n-u.de
6b8c4d7
 Frank Koenen		koenfr@lidp.com
6b8c4d7
 Franz-Werner Gergen	gergen@edvulx.mpi-stuttgart.mpg.de
6b8c4d7
+Fran├žois Ouellet	fouell@gmail.com
6b8c4d7
 Fran├žois Pinard		pinard@iro.umontreal.ca
6b8c4d7
 Fritz Elfert		fritz@fsun.triltsch.de
6b8c4d7
 George Chyu		gschyu@ccgate.dp.beckman.com
6b8c4d7
diff --git a/src/sparse.c b/src/sparse.c
6b8c4d7
index f0268f4..250e845 100644
6b8c4d7
--- a/src/sparse.c
6b8c4d7
+++ b/src/sparse.c
6b8c4d7
@@ -301,6 +301,7 @@ sparse_dump_region (struct tar_sparse_file *file, size_t i)
6b8c4d7
 {
6b8c4d7
   union block *blk;
6b8c4d7
   off_t bytes_left = file->stat_info->sparse_map[i].numbytes;
6b8c4d7
+  const char *file_name = file->stat_info->orig_file_name;
6b8c4d7
 
6b8c4d7
   if (!lseek_or_error (file, file->stat_info->sparse_map[i].offset))
6b8c4d7
     return false;
6b8c4d7
@@ -314,13 +315,23 @@ sparse_dump_region (struct tar_sparse_file *file, size_t i)
6b8c4d7
       bytes_read = safe_read (file->fd, blk->buffer, bufsize);
6b8c4d7
       if (bytes_read == SAFE_READ_ERROR)
6b8c4d7
 	{
6b8c4d7
-          read_diag_details (file->stat_info->orig_file_name,
6b8c4d7
+          read_diag_details (file_name,
6b8c4d7
 	                     (file->stat_info->sparse_map[i].offset
6b8c4d7
 			      + file->stat_info->sparse_map[i].numbytes
6b8c4d7
 			      - bytes_left),
6b8c4d7
 			     bufsize);
6b8c4d7
 	  return false;
6b8c4d7
 	}
6b8c4d7
+      if (bytes_read == 0)
6b8c4d7
+        {
6b8c4d7
+          char buf[UINTMAX_STRSIZE_BOUND];
6b8c4d7
+          FATAL_ERROR ((0, 0,
6b8c4d7
+                        ngettext ("%s: File shrank by %s byte",
6b8c4d7
+                                  "%s: File shrank by %s bytes",
6b8c4d7
+                                  bytes_left),
6b8c4d7
+                        quotearg_colon (file_name),
6b8c4d7
+                        offtostr (bytes_left, buf)));
6b8c4d7
+        }
6b8c4d7
 
6b8c4d7
       memset (blk->buffer + bytes_read, 0, BLOCKSIZE - bytes_read);
6b8c4d7
       bytes_left -= bytes_read;
6b8c4d7
@@ -475,33 +486,37 @@ sparse_skip_file (struct tar_stat_info *st)
6b8c4d7
 static bool
6b8c4d7
 check_sparse_region (struct tar_sparse_file *file, off_t beg, off_t end)
6b8c4d7
 {
6b8c4d7
-  if (!lseek_or_error (file, beg))
6b8c4d7
+  off_t offset = beg;
6b8c4d7
+
6b8c4d7
+  if (!lseek_or_error (file, offset))
6b8c4d7
     return false;
6b8c4d7
 
6b8c4d7
-  while (beg < end)
6b8c4d7
+  while (offset < end)
6b8c4d7
     {
6b8c4d7
       size_t bytes_read;
6b8c4d7
-      size_t rdsize = BLOCKSIZE < end - beg ? BLOCKSIZE : end - beg;
6b8c4d7
+      size_t rdsize = BLOCKSIZE < end - offset ? BLOCKSIZE : end - offset;
6b8c4d7
       char diff_buffer[BLOCKSIZE];
6b8c4d7
 
6b8c4d7
       bytes_read = safe_read (file->fd, diff_buffer, rdsize);
6b8c4d7
       if (bytes_read == SAFE_READ_ERROR)
6b8c4d7
 	{
6b8c4d7
           read_diag_details (file->stat_info->orig_file_name,
6b8c4d7
-	                     beg,
6b8c4d7
-			     rdsize);
6b8c4d7
-	  return false;
6b8c4d7
-	}
6b8c4d7
-      if (!zero_block_p (diff_buffer, bytes_read))
6b8c4d7
-	{
6b8c4d7
-	  char begbuf[INT_BUFSIZE_BOUND (off_t)];
6b8c4d7
- 	  report_difference (file->stat_info,
6b8c4d7
-			     _("File fragment at %s is not a hole"),
6b8c4d7
-			     offtostr (beg, begbuf));
6b8c4d7
+	                     offset, rdsize);
6b8c4d7
 	  return false;
6b8c4d7
 	}
6b8c4d7
 
6b8c4d7
-      beg += bytes_read;
6b8c4d7
+      if (bytes_read == 0
6b8c4d7
+          || !zero_block_p (diff_buffer, bytes_read))
6b8c4d7
+        {
6b8c4d7
+          char begbuf[INT_BUFSIZE_BOUND (off_t)];
6b8c4d7
+          const char *msg = bytes_read ? _("File fragment at %s is not a hole")
6b8c4d7
+                                       : _("Hole starting at %s is truncated");
6b8c4d7
+
6b8c4d7
+          report_difference (file->stat_info, msg, offtostr (beg, begbuf));
6b8c4d7
+          return false;
6b8c4d7
+        }
6b8c4d7
+
6b8c4d7
+      offset += bytes_read;
6b8c4d7
     }
6b8c4d7
   return true;
6b8c4d7
 }
6b8c4d7
@@ -542,7 +557,8 @@ check_data_region (struct tar_sparse_file *file, size_t i)
6b8c4d7
       file->dumped_size += bytes_read;
6b8c4d7
       size_left -= bytes_read;
6b8c4d7
       mv_size_left (file->stat_info->archive_file_size - file->dumped_size);
6b8c4d7
-      if (memcmp (blk->buffer, diff_buffer, rdsize))
6b8c4d7
+      if (bytes_read == 0
6b8c4d7
+          || memcmp (blk->buffer, diff_buffer, bytes_read))
6b8c4d7
 	{
6b8c4d7
 	  report_difference (file->stat_info, _("Contents differ"));
6b8c4d7
 	  return false;