Co-authored-by: Henry Li <lihl@microsoft.com>
This commit is contained in:
Henry Li 2022-01-07 10:03:01 -08:00 коммит произвёл GitHub
Родитель e5db731693
Коммит 5026c400b3
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
4 изменённых файлов: 8 добавлений и 63 удалений

Просмотреть файл

@ -1,56 +0,0 @@
From f25d254dfd4243698c31a4f3153d4ac72aa9e9bd Mon Sep 17 00:00:00 2001
From: Samuel Sapalski <samuel.sapalski@nokia.com>
Date: Wed, 3 Mar 2021 16:31:22 +0100
Subject: decompress_gunzip: Fix DoS if gzip is corrupt
On certain corrupt gzip files, huft_build will set the error bit on
the result pointer. If afterwards abort_unzip is called huft_free
might run into a segmentation fault or an invalid pointer to
free(p).
In order to mitigate this, we check in huft_free if the error bit
is set and clear it before the linked list is freed.
Signed-off-by: Samuel Sapalski <samuel.sapalski@nokia.com>
Signed-off-by: Peter Kaestle <peter.kaestle@nokia.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
---
archival/libarchive/decompress_gunzip.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/archival/libarchive/decompress_gunzip.c b/archival/libarchive/decompress_gunzip.c
index eb3b64930..e93cd5005 100644
--- a/archival/libarchive/decompress_gunzip.c
+++ b/archival/libarchive/decompress_gunzip.c
@@ -220,10 +220,20 @@ static const uint8_t border[] ALIGN1 = {
* each table.
* t: table to free
*/
+#define BAD_HUFT(p) ((uintptr_t)(p) & 1)
+#define ERR_RET ((huft_t*)(uintptr_t)1)
static void huft_free(huft_t *p)
{
huft_t *q;
+ /*
+ * If 'p' has the error bit set we have to clear it, otherwise we might run
+ * into a segmentation fault or an invalid pointer to free(p)
+ */
+ if (BAD_HUFT(p)) {
+ p = (huft_t*)((uintptr_t)(p) ^ (uintptr_t)(ERR_RET));
+ }
+
/* Go through linked list, freeing from the malloced (t[-1]) address. */
while (p) {
q = (--p)->v.t;
@@ -289,8 +299,6 @@ static unsigned fill_bitbuffer(STATE_PARAM unsigned bitbuffer, unsigned *current
* or a valid pointer to a Huffman table, ORed with 0x1 if incompete table
* is given: "fixed inflate" decoder feeds us such data.
*/
-#define BAD_HUFT(p) ((uintptr_t)(p) & 1)
-#define ERR_RET ((huft_t*)(uintptr_t)1)
static huft_t* huft_build(const unsigned *b, const unsigned n,
const unsigned s, const struct cp_ext *cp_ext,
unsigned *m)
--
cgit v1.2.1

Просмотреть файл

@ -1,6 +1,6 @@
{ {
"Signatures": { "Signatures": {
"busybox-1.32.0.tar.bz2": "c35d87f1d04b2b153d33c275c2632e40d388a88f19a9e71727e0bbbff51fe689", "busybox-1.35.0.tar.bz2": "faeeb244c35a348a334f4a59e44626ee870fb07b6884d68c10ae8bc19f83a694",
"busybox-petitboot.config": "28a4006863e0125bb564159c120067cb83b52ee0a829579cd399274cc78a10be", "busybox-petitboot.config": "28a4006863e0125bb564159c120067cb83b52ee0a829579cd399274cc78a10be",
"busybox-static.config": "6f2f534548da57df8b1f5fd4dfe6ceece0f1b97bf7d0baa4c484ac9850cf8e37" "busybox-static.config": "6f2f534548da57df8b1f5fd4dfe6ceece0f1b97bf7d0baa4c484ac9850cf8e37"
} }

Просмотреть файл

@ -1,7 +1,7 @@
Summary: Statically linked binary providing simplified versions of system commands Summary: Statically linked binary providing simplified versions of system commands
Name: busybox Name: busybox
Version: 1.32.0 Version: 1.35.0
Release: 2%{?dist} Release: 1%{?dist}
License: GPLv2 License: GPLv2
Vendor: Microsoft Corporation Vendor: Microsoft Corporation
Distribution: Mariner Distribution: Mariner
@ -10,7 +10,6 @@ Source: https://www.busybox.net/downloads/%{name}-%{version}.tar.bz2
Source1: busybox-static.config Source1: busybox-static.config
Source2: busybox-petitboot.config Source2: busybox-petitboot.config
Patch0: busybox-1.31.1-stime-fix.patch Patch0: busybox-1.31.1-stime-fix.patch
Patch1: CVE-2021-28831.patch
BuildRequires: gcc BuildRequires: gcc
BuildRequires: glibc-devel BuildRequires: glibc-devel
BuildRequires: libselinux-devel >= 1.27.7-2 BuildRequires: libselinux-devel >= 1.27.7-2
@ -43,7 +42,6 @@ better suited to normal use.
%prep %prep
%setup -q %setup -q
%patch0 -p1 -b .stime %patch0 -p1 -b .stime
%patch1 -p1
%build %build
# create static busybox - the executable is kept as busybox-static # create static busybox - the executable is kept as busybox-static
@ -130,6 +128,9 @@ install -m 644 docs/busybox.petitboot.1 %{buildroot}/%{_mandir}/man1/busybox.pet
%{_mandir}/man1/busybox.petitboot.1.gz %{_mandir}/man1/busybox.petitboot.1.gz
%changelog %changelog
* Thu Jan 06 2022 Henry Li <lihl@microsoft.com> - 1.35.0-1
- Upgrade to version 1.35.0
* Fri Mar 26 2021 Henry Beberman <henry.beberman@microsoft.com> - 1.32.0-2 * Fri Mar 26 2021 Henry Beberman <henry.beberman@microsoft.com> - 1.32.0-2
- Patch CVE-2021-28831 - Patch CVE-2021-28831

Просмотреть файл

@ -1487,8 +1487,8 @@
"type": "other", "type": "other",
"other": { "other": {
"name": "busybox", "name": "busybox",
"version": "1.32.0", "version": "1.35.0",
"downloadUrl": "http://www.busybox.net/downloads/busybox-1.32.0.tar.bz2" "downloadUrl": "http://www.busybox.net/downloads/busybox-1.35.0.tar.bz2"
} }
} }
}, },