Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 28 Jun 2019 22:41:17 +0000 (UTC)
From:      Martin Matuska <mm@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r349527 - in head/contrib/libarchive/libarchive: . test
Message-ID:  <201906282241.x5SMfHdU069419@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mm
Date: Fri Jun 28 22:41:17 2019
New Revision: 349527
URL: https://svnweb.freebsd.org/changeset/base/349527

Log:
  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

Added:
  head/contrib/libarchive/libarchive/test/test_read_format_rar5_arm_filter_on_window_boundary.rar.uu
     - copied unchanged from r349454, vendor/libarchive/dist/libarchive/test/test_read_format_rar5_arm_filter_on_window_boundary.rar.uu
Modified:
  head/contrib/libarchive/libarchive/archive_read.c
  head/contrib/libarchive/libarchive/archive_read_disk_posix.c
  head/contrib/libarchive/libarchive/archive_read_support_format_rar5.c
  head/contrib/libarchive/libarchive/test/test_read_format_rar5.c
Directory Properties:
  head/contrib/libarchive/   (props changed)

Modified: head/contrib/libarchive/libarchive/archive_read.c
==============================================================================
--- head/contrib/libarchive/libarchive/archive_read.c	Fri Jun 28 22:40:34 2019	(r349526)
+++ head/contrib/libarchive/libarchive/archive_read.c	Fri Jun 28 22:41:17 2019	(r349527)
@@ -844,7 +844,8 @@ archive_read_data(struct archive *_a, void *buff, size
 	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;

Modified: head/contrib/libarchive/libarchive/archive_read_disk_posix.c
==============================================================================
--- head/contrib/libarchive/libarchive/archive_read_disk_posix.c	Fri Jun 28 22:40:34 2019	(r349526)
+++ head/contrib/libarchive/libarchive/archive_read_disk_posix.c	Fri Jun 28 22:41:17 2019	(r349527)
@@ -1143,6 +1143,8 @@ _archive_read_next_header2(struct archive *_a, struct 
 		t->entry_fd = -1;
 	}
 
+	archive_entry_clear(entry);
+
 	for (;;) {
 		r = next_entry(a, t, entry);
 		if (t->entry_fd >= 0) {

Modified: head/contrib/libarchive/libarchive/archive_read_support_format_rar5.c
==============================================================================
--- head/contrib/libarchive/libarchive/archive_read_support_format_rar5.c	Fri Jun 28 22:40:34 2019	(r349526)
+++ head/contrib/libarchive/libarchive/archive_read_support_format_rar5.c	Fri Jun 28 22:41:17 2019	(r349527)
@@ -623,9 +623,9 @@ static int run_arm_filter(struct rar5* rar, struct fil
 	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) &

Modified: head/contrib/libarchive/libarchive/test/test_read_format_rar5.c
==============================================================================
--- head/contrib/libarchive/libarchive/test/test_read_format_rar5.c	Fri Jun 28 22:40:34 2019	(r349526)
+++ head/contrib/libarchive/libarchive/test/test_read_format_rar5.c	Fri Jun 28 22:41:17 2019	(r349527)
@@ -1215,3 +1215,18 @@ DEFINE_TEST(test_read_format_rar5_different_window_siz
 
 	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();
+}

Copied: head/contrib/libarchive/libarchive/test/test_read_format_rar5_arm_filter_on_window_boundary.rar.uu (from r349454, vendor/libarchive/dist/libarchive/test/test_read_format_rar5_arm_filter_on_window_boundary.rar.uu)
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/contrib/libarchive/libarchive/test/test_read_format_rar5_arm_filter_on_window_boundary.rar.uu	Fri Jun 28 22:41:17 2019	(r349527, copy of r349454, vendor/libarchive/dist/libarchive/test/test_read_format_rar5_arm_filter_on_window_boundary.rar.uu)
@@ -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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201906282241.x5SMfHdU069419>