diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2009-08-28 21:09:51 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2009-08-28 21:09:51 +0200 |
commit | 0a130d510dc5272dc2fc7bc5a116f0694c8bd2a9 (patch) | |
tree | 029e91a6302547df06524ae6d73ecb5b85e35360 /archival/libunarchive | |
parent | 7f2149489fe62373a13792d3de6c33c39725c76c (diff) | |
download | busybox-w32-0a130d510dc5272dc2fc7bc5a116f0694c8bd2a9.tar.gz busybox-w32-0a130d510dc5272dc2fc7bc5a116f0694c8bd2a9.tar.bz2 busybox-w32-0a130d510dc5272dc2fc7bc5a116f0694c8bd2a9.zip |
rpm2cpio: handle unseekable input correctly
function old new delta
data_skip 14 20 +6
seek_by_jump 67 72 +5
data_align 81 84 +3
seek_by_read 20 19 -1
skip_header 99 94 -5
rpm2cpio_main 183 177 -6
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 3/3 up/down: 14/-12) Total: 2 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'archival/libunarchive')
-rw-r--r-- | archival/libunarchive/data_align.c | 2 | ||||
-rw-r--r-- | archival/libunarchive/data_skip.c | 2 | ||||
-rw-r--r-- | archival/libunarchive/seek_by_jump.c | 6 | ||||
-rw-r--r-- | archival/libunarchive/seek_by_read.c | 6 |
4 files changed, 8 insertions, 8 deletions
diff --git a/archival/libunarchive/data_align.c b/archival/libunarchive/data_align.c index 9f2e8432f..0c8542bf5 100644 --- a/archival/libunarchive/data_align.c +++ b/archival/libunarchive/data_align.c | |||
@@ -10,6 +10,6 @@ void FAST_FUNC data_align(archive_handle_t *archive_handle, unsigned boundary) | |||
10 | { | 10 | { |
11 | unsigned skip_amount = (boundary - (archive_handle->offset % boundary)) % boundary; | 11 | unsigned skip_amount = (boundary - (archive_handle->offset % boundary)) % boundary; |
12 | 12 | ||
13 | archive_handle->seek(archive_handle, skip_amount); | 13 | archive_handle->seek(archive_handle->src_fd, skip_amount); |
14 | archive_handle->offset += skip_amount; | 14 | archive_handle->offset += skip_amount; |
15 | } | 15 | } |
diff --git a/archival/libunarchive/data_skip.c b/archival/libunarchive/data_skip.c index 438750fe5..06d3dced4 100644 --- a/archival/libunarchive/data_skip.c +++ b/archival/libunarchive/data_skip.c | |||
@@ -8,5 +8,5 @@ | |||
8 | 8 | ||
9 | void FAST_FUNC data_skip(archive_handle_t *archive_handle) | 9 | void FAST_FUNC data_skip(archive_handle_t *archive_handle) |
10 | { | 10 | { |
11 | archive_handle->seek(archive_handle, archive_handle->file_header->size); | 11 | archive_handle->seek(archive_handle->src_fd, archive_handle->file_header->size); |
12 | } | 12 | } |
diff --git a/archival/libunarchive/seek_by_jump.c b/archival/libunarchive/seek_by_jump.c index 0a259c963..7181cb3ce 100644 --- a/archival/libunarchive/seek_by_jump.c +++ b/archival/libunarchive/seek_by_jump.c | |||
@@ -6,13 +6,13 @@ | |||
6 | #include "libbb.h" | 6 | #include "libbb.h" |
7 | #include "unarchive.h" | 7 | #include "unarchive.h" |
8 | 8 | ||
9 | void FAST_FUNC seek_by_jump(const archive_handle_t *archive_handle, unsigned amount) | 9 | void FAST_FUNC seek_by_jump(int fd, off_t amount) |
10 | { | 10 | { |
11 | if (amount | 11 | if (amount |
12 | && lseek(archive_handle->src_fd, (off_t) amount, SEEK_CUR) == (off_t) -1 | 12 | && lseek(fd, amount, SEEK_CUR) == (off_t) -1 |
13 | ) { | 13 | ) { |
14 | if (errno == ESPIPE) | 14 | if (errno == ESPIPE) |
15 | seek_by_read(archive_handle, amount); | 15 | seek_by_read(fd, amount); |
16 | else | 16 | else |
17 | bb_perror_msg_and_die("seek failure"); | 17 | bb_perror_msg_and_die("seek failure"); |
18 | } | 18 | } |
diff --git a/archival/libunarchive/seek_by_read.c b/archival/libunarchive/seek_by_read.c index 2326a7512..af65e5d85 100644 --- a/archival/libunarchive/seek_by_read.c +++ b/archival/libunarchive/seek_by_read.c | |||
@@ -9,8 +9,8 @@ | |||
9 | /* If we are reading through a pipe, or from stdin then we can't lseek, | 9 | /* If we are reading through a pipe, or from stdin then we can't lseek, |
10 | * we must read and discard the data to skip over it. | 10 | * we must read and discard the data to skip over it. |
11 | */ | 11 | */ |
12 | void FAST_FUNC seek_by_read(const archive_handle_t *archive_handle, unsigned jump_size) | 12 | void FAST_FUNC seek_by_read(int fd, off_t amount) |
13 | { | 13 | { |
14 | if (jump_size) | 14 | if (amount) |
15 | bb_copyfd_exact_size(archive_handle->src_fd, -1, jump_size); | 15 | bb_copyfd_exact_size(fd, -1, amount); |
16 | } | 16 | } |