diff options
| author | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2023-02-08 14:08:34 +0100 |
|---|---|---|
| committer | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2023-02-08 14:08:34 +0100 |
| commit | 92e2c77a5b5eeabc3252ea90953ab6bd1a8944d1 (patch) | |
| tree | f1aed9639259b13f4fd9d5db6414cd914e7c8fa2 /lib/tar/test | |
| parent | d8aba88445b3ca51f1e2b45d12688378d69745e9 (diff) | |
libtar: remove need for skip_padding function
In the istream implementation, automatically skip the padding when we
reach end-of-file. Also skip file AND padding when we destroy the
object. Replace the remaining instances with a simple istream_skip
instead and remove the wrapper from libtar.
Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'lib/tar/test')
| -rw-r--r-- | lib/tar/test/tar_istream2.c | 4 | ||||
| -rw-r--r-- | lib/tar/test/tar_target_filled.c | 4 |
2 files changed, 2 insertions, 6 deletions
diff --git a/lib/tar/test/tar_istream2.c b/lib/tar/test/tar_istream2.c index a3f27d5..e5cdca8 100644 --- a/lib/tar/test/tar_istream2.c +++ b/lib/tar/test/tar_istream2.c @@ -77,8 +77,6 @@ int main(int argc, char **argv) ti = sqfs_drop(ti); TEST_EQUAL_UI(((sqfs_object_t *)fp)->refcount, 1); - TEST_ASSERT(skip_padding(fp, 5) == 0); - TEST_EQUAL_UI(((sqfs_object_t *)fp)->refcount, 1); TEST_ASSERT(read_header(fp, &hdr) == 0); TEST_EQUAL_UI(((sqfs_object_t *)fp)->refcount, 1); @@ -102,8 +100,6 @@ int main(int argc, char **argv) ti = sqfs_drop(ti); TEST_EQUAL_UI(((sqfs_object_t *)fp)->refcount, 1); - TEST_ASSERT(skip_padding(fp, 5) == 0); - /* "deep" directory hierarchy containg a hard link */ TEST_EQUAL_UI(((sqfs_object_t *)fp)->refcount, 1); TEST_ASSERT(read_header(fp, &hdr) == 0); diff --git a/lib/tar/test/tar_target_filled.c b/lib/tar/test/tar_target_filled.c index abc6a47..e2e1798 100644 --- a/lib/tar/test/tar_target_filled.c +++ b/lib/tar/test/tar_target_filled.c @@ -53,7 +53,7 @@ int main(int argc, char **argv) TEST_ASSERT(istream_read(fp, buffer, 5) == 5); buffer[5] = '\0'; TEST_STR_EQUAL(buffer, "test\n"); - TEST_ASSERT(skip_padding(fp, 5) == 0); + TEST_ASSERT(istream_skip(fp, 512 - 5) == 0); clear_header(&hdr); TEST_ASSERT(read_header(fp, &hdr) == 0); @@ -65,7 +65,7 @@ int main(int argc, char **argv) TEST_ASSERT(istream_read(fp, buffer, 5) == 5); buffer[5] = '\0'; TEST_STR_EQUAL(buffer, "test\n"); - TEST_ASSERT(skip_padding(fp, 5) == 0); + TEST_ASSERT(istream_skip(fp, 512 - 5) == 0); clear_header(&hdr); /* "deep" directory hierarchy containg a hard link */ |