|
|
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;
|