diff mbox series

[kirkstone] libarchive: CVE-2022-36227 NULL pointer dereference in archive_write.c

Message ID 20221202061812.62788-1-hprajapati@mvista.com
State Accepted, archived
Commit e723e791b4faa3d5c755b4669b0901ebd0c368c9
Headers show
Series [kirkstone] libarchive: CVE-2022-36227 NULL pointer dereference in archive_write.c | expand

Commit Message

Hitendra Prajapati Dec. 2, 2022, 6:18 a.m. UTC
Upstream-Status: Backport from https://github.com/libarchive/libarchive/commit/bff38efe8c110469c5080d387bec62a6ca15b1a5

Signed-off-by: Hitendra Prajapati <hprajapati@mvista.com>
---
 .../libarchive/CVE-2022-36227.patch           | 41 +++++++++++++++++++
 .../libarchive/libarchive_3.6.1.bb            |  4 +-
 2 files changed, 44 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-extended/libarchive/libarchive/CVE-2022-36227.patch

Comments

Martin Jansa Dec. 2, 2022, 7:49 a.m. UTC | #1
>
>
> +From afa85b75607649f3a89cb4d17cf3f003738d3576 Mon Sep 17 00:00:00 2001
> +From: Hitendra Prajapati <hprajapati@mvista.com>
> +Date: Fri, 2 Dec 2022 11:22:39 +0530
> +Subject: [PATCH] CVE-2022-36227
> +
> +Upstream-Status: Backport [
> https://github.com/libarchive/libarchive/commit/bff38efe8c110469c5080d387bec62a6ca15b1a5
> ]
> +CVE: CVE-2022-36227
>

Please don't change the Author of the patch when backporting something, the
original change says:

From bff38efe8c110469c5080d387bec62a6ca15b1a5 Mon Sep 17 00:00:00 2001
From: obiwac <obiwac@gmail.com>
Date: Fri, 22 Jul 2022 22:41:10 +0200
Subject: [PATCH] libarchive: Handle a `calloc` returning NULL (fixes #1754)

And you should keep it.

Also please check your e-mail, my previous 2 reply were returned with 550
(hopefully you might see them in the ML:
https://lists.openembedded.org/g/openembedded-core/message/174225
https://lists.openembedded.org/g/openembedded-core/message/174224
that might explain why you haven't noticed the answer from Steve).

 ** Address not found **

Your message wasn't delivered to hpprajapati@mvista.com because the address=
 couldn't be found, or is unable to receive mail.

Learn more here: https://support.google.com/mail/?p=3DNoSuchUser

The response was:

550 5.1.1 The email account that you tried to reach does not exist. Please =
try double-checking the recipient's email address for typos or unnecessary =
spaces. Learn more at https://support.google.com/mail/?p=3DNoSuchUser u6-20=
020adfc646000000b0024187597f81sor4016963wrg.5 - gsmtp
Hitendra Prajapati Dec. 2, 2022, 8:49 a.m. UTC | #2
Hi Martin/Team,

Yes, I missed that part.

It is my mistake. I'm extremely sorry for that.

I'll correct the patch and resubmit.

Regards,
Hitendra Prajapati

On Fri, 2 Dec 2022, 1:19 pm Martin Jansa, <martin.jansa@gmail.com> wrote:

>
>> +From afa85b75607649f3a89cb4d17cf3f003738d3576 Mon Sep 17 00:00:00 2001
>> +From: Hitendra Prajapati <hprajapati@mvista.com>
>> +Date: Fri, 2 Dec 2022 11:22:39 +0530
>> +Subject: [PATCH] CVE-2022-36227
>> +
>> +Upstream-Status: Backport [
>> https://github.com/libarchive/libarchive/commit/bff38efe8c110469c5080d387bec62a6ca15b1a5
>> ]
>> +CVE: CVE-2022-36227
>>
>
> Please don't change the Author of the patch when backporting something,
> the original change says:
>
> From bff38efe8c110469c5080d387bec62a6ca15b1a5 Mon Sep 17 00:00:00 2001
> From: obiwac <obiwac@gmail.com>
> Date: Fri, 22 Jul 2022 22:41:10 +0200
> Subject: [PATCH] libarchive: Handle a `calloc` returning NULL (fixes #1754)
>
> And you should keep it.
>
> Also please check your e-mail, my previous 2 reply were returned with 550
> (hopefully you might see them in the ML:
> https://lists.openembedded.org/g/openembedded-core/message/174225
> https://lists.openembedded.org/g/openembedded-core/message/174224
> that might explain why you haven't noticed the answer from Steve).
>
>  ** Address not found **
>
> Your message wasn't delivered to hpprajapati@mvista.com because the
> address=
>  couldn't be found, or is unable to receive mail.
>
> Learn more here: https://support.google.com/mail/?p=3DNoSuchUser
>
> The response was:
>
> 550 5.1.1 The email account that you tried to reach does not exist. Please
> =
> try double-checking the recipient's email address for typos or unnecessary
> =
> spaces. Learn more at https://support.google.com/mail/?p=3DNoSuchUser
> u6-20=
> 020adfc646000000b0024187597f81sor4016963wrg.5 - gsmtp
>
diff mbox series

Patch

diff --git a/meta/recipes-extended/libarchive/libarchive/CVE-2022-36227.patch b/meta/recipes-extended/libarchive/libarchive/CVE-2022-36227.patch
new file mode 100644
index 0000000000..99242fc826
--- /dev/null
+++ b/meta/recipes-extended/libarchive/libarchive/CVE-2022-36227.patch
@@ -0,0 +1,41 @@ 
+From afa85b75607649f3a89cb4d17cf3f003738d3576 Mon Sep 17 00:00:00 2001
+From: Hitendra Prajapati <hprajapati@mvista.com>
+Date: Fri, 2 Dec 2022 11:22:39 +0530
+Subject: [PATCH] CVE-2022-36227
+
+Upstream-Status: Backport [https://github.com/libarchive/libarchive/commit/bff38efe8c110469c5080d387bec62a6ca15b1a5]
+CVE: CVE-2022-36227
+Signed-off-by: Hitendra Prajapati <hprajapati@mvista.com>
+---
+ libarchive/archive_write.c | 8 ++++++++
+ 1 file changed, 8 insertions(+)
+
+diff --git a/libarchive/archive_write.c b/libarchive/archive_write.c
+index 66592e8..27626b5 100644
+--- a/libarchive/archive_write.c
++++ b/libarchive/archive_write.c
+@@ -201,6 +201,10 @@ __archive_write_allocate_filter(struct archive *_a)
+ 	struct archive_write_filter *f;
+ 
+ 	f = calloc(1, sizeof(*f));
++
++	if (f == NULL)
++		return (NULL);
++
+ 	f->archive = _a;
+ 	f->state = ARCHIVE_WRITE_FILTER_STATE_NEW;
+ 	if (a->filter_first == NULL)
+@@ -548,6 +552,10 @@ archive_write_open2(struct archive *_a, void *client_data,
+ 	a->client_data = client_data;
+ 
+ 	client_filter = __archive_write_allocate_filter(_a);
++
++	if (client_filter == NULL)
++		return (ARCHIVE_FATAL);
++
+ 	client_filter->open = archive_write_client_open;
+ 	client_filter->write = archive_write_client_write;
+ 	client_filter->close = archive_write_client_close;
+-- 
+2.25.1
+
diff --git a/meta/recipes-extended/libarchive/libarchive_3.6.1.bb b/meta/recipes-extended/libarchive/libarchive_3.6.1.bb
index c795b41628..df9df5e0a6 100644
--- a/meta/recipes-extended/libarchive/libarchive_3.6.1.bb
+++ b/meta/recipes-extended/libarchive/libarchive_3.6.1.bb
@@ -32,7 +32,9 @@  PACKAGECONFIG[zstd] = "--with-zstd,--without-zstd,zstd,"
 
 EXTRA_OECONF += "--enable-largefile"
 
-SRC_URI = "http://libarchive.org/downloads/libarchive-${PV}.tar.gz"
+SRC_URI = "http://libarchive.org/downloads/libarchive-${PV}.tar.gz \
+           file://CVE-2022-36227.patch \
+	   "
 UPSTREAM_CHECK_URI = "http://libarchive.org/"
 
 SRC_URI[sha256sum] = "c676146577d989189940f1959d9e3980d28513d74eedfbc6b7f15ea45fe54ee2"