MFV r349454:
Sync libarchive with vendor. Relevant vendor changes: PR #1217: RAR5 reader - fix ARM filter going beyond window buffer boundary (OSS-Fuzz 15431) PR #1218: Fixes to sparse file handling MFC after: 1 week
This commit is contained in:
commit
74e515127c
@ -844,7 +844,8 @@ archive_read_data(struct archive *_a, void *buff, size_t s)
|
||||
dest = (char *)buff;
|
||||
|
||||
while (s > 0) {
|
||||
if (a->read_data_remaining == 0) {
|
||||
if (a->read_data_offset == a->read_data_output_offset &&
|
||||
a->read_data_remaining == 0) {
|
||||
read_buf = a->read_data_block;
|
||||
a->read_data_is_posix_read = 1;
|
||||
a->read_data_requested = s;
|
||||
|
@ -1143,6 +1143,8 @@ _archive_read_next_header2(struct archive *_a, struct archive_entry *entry)
|
||||
t->entry_fd = -1;
|
||||
}
|
||||
|
||||
archive_entry_clear(entry);
|
||||
|
||||
for (;;) {
|
||||
r = next_entry(a, t, entry);
|
||||
if (t->entry_fd >= 0) {
|
||||
|
@ -623,9 +623,9 @@ static int run_arm_filter(struct rar5* rar, struct filter_info* flt) {
|
||||
for(i = 0; i < flt->block_length - 3; i += 4) {
|
||||
uint8_t* b = &rar->cstate.window_buf[
|
||||
(rar->cstate.solid_offset +
|
||||
flt->block_start + i) & rar->cstate.window_mask];
|
||||
flt->block_start + i + 3) & rar->cstate.window_mask];
|
||||
|
||||
if(b[3] == 0xEB) {
|
||||
if(*b == 0xEB) {
|
||||
/* 0xEB = ARM's BL (branch + link) instruction. */
|
||||
offset = read_filter_data(rar,
|
||||
(rar->cstate.solid_offset + flt->block_start + i) &
|
||||
|
@ -1215,3 +1215,18 @@ DEFINE_TEST(test_read_format_rar5_different_window_size)
|
||||
|
||||
EPILOGUE();
|
||||
}
|
||||
|
||||
DEFINE_TEST(test_read_format_rar5_arm_filter_on_window_boundary)
|
||||
{
|
||||
char buf[4096];
|
||||
PROLOGUE("test_read_format_rar5_arm_filter_on_window_boundary.rar");
|
||||
|
||||
/* Return codes of those calls are ignored, because this sample file
|
||||
* is invalid. However, the unpacker shouldn't produce any SIGSEGV
|
||||
* errors during processing. */
|
||||
|
||||
(void) archive_read_next_header(a, &ae);
|
||||
while(0 != archive_read_data(a, buf, sizeof(buf))) {}
|
||||
|
||||
EPILOGUE();
|
||||
}
|
||||
|
@ -0,0 +1,9 @@
|
||||
begin 600 test_read_format_rar5_arm_filter_on_window_boundary.rar
|
||||
M4F%R(1H'`0"-[P+2``(''(`'`/[_(`#_!``"(0$``/X(TB`!'O___P@``/W_
|
||||
M_Q``_]U84%"0_P1LAFVQ9,S,M[$`20"#__\`_P#_`/G___!DSR0V2+$`20`Z
|
||||
M@R[_______\I:!<**-@P70D`KB1!<YOZFQ/___^<`^5L*0```/________\_
|
||||
M`0#__RE@%PHHV#!="0"N)$%S"```_?]84/7___]0D/\$;(9ML63,S/\R'Q\?
|
||||
M'Q\?'Q\?'Q\?'Q\?'[$`20"#__\`_P#_`/G___!DSR0V2+$`20`Z@R[_____
|
||||
0_Q\?'Q\?'Q\?'Q]5"E`*4```
|
||||
`
|
||||
end
|
Loading…
Reference in New Issue
Block a user