libarchive: security bump to version 3.3.3
authorBaruch Siach <baruch@tkos.co.il>
Fri, 19 Oct 2018 05:20:05 +0000 (08:20 +0300)
committerPeter Korsgaard <peter@korsgaard.com>
Sat, 20 Oct 2018 12:17:46 +0000 (14:17 +0200)
Fixes CVE-2017-14501: An out-of-bounds read flaw exists in
parse_file_info in archive_read_support_format_iso9660.c in libarchive
3.3.2 when extracting a specially crafted iso9660 iso file, related to
archive_read_format_iso9660_read_header.

Drop upstream patches.

Use upstream provided tarball hash.

Signed-off-by: Baruch Siach <baruch@tkos.co.il>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
package/libarchive/0001-Do-something-sensible-for-empty-strings-to-make-fuzz.patch [deleted file]
package/libarchive/0002-iso9660-validate-directory-record-length.patch [deleted file]
package/libarchive/0003-Avoid-a-read-off-by-one-error-for-UTF16-names-in-RAR.patch [deleted file]
package/libarchive/0004-Reject-LHA-archive-entries-with-negative-size.patch [deleted file]
package/libarchive/libarchive.hash
package/libarchive/libarchive.mk

diff --git a/package/libarchive/0001-Do-something-sensible-for-empty-strings-to-make-fuzz.patch b/package/libarchive/0001-Do-something-sensible-for-empty-strings-to-make-fuzz.patch
deleted file mode 100644 (file)
index 1d1d80d..0000000
+++ /dev/null
@@ -1,42 +0,0 @@
-From fa7438a0ff4033e4741c807394a9af6207940d71 Mon Sep 17 00:00:00 2001
-From: Joerg Sonnenberger <joerg@bec.de>
-Date: Tue, 5 Sep 2017 18:12:19 +0200
-Subject: [PATCH] Do something sensible for empty strings to make fuzzers
- happy.
-
-Signed-off-by: Baruch Siach <baruch@tkos.co.il>
----
-Upstream status: commit fa7438a0ff
-
- libarchive/archive_read_support_format_xar.c | 8 +++++++-
- 1 file changed, 7 insertions(+), 1 deletion(-)
-
-diff --git a/libarchive/archive_read_support_format_xar.c b/libarchive/archive_read_support_format_xar.c
-index 7a22beb9d8e4..93eeacc5e6eb 100644
---- a/libarchive/archive_read_support_format_xar.c
-+++ b/libarchive/archive_read_support_format_xar.c
-@@ -1040,6 +1040,9 @@ atol10(const char *p, size_t char_cnt)
-       uint64_t l;
-       int digit;
-+      if (char_cnt == 0)
-+              return (0);
-+
-       l = 0;
-       digit = *p - '0';
-       while (digit >= 0 && digit < 10  && char_cnt-- > 0) {
-@@ -1054,7 +1057,10 @@ atol8(const char *p, size_t char_cnt)
- {
-       int64_t l;
-       int digit;
--        
-+
-+      if (char_cnt == 0)
-+              return (0);
-+
-       l = 0;
-       while (char_cnt-- > 0) {
-               if (*p >= '0' && *p <= '7')
--- 
-2.14.1
-
diff --git a/package/libarchive/0002-iso9660-validate-directory-record-length.patch b/package/libarchive/0002-iso9660-validate-directory-record-length.patch
deleted file mode 100644 (file)
index 8a0c34f..0000000
+++ /dev/null
@@ -1,78 +0,0 @@
-From f9569c086ff29259c73790db9cbf39fe8fb9d862 Mon Sep 17 00:00:00 2001
-From: John Starks <jostarks@microsoft.com>
-Date: Wed, 25 Jul 2018 12:16:34 -0700
-Subject: [PATCH] iso9660: validate directory record length
-
-Signed-off-by: Baruch Siach <baruch@tkos.co.il>
----
-Upstream status: commit f9569c086ff
-
- .../archive_read_support_format_iso9660.c       | 17 +++++++++++------
- 1 file changed, 11 insertions(+), 6 deletions(-)
-
-diff --git a/libarchive/archive_read_support_format_iso9660.c b/libarchive/archive_read_support_format_iso9660.c
-index f01d37bf682e..089bb7236cd1 100644
---- a/libarchive/archive_read_support_format_iso9660.c
-+++ b/libarchive/archive_read_support_format_iso9660.c
-@@ -409,7 +409,8 @@ static int next_entry_seek(struct archive_read *, struct iso9660 *,
-                   struct file_info **);
- static struct file_info *
-               parse_file_info(struct archive_read *a,
--                  struct file_info *parent, const unsigned char *isodirrec);
-+                  struct file_info *parent, const unsigned char *isodirrec,
-+                  size_t reclen);
- static int    parse_rockridge(struct archive_read *a,
-                   struct file_info *file, const unsigned char *start,
-                   const unsigned char *end);
-@@ -1022,7 +1023,7 @@ read_children(struct archive_read *a, struct file_info *parent)
-                       if (*(p + DR_name_len_offset) == 1
-                           && *(p + DR_name_offset) == '\001')
-                               continue;
--                      child = parse_file_info(a, parent, p);
-+                      child = parse_file_info(a, parent, p, b - p);
-                       if (child == NULL) {
-                               __archive_read_consume(a, skip_size);
-                               return (ARCHIVE_FATAL);
-@@ -1112,7 +1113,7 @@ choose_volume(struct archive_read *a, struct iso9660 *iso9660)
-        */
-       seenJoliet = iso9660->seenJoliet;/* Save flag. */
-       iso9660->seenJoliet = 0;
--      file = parse_file_info(a, NULL, block);
-+      file = parse_file_info(a, NULL, block, vd->size);
-       if (file == NULL)
-               return (ARCHIVE_FATAL);
-       iso9660->seenJoliet = seenJoliet;
-@@ -1144,7 +1145,7 @@ choose_volume(struct archive_read *a, struct iso9660 *iso9660)
-                       return (ARCHIVE_FATAL);
-               }
-               iso9660->seenJoliet = 0;
--              file = parse_file_info(a, NULL, block);
-+              file = parse_file_info(a, NULL, block, vd->size);
-               if (file == NULL)
-                       return (ARCHIVE_FATAL);
-               iso9660->seenJoliet = seenJoliet;
-@@ -1749,7 +1750,7 @@ archive_read_format_iso9660_cleanup(struct archive_read *a)
-  */
- static struct file_info *
- parse_file_info(struct archive_read *a, struct file_info *parent,
--    const unsigned char *isodirrec)
-+    const unsigned char *isodirrec, size_t reclen)
- {
-       struct iso9660 *iso9660;
-       struct file_info *file, *filep;
-@@ -1763,7 +1764,11 @@ parse_file_info(struct archive_read *a, struct file_info *parent,
-       iso9660 = (struct iso9660 *)(a->format->data);
--      dr_len = (size_t)isodirrec[DR_length_offset];
-+      if (reclen == 0 || reclen < (dr_len = (size_t)isodirrec[DR_length_offset])) {
-+              archive_set_error(&a->archive, ARCHIVE_ERRNO_MISC,
-+                      "Invalid directory record length");
-+              return (NULL);
-+      }
-       name_len = (size_t)isodirrec[DR_name_len_offset];
-       location = archive_le32dec(isodirrec + DR_extent_offset);
-       fsize = toi(isodirrec + DR_size_offset, DR_size_size);
--- 
-2.18.0
-
diff --git a/package/libarchive/0003-Avoid-a-read-off-by-one-error-for-UTF16-names-in-RAR.patch b/package/libarchive/0003-Avoid-a-read-off-by-one-error-for-UTF16-names-in-RAR.patch
deleted file mode 100644 (file)
index 13a80d1..0000000
+++ /dev/null
@@ -1,34 +0,0 @@
-From 5562545b5562f6d12a4ef991fae158bf4ccf92b6 Mon Sep 17 00:00:00 2001
-From: Joerg Sonnenberger <joerg@bec.de>
-Date: Sat, 9 Sep 2017 17:47:32 +0200
-Subject: [PATCH] Avoid a read off-by-one error for UTF16 names in RAR
- archives.
-
-Reported-By: OSS-Fuzz issue 573
-Signed-off-by: Baruch Siach <baruch@tkos.co.il>
----
-Upstream status: commit 5562545b5562
-
- libarchive/archive_read_support_format_rar.c | 6 +++++-
- 1 file changed, 5 insertions(+), 1 deletion(-)
-
-diff --git a/libarchive/archive_read_support_format_rar.c b/libarchive/archive_read_support_format_rar.c
-index cbb14c32dc3b..751de6979ba5 100644
---- a/libarchive/archive_read_support_format_rar.c
-+++ b/libarchive/archive_read_support_format_rar.c
-@@ -1496,7 +1496,11 @@ read_header(struct archive_read *a, struct archive_entry *entry,
-         return (ARCHIVE_FATAL);
-       }
-       filename[filename_size++] = '\0';
--      filename[filename_size++] = '\0';
-+      /*
-+       * Do not increment filename_size here as the computations below
-+       * add the space for the terminating NUL explicitly.
-+       */
-+      filename[filename_size] = '\0';
-       /* Decoded unicode form is UTF-16BE, so we have to update a string
-        * conversion object for it. */
--- 
-2.18.0
-
diff --git a/package/libarchive/0004-Reject-LHA-archive-entries-with-negative-size.patch b/package/libarchive/0004-Reject-LHA-archive-entries-with-negative-size.patch
deleted file mode 100644 (file)
index 628e74a..0000000
+++ /dev/null
@@ -1,32 +0,0 @@
-From 2c8c83b9731ff822fad6cc8c670ea5519c366a14 Mon Sep 17 00:00:00 2001
-From: Joerg Sonnenberger <joerg@bec.de>
-Date: Thu, 19 Jul 2018 21:14:53 +0200
-Subject: [PATCH] Reject LHA archive entries with negative size.
-
-Signed-off-by: Baruch Siach <baruch@tkos.co.il>
----
-Upstream status: commit 2c8c83b9731
-
- libarchive/archive_read_support_format_lha.c | 6 ++++++
- 1 file changed, 6 insertions(+)
-
-diff --git a/libarchive/archive_read_support_format_lha.c b/libarchive/archive_read_support_format_lha.c
-index b8ef4ae10ece..95c99bb1f31e 100644
---- a/libarchive/archive_read_support_format_lha.c
-+++ b/libarchive/archive_read_support_format_lha.c
-@@ -701,6 +701,12 @@ archive_read_format_lha_read_header(struct archive_read *a,
-        * Prepare variables used to read a file content.
-        */
-       lha->entry_bytes_remaining = lha->compsize;
-+      if (lha->entry_bytes_remaining < 0) {
-+              archive_set_error(&a->archive,
-+                  ARCHIVE_ERRNO_FILE_FORMAT,
-+                  "Invalid LHa entry size");
-+              return (ARCHIVE_FATAL);
-+      }
-       lha->entry_offset = 0;
-       lha->entry_crc_calculated = 0;
--- 
-2.18.0
-
index 60ea0d16025eca3c7c15e3e9dcca0d306c40c59e..cde48c842366edae3f84d1e373bb854004f0f01b 100644 (file)
@@ -1,3 +1,4 @@
+# From http://www.libarchive.org/downloads/libarchive-3.3.3.sha512.txt
+sha512 9d12b47d6976efa9f98e62c25d8b85fd745d4e9ca7b7e6d36bfe095dfe5c4db017d4e785d110f3758f5938dad6f1a1b009267fd7e82cb7212e93e1aea237bab7  libarchive-3.3.3.tar.gz
 # Locally computed:
-sha256  ed2dbd6954792b2c054ccf8ec4b330a54b85904a80cef477a1c74643ddafa0ce  libarchive-3.3.2.tar.gz
 sha256  ae6f35cc1979beb316e4d6431fc34c6fc59f0dd126b425c8552bb41c86e4825d  COPYING
index 1c8391bda9b9455f180e8caa2598c8d21860ca8a..b7619b03dbbbb52ffc2754a8c0fefb5c63c80373 100644 (file)
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-LIBARCHIVE_VERSION = 3.3.2
+LIBARCHIVE_VERSION = 3.3.3
 LIBARCHIVE_SITE = http://www.libarchive.org/downloads
 LIBARCHIVE_INSTALL_STAGING = YES
 LIBARCHIVE_LICENSE = BSD-2-Clause, BSD-3-Clause