diff options
| author | Denys Vlasenko <vda.linux@googlemail.com> | 2017-07-11 22:10:52 +0200 |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2017-07-11 22:10:52 +0200 |
| commit | 38d966943f5288bb1f2e7219f50a92753c730b14 (patch) | |
| tree | 81c4f3651e86b8aaed94cbae0ffaec3e3fe2dbc3 /archival/unzip.c | |
| parent | ce55284ed6f5b5303d9e45d034287c728ef9b05a (diff) | |
| download | busybox-38d966943f5288bb1f2e7219f50a92753c730b14.tar.gz | |
unzip: TODO for symlink support
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'archival/unzip.c')
| -rw-r--r-- | archival/unzip.c | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/archival/unzip.c b/archival/unzip.c index 51025529d..488dcf31d 100644 --- a/archival/unzip.c +++ b/archival/unzip.c @@ -346,6 +346,12 @@ static void unzip_extract(zip_header_t *zip, int dst_fd) return; } +// NB: to support symlinks, need to extract symlink target. A-la: +// xstate.mem_output_size_max = zip->fmt.ucmpsize; +// ...unpack... +// if (xstate.mem_output_buf) { success, xstate.mem_output_size is the size } +// Although archives I've seen have fmt.method == 0 for symlinks. + init_transformer_state(&xstate); xstate.bytes_in = zip->fmt.cmpsize; xstate.src_fd = zip_fd; @@ -685,6 +691,20 @@ int unzip_main(int argc, char **argv) zip.fmt.cmpsize = cdf.fmt.cmpsize; zip.fmt.ucmpsize = cdf.fmt.ucmpsize; } +// Seen in some zipfiles: central directory 9 byte extra field contains +// a subfield with ID 0x5455 and 5 data bytes, which is a Unix-style UTC mtime. +// Local header version: +// u16 0x5455 ("UT") +// u16 size (1 + 4 * n) +// u8 flags: bit 0:mtime is present, bit 1:atime is present, bit 2:ctime is present +// u32 mtime +// u32 atime +// u32 ctime +// Central header version: +// u16 0x5455 ("UT") +// u16 size (5 (or 1?)) +// u8 flags: bit 0:mtime is present, bit 1:atime is present, bit 2:ctime is present +// u32 mtime (CDF does not store atime/ctime) # else /* CDF has the same data as local header, no need to read the latter... * ...not really. An archive was seen with cdf.extra_len == 6 but @@ -699,6 +719,7 @@ int unzip_main(int argc, char **argv) if ((cdf.fmt.version_made_by >> 8) == 3) { /* This archive is created on Unix */ dir_mode = file_mode = (cdf.fmt.external_attributes >> 16); +//TODO: if (S_ISLNK(file_mode)) this is a symlink } } #endif |