diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-07-12 13:20:44 +0400 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-07-12 13:20:44 +0400 |
commit | a46dd89e9451ec73a4df54427110cdfc28d8b031 (patch) | |
tree | 2e470b3c0236524905a6d399c5207cccaef2fc7b /archival/libunarchive/get_header_cpio.c | |
parent | 39acf453353a41a78fbc220360e884eb0eb33a59 (diff) |
cpio: internalize archive_xread_all_eof. add a few paranoia checks
for corrupted cpio files.
modprobe-small: remove stray include
route: small code shrink
function old new delta
get_header_cpio 958 980 +22
archive_xread_all_eof 33 - -33
------------------------------------------------------------------------------
(add/remove: 0/1 grow/shrink: 1/0 up/down: 22/-33) Total: -11 bytes
Diffstat (limited to 'archival/libunarchive/get_header_cpio.c')
-rw-r--r-- | archival/libunarchive/get_header_cpio.c | 69 |
1 files changed, 37 insertions, 32 deletions
diff --git a/archival/libunarchive/get_header_cpio.c b/archival/libunarchive/get_header_cpio.c index 307d2a66d..93f6c61aa 100644 --- a/archival/libunarchive/get_header_cpio.c +++ b/archival/libunarchive/get_header_cpio.c @@ -21,26 +21,28 @@ char FAST_FUNC get_header_cpio(archive_handle_t *archive_handle) { file_header_t *file_header = archive_handle->file_header; char cpio_header[110]; - char dummy[16]; int namesize; int major, minor, nlink, mode, inode; unsigned size, uid, gid, mtime; -#define saved_hardlinks (*(hardlinks_t **)(&archive_handle->ah_priv[0])) -#define saved_hardlinks_created (*(hardlinks_t **)(&archive_handle->ah_priv[1])) +#define hardlinks_to_create (*(hardlinks_t **)(&archive_handle->ah_priv[0])) +#define created_hardlinks (*(hardlinks_t **)(&archive_handle->ah_priv[1])) // if (!archive_handle->ah_priv_inited) { // archive_handle->ah_priv_inited = 1; -// saved_hardlinks = NULL; -// saved_hardlinks_created = NULL; +// hardlinks_to_create = NULL; +// created_hardlinks = NULL; // } /* There can be padding before archive header */ data_align(archive_handle, 4); -//TODO: this function is used only here, make it static? - if (archive_xread_all_eof(archive_handle, (unsigned char*)cpio_header, 110) == 0) { + size = full_read(archive_handle->src_fd, cpio_header, 110); + if (size == 0) { goto create_hardlinks; } + if (size != 110) { + bb_error_msg_and_die("short read"); + } archive_handle->offset += 110; if (strncmp(&cpio_header[0], "07070", 5) != 0 @@ -49,20 +51,21 @@ char FAST_FUNC get_header_cpio(archive_handle_t *archive_handle) bb_error_msg_and_die("unsupported cpio format, use newc or crc"); } - sscanf(cpio_header + 6, + if (sscanf(cpio_header + 6, "%8x" "%8x" "%8x" "%8x" - "%8x" "%8x" "%8x" /*maj,min:*/ "%16c" - /*rmaj,rmin:*/"%8x" "%8x" "%8x" /*chksum:*/ "%8c", + "%8x" "%8x" "%8x" /*maj,min:*/ "%*16c" + /*rmaj,rmin:*/"%8x" "%8x" "%8x" /*chksum: "%*8c"*/, &inode, &mode, &uid, &gid, - &nlink, &mtime, &size, dummy, - &major, &minor, &namesize, dummy); + &nlink, &mtime, &size, + &major, &minor, &namesize) != 10) + bb_error_msg_and_die("damaged cpio file"); file_header->mode = mode; file_header->uid = uid; file_header->gid = gid; file_header->mtime = mtime; file_header->size = size; - namesize &= 0x1fff; /* paranoia: names can't be that long */ + namesize &= 0x1fff; /* paranoia: limit names to 8k chars */ file_header->name = xzalloc(namesize + 1); /* Read in filename */ xread(archive_handle->src_fd, file_header->name, namesize); @@ -77,17 +80,17 @@ char FAST_FUNC get_header_cpio(archive_handle_t *archive_handle) goto create_hardlinks; } + file_header->link_target = NULL; if (S_ISLNK(file_header->mode)) { + file_header->size &= 0x1fff; /* paranoia: limit names to 8k chars */ file_header->link_target = xzalloc(file_header->size + 1); xread(archive_handle->src_fd, file_header->link_target, file_header->size); archive_handle->offset += file_header->size; file_header->size = 0; /* Stop possible seeks in future */ - } else { - file_header->link_target = NULL; } // TODO: data_extract_all can't deal with hardlinks to non-files... -// (should be !S_ISDIR instead of S_ISREG here) +// when fixed, change S_ISREG to !S_ISDIR here if (nlink > 1 && S_ISREG(file_header->mode)) { hardlinks_t *new = xmalloc(sizeof(*new) + namesize); @@ -99,13 +102,13 @@ char FAST_FUNC get_header_cpio(archive_handle_t *archive_handle) strcpy(new->name, file_header->name); /* Put file on a linked list for later */ if (size == 0) { - new->next = saved_hardlinks; - saved_hardlinks = new; + new->next = hardlinks_to_create; + hardlinks_to_create = new; return EXIT_SUCCESS; /* Skip this one */ /* TODO: this breaks cpio -t (it does not show hardlinks) */ } - new->next = saved_hardlinks_created; - saved_hardlinks_created = new; + new->next = created_hardlinks; + created_hardlinks = new; } file_header->device = makedev(major, minor); @@ -129,18 +132,23 @@ char FAST_FUNC get_header_cpio(archive_handle_t *archive_handle) free(file_header->link_target); free(file_header->name); - while (saved_hardlinks) { + while (hardlinks_to_create) { hardlinks_t *cur; - hardlinks_t *make_me = saved_hardlinks; - saved_hardlinks = make_me->next; + hardlinks_t *make_me = hardlinks_to_create; + + hardlinks_to_create = make_me->next; memset(file_header, 0, sizeof(*file_header)); + file_header->mtime = make_me->mtime; file_header->name = make_me->name; file_header->mode = make_me->mode; + file_header->uid = make_me->uid; + file_header->gid = make_me->gid; /*file_header->size = 0;*/ + /*file_header->link_target = NULL;*/ /* Try to find a file we are hardlinked to */ - cur = saved_hardlinks_created; + cur = created_hardlinks; while (cur) { /* TODO: must match maj/min too! */ if (cur->inode == make_me->inode) { @@ -155,20 +163,17 @@ char FAST_FUNC get_header_cpio(archive_handle_t *archive_handle) } /* Oops... no file with such inode was created... do it now * (happens when hardlinked files are empty (zero length)) */ - file_header->mtime = make_me->mtime; - file_header->uid = make_me->uid ; - file_header->gid = make_me->gid ; if (archive_handle->filter(archive_handle) == EXIT_SUCCESS) archive_handle->action_data(archive_handle); /* Move to the list of created hardlinked files */ - make_me->next = saved_hardlinks_created; - saved_hardlinks_created = make_me; + make_me->next = created_hardlinks; + created_hardlinks = make_me; next_link: ; } - while (saved_hardlinks_created) { - hardlinks_t *p = saved_hardlinks_created; - saved_hardlinks_created = p->next; + while (created_hardlinks) { + hardlinks_t *p = created_hardlinks; + created_hardlinks = p->next; free(p); } |