diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2022-09-03 18:22:34 -0500 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2022-09-03 18:23:11 -0500 |
commit | bc277c7069cdebb6a6140326636555267b142e0e (patch) | |
tree | 73707e333fa7d181ad86f1fa2d97fac6e1031811 /src/extract.c | |
parent | f8e14746d2ca72804a4520c059d7bf65ca00c5ac (diff) | |
download | tar-bc277c7069cdebb6a6140326636555267b142e0e.tar.gz |
Fix data loss when acting as filter
This bug was introduced by the recent lseek-related changes.
* src/delete.c (delete_archive_members):
* src/update.c (update_archive):
Copy the member if acting as a filter, rather than lseeking over
it, which is possible if stdin is a regular file.
* src/list.c (skim_file, skim_member):
* src/sparse.c (sparse_skim_file):
New functions, for copying when a filter.
* src/list.c (skip_file): Remove; replaced with skim_file.
All callers changed.
(skip_member): Reimplement in terms of skim_member.
* src/sparse.c (sparse_skip_file):
Remove; replaced with sparse_skim_file. All callers changed.
* src/update.c (acting_as_filter): New static var.
(update_archive): Set it; this is like delete.c.
* tests/delete01.at (deleting a member after a big one):
* tests/delete02.at (deleting a member from stdin archive):
Also test filter case.
Diffstat (limited to 'src/extract.c')
-rw-r--r-- | src/extract.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/extract.c b/src/extract.c index 7696d5e4..78de47f5 100644 --- a/src/extract.c +++ b/src/extract.c @@ -1320,7 +1320,7 @@ extract_file (char *file_name, int typeflag) } } - skip_file (size); + skim_file (size, false); mv_end (); |