diff options
author | bkylerussell@gmail.com <bkylerussell@gmail.com> | 2022-02-08 07:15:08 -0500 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2022-02-23 23:43:42 +0000 |
commit | 7b27c85ed93cd6d193971a0d535bfa47454801f9 (patch) | |
tree | 7e9bd117d108e7bbc20cb9e40ac1d3a3a00b0dc0 /meta/recipes-devtools/rpm | |
parent | e3f9b3a49750ab71c2e2fb0f1c40dc10e3eeea05 (diff) | |
download | poky-7b27c85ed93cd6d193971a0d535bfa47454801f9.tar.gz |
rpm: fix intermittent compression failure in do_package_write_rpm
rpmbuild can start processing random memory when processing the value
provided by XZ_THREADS, and unintentionally disable encoding for a
file descriptor that in fact requires encoding to be enabled in order
for lzwrite() to actually create an rpm.
*** Fdopen(0x7f2030002b30,w6T16.xzdio) | fdio 23 fp (nil)
==> lzopen_internal("w6T16", 23, 1)
==> lzopen_internal set encoding
==> lzopen_internal clear encoding
==> Fdopen(0x7f2030002b30,"w6T16.xzdio") returns fd 0x7f2030002b30 | xzdio 0x7f2030004e30 fp 23 | fdio -1 fp (nil)
==> lzwrite(0x7f2030004e30, 0x7f20789d8070, 6) encoding 0
==> Fwrite(0x7f2030002b30,0x7f20789d8070,6) rc -1 | xzdio 0x7f2030004e30 fp 23 | fdio -1 fp (nil)
error: create archive failed: cpio: write
When the encoding bit gets cleared on the LZFILE* struct, lzwrite() then
rightfully complains when it detects !lzfile->encoding, which then gets
bubbled up as a write failure when we go to create the archive.
This fix is available in the rpm 4.17-release.
(From OE-Core rev: b093005d31467d89b00af621f86eb5cac7f845af)
Signed-off-by: Kyle Russell <bkylerussell@gmail.com>
Signed-off-by: Steve Sakoman <steve@sakoman.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-devtools/rpm')
-rw-r--r-- | meta/recipes-devtools/rpm/files/0001-rpmio-Fix-lzopen_internal-mode-parsing-when-Tn-is-us.patch | 34 | ||||
-rw-r--r-- | meta/recipes-devtools/rpm/rpm_4.14.2.1.bb | 1 |
2 files changed, 35 insertions, 0 deletions
diff --git a/meta/recipes-devtools/rpm/files/0001-rpmio-Fix-lzopen_internal-mode-parsing-when-Tn-is-us.patch b/meta/recipes-devtools/rpm/files/0001-rpmio-Fix-lzopen_internal-mode-parsing-when-Tn-is-us.patch new file mode 100644 index 0000000000..9a5ebb9115 --- /dev/null +++ b/meta/recipes-devtools/rpm/files/0001-rpmio-Fix-lzopen_internal-mode-parsing-when-Tn-is-us.patch | |||
@@ -0,0 +1,34 @@ | |||
1 | From 405fc8998181353bd510864ca251dc233afec276 Mon Sep 17 00:00:00 2001 | ||
2 | From: Vitaly Chikunov <vt@altlinux.org> | ||
3 | Date: Wed, 6 Jan 2021 23:43:41 +0300 | ||
4 | Subject: [PATCH] rpmio: Fix lzopen_internal mode parsing when 'Tn' is used | ||
5 | |||
6 | When there is number after "T" (suggested number of threads or "0" for | ||
7 | getncpus), lzopen_internal() mode parser would skip one byte, and when | ||
8 | it's at the end of the string it would then parse undesired garbage from | ||
9 | the memory, making intermittent compression failures. | ||
10 | |||
11 | Fixes: 7740d1098 ("Add support for multithreaded xz compression") | ||
12 | Signed-off-by: Vitaly Chikunov <vt@altlinux.org> | ||
13 | |||
14 | Upstream-Status: Backport [https://github.com/rpm-software-management/rpm/commit/405fc8998181353bd510864ca251dc233afec276] | ||
15 | |||
16 | --- | ||
17 | rpmio/rpmio.c | 1 + | ||
18 | 1 file changed, 1 insertion(+) | ||
19 | |||
20 | diff --git a/rpmio/rpmio.c b/rpmio/rpmio.c | ||
21 | index ed1e25140..9d32ec6d9 100644 | ||
22 | --- a/rpmio/rpmio.c | ||
23 | +++ b/rpmio/rpmio.c | ||
24 | @@ -798,6 +798,7 @@ static LZFILE *lzopen_internal(const char *mode, int fd, int xz) | ||
25 | * should've processed | ||
26 | * */ | ||
27 | while (isdigit(*++mode)); | ||
28 | + --mode; | ||
29 | } | ||
30 | #ifdef HAVE_LZMA_MT | ||
31 | else | ||
32 | -- | ||
33 | 2.25.1 | ||
34 | |||
diff --git a/meta/recipes-devtools/rpm/rpm_4.14.2.1.bb b/meta/recipes-devtools/rpm/rpm_4.14.2.1.bb index c39a5208e5..376021d913 100644 --- a/meta/recipes-devtools/rpm/rpm_4.14.2.1.bb +++ b/meta/recipes-devtools/rpm/rpm_4.14.2.1.bb | |||
@@ -44,6 +44,7 @@ SRC_URI = "git://github.com/rpm-software-management/rpm;branch=rpm-4.14.x;protoc | |||
44 | file://0001-mono-find-provides-requires-do-not-use-monodis-from-.patch \ | 44 | file://0001-mono-find-provides-requires-do-not-use-monodis-from-.patch \ |
45 | file://0001-Rip-out-partial-support-for-unused-MD2-and-RIPEMD160.patch \ | 45 | file://0001-Rip-out-partial-support-for-unused-MD2-and-RIPEMD160.patch \ |
46 | file://0001-rpmplugins.c-call-dlerror-prior-to-dlsym.patch \ | 46 | file://0001-rpmplugins.c-call-dlerror-prior-to-dlsym.patch \ |
47 | file://0001-rpmio-Fix-lzopen_internal-mode-parsing-when-Tn-is-us.patch \ | ||
47 | file://CVE-2021-3421.patch \ | 48 | file://CVE-2021-3421.patch \ |
48 | file://CVE-2021-20266.patch \ | 49 | file://CVE-2021-20266.patch \ |
49 | " | 50 | " |