Message ID | 20230123163404.2725145-1-vkumbhar@mvista.com |
---|---|
State | New |
Headers | show |
Series | [langdale] openssl: fix CVE-2022-3996 double locking leads to denial of service | expand |
Hi Vivek, You sent two seemingly identical patches for langdale. Was one perhaps intended for master? Steve On Mon, Jan 23, 2023 at 6:34 AM vkumbhar <vkumbhar@mvista.com> wrote: > > From: Vivek Kumbhar <vkumbhar@mvista.com> > > Signed-off-by: Vivek Kumbhar <vkumbhar@mvista.com> > --- > .../openssl/openssl/CVE-2022-3996.patch | 43 +++++++++++++++++++ > .../openssl/openssl_3.0.7.bb | 1 + > 2 files changed, 44 insertions(+) > create mode 100644 meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch > > diff --git a/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch b/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch > new file mode 100644 > index 0000000000..6d70b323d1 > --- /dev/null > +++ b/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch > @@ -0,0 +1,43 @@ > +From 7725e7bfe6f2ce8146b6552b44e0d226be7638e7 Mon Sep 17 00:00:00 2001 > +From: Pauli <pauli@openssl.org> > +Date: Fri, 11 Nov 2022 09:40:19 +1100 > +Subject: [PATCH] x509: fix double locking problem > + > +This reverts commit 9aa4be691f5c73eb3c68606d824c104550c053f7 and removed the > +redundant flag setting. > + > +Fixes #19643 > + > +Fixes LOW CVE-2022-3996 > + > +Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> > +Reviewed-by: Tomas Mraz <tomas@openssl.org> > +(Merged from https://github.com/openssl/openssl/pull/19652) > + > +(cherry picked from commit 4d0340a6d2f327700a059f0b8f954d6160f8eef5) > + > +Upstream-Status: Backport [https://github.com/openssl/openssl/commit/7725e7bfe6f2ce8146b6552b44e0d226be7638e7] > +CVE: CVE-2022-3996 > +Signed-off-by: Vivek Kumbhar <vkumbhar@mvista.com> > +--- > + crypto/x509/pcy_map.c | 4 ---- > + 1 file changed, 4 deletions(-) > + > +diff --git a/crypto/x509/pcy_map.c b/crypto/x509/pcy_map.c > +index 05406c6493..60dfd1e320 100644 > +--- a/crypto/x509/pcy_map.c > ++++ b/crypto/x509/pcy_map.c > +@@ -73,10 +73,6 @@ int ossl_policy_cache_set_mapping(X509 *x, POLICY_MAPPINGS *maps) > + > + ret = 1; > + bad_mapping: > +- if (ret == -1 && CRYPTO_THREAD_write_lock(x->lock)) { > +- x->ex_flags |= EXFLAG_INVALID_POLICY; > +- CRYPTO_THREAD_unlock(x->lock); > +- } > + sk_POLICY_MAPPING_pop_free(maps, POLICY_MAPPING_free); > + return ret; > + > +-- > +2.30.2 > + > diff --git a/meta/recipes-connectivity/openssl/openssl_3.0.7.bb b/meta/recipes-connectivity/openssl/openssl_3.0.7.bb > index 45fd1de2fd..1842148592 100644 > --- a/meta/recipes-connectivity/openssl/openssl_3.0.7.bb > +++ b/meta/recipes-connectivity/openssl/openssl_3.0.7.bb > @@ -12,6 +12,7 @@ SRC_URI = "http://www.openssl.org/source/openssl-${PV}.tar.gz \ > file://0001-buildinfo-strip-sysroot-and-debug-prefix-map-from-co.patch \ > file://afalg.patch \ > file://0001-Configure-do-not-tweak-mips-cflags.patch \ > + file://CVE-2022-3996.patch \ > " > > SRC_URI:append:class-nativesdk = " \ > -- > 2.30.2 > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#176311): https://lists.openembedded.org/g/openembedded-core/message/176311 > Mute This Topic: https://lists.openembedded.org/mt/96477598/3620601 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [steve@sakoman.com] > -=-=-=-=-=-=-=-=-=-=-=- >
Yes, It was intended for the master. Kind regards, Vivek On Mon, 23 Jan 2023 at 10:19 PM, Steve Sakoman <steve@sakoman.com> wrote: > Hi Vivek, > > You sent two seemingly identical patches for langdale. Was one > perhaps intended for master? > > Steve > > On Mon, Jan 23, 2023 at 6:34 AM vkumbhar <vkumbhar@mvista.com> wrote: > > > > From: Vivek Kumbhar <vkumbhar@mvista.com> > > > > Signed-off-by: Vivek Kumbhar <vkumbhar@mvista.com> > > --- > > .../openssl/openssl/CVE-2022-3996.patch | 43 +++++++++++++++++++ > > .../openssl/openssl_3.0.7.bb | 1 + > > 2 files changed, 44 insertions(+) > > create mode 100644 > meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch > > > > diff --git > a/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch > b/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch > > new file mode 100644 > > index 0000000000..6d70b323d1 > > --- /dev/null > > +++ b/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch > > @@ -0,0 +1,43 @@ > > +From 7725e7bfe6f2ce8146b6552b44e0d226be7638e7 Mon Sep 17 00:00:00 2001 > > +From: Pauli <pauli@openssl.org> > > +Date: Fri, 11 Nov 2022 09:40:19 +1100 > > +Subject: [PATCH] x509: fix double locking problem > > + > > +This reverts commit 9aa4be691f5c73eb3c68606d824c104550c053f7 and > removed the > > +redundant flag setting. > > + > > +Fixes #19643 > > + > > +Fixes LOW CVE-2022-3996 > > + > > +Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> > > +Reviewed-by: Tomas Mraz <tomas@openssl.org> > > +(Merged from https://github.com/openssl/openssl/pull/19652) > > + > > +(cherry picked from commit 4d0340a6d2f327700a059f0b8f954d6160f8eef5) > > + > > +Upstream-Status: Backport [ > https://github.com/openssl/openssl/commit/7725e7bfe6f2ce8146b6552b44e0d226be7638e7 > ] > > +CVE: CVE-2022-3996 > > +Signed-off-by: Vivek Kumbhar <vkumbhar@mvista.com> > > +--- > > + crypto/x509/pcy_map.c | 4 ---- > > + 1 file changed, 4 deletions(-) > > + > > +diff --git a/crypto/x509/pcy_map.c b/crypto/x509/pcy_map.c > > +index 05406c6493..60dfd1e320 100644 > > +--- a/crypto/x509/pcy_map.c > > ++++ b/crypto/x509/pcy_map.c > > +@@ -73,10 +73,6 @@ int ossl_policy_cache_set_mapping(X509 *x, > POLICY_MAPPINGS *maps) > > + > > + ret = 1; > > + bad_mapping: > > +- if (ret == -1 && CRYPTO_THREAD_write_lock(x->lock)) { > > +- x->ex_flags |= EXFLAG_INVALID_POLICY; > > +- CRYPTO_THREAD_unlock(x->lock); > > +- } > > + sk_POLICY_MAPPING_pop_free(maps, POLICY_MAPPING_free); > > + return ret; > > + > > +-- > > +2.30.2 > > + > > diff --git a/meta/recipes-connectivity/openssl/openssl_3.0.7.bb > b/meta/recipes-connectivity/openssl/openssl_3.0.7.bb > > index 45fd1de2fd..1842148592 100644 > > --- a/meta/recipes-connectivity/openssl/openssl_3.0.7.bb > > +++ b/meta/recipes-connectivity/openssl/openssl_3.0.7.bb > > @@ -12,6 +12,7 @@ SRC_URI = " > http://www.openssl.org/source/openssl-${PV}.tar.gz \ > > > file://0001-buildinfo-strip-sysroot-and-debug-prefix-map-from-co.patch \ > > file://afalg.patch \ > > file://0001-Configure-do-not-tweak-mips-cflags.patch \ > > + file://CVE-2022-3996.patch \ > > " > > > > SRC_URI:append:class-nativesdk = " \ > > -- > > 2.30.2 > > > > > > -=-=-=-=-=-=-=-=-=-=-=- > > Links: You receive all messages sent to this group. > > View/Reply Online (#176311): > https://lists.openembedded.org/g/openembedded-core/message/176311 > > Mute This Topic: https://lists.openembedded.org/mt/96477598/3620601 > > Group Owner: openembedded-core+owner@lists.openembedded.org > > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [ > steve@sakoman.com] > > -=-=-=-=-=-=-=-=-=-=-=- > > >
diff --git a/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch b/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch new file mode 100644 index 0000000000..6d70b323d1 --- /dev/null +++ b/meta/recipes-connectivity/openssl/openssl/CVE-2022-3996.patch @@ -0,0 +1,43 @@ +From 7725e7bfe6f2ce8146b6552b44e0d226be7638e7 Mon Sep 17 00:00:00 2001 +From: Pauli <pauli@openssl.org> +Date: Fri, 11 Nov 2022 09:40:19 +1100 +Subject: [PATCH] x509: fix double locking problem + +This reverts commit 9aa4be691f5c73eb3c68606d824c104550c053f7 and removed the +redundant flag setting. + +Fixes #19643 + +Fixes LOW CVE-2022-3996 + +Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> +Reviewed-by: Tomas Mraz <tomas@openssl.org> +(Merged from https://github.com/openssl/openssl/pull/19652) + +(cherry picked from commit 4d0340a6d2f327700a059f0b8f954d6160f8eef5) + +Upstream-Status: Backport [https://github.com/openssl/openssl/commit/7725e7bfe6f2ce8146b6552b44e0d226be7638e7] +CVE: CVE-2022-3996 +Signed-off-by: Vivek Kumbhar <vkumbhar@mvista.com> +--- + crypto/x509/pcy_map.c | 4 ---- + 1 file changed, 4 deletions(-) + +diff --git a/crypto/x509/pcy_map.c b/crypto/x509/pcy_map.c +index 05406c6493..60dfd1e320 100644 +--- a/crypto/x509/pcy_map.c ++++ b/crypto/x509/pcy_map.c +@@ -73,10 +73,6 @@ int ossl_policy_cache_set_mapping(X509 *x, POLICY_MAPPINGS *maps) + + ret = 1; + bad_mapping: +- if (ret == -1 && CRYPTO_THREAD_write_lock(x->lock)) { +- x->ex_flags |= EXFLAG_INVALID_POLICY; +- CRYPTO_THREAD_unlock(x->lock); +- } + sk_POLICY_MAPPING_pop_free(maps, POLICY_MAPPING_free); + return ret; + +-- +2.30.2 + diff --git a/meta/recipes-connectivity/openssl/openssl_3.0.7.bb b/meta/recipes-connectivity/openssl/openssl_3.0.7.bb index 45fd1de2fd..1842148592 100644 --- a/meta/recipes-connectivity/openssl/openssl_3.0.7.bb +++ b/meta/recipes-connectivity/openssl/openssl_3.0.7.bb @@ -12,6 +12,7 @@ SRC_URI = "http://www.openssl.org/source/openssl-${PV}.tar.gz \ file://0001-buildinfo-strip-sysroot-and-debug-prefix-map-from-co.patch \ file://afalg.patch \ file://0001-Configure-do-not-tweak-mips-cflags.patch \ + file://CVE-2022-3996.patch \ " SRC_URI:append:class-nativesdk = " \