From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3C7F5EB64DC for ; Mon, 17 Jul 2023 13:45:07 +0000 (UTC) Received: from relay5-d.mail.gandi.net (relay5-d.mail.gandi.net [217.70.183.197]) by mx.groups.io with SMTP id smtpd.web11.8546.1689601498600012681 for ; Mon, 17 Jul 2023 06:44:59 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@bootlin.com header.s=gm1 header.b=avUv1X01; spf=pass (domain: bootlin.com, ip: 217.70.183.197, mailfrom: alexandre.belloni@bootlin.com) Received: by mail.gandi.net (Postfix) with ESMTPSA id 878A41C0013; Mon, 17 Jul 2023 13:44:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1689601496; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=eEksmaHQHZIfRjTMcq8FB0+khWSghoMp8e2geUZEd/w=; b=avUv1X01jTNhHwut8h3zXmiBmy+M9lBxz5mYIhU5XgXHpsk4z63ZLKAv+d2cMgBNv9i28r 2QfuPfcX/iqhqnfyJMCbxlBD6gBUa3vYXpQYg0BUHQyAAo5W6mPUllyNJxrdNSJs6t9ak8 wg+OLNrAc/1b/Bc46c4R8qmDdv/Wm53UnNBCDQLC5/V6yKXfY5PeyanI1JNCe01v9zjncG bQLSpTf+IxWPkuxlVBM0YRmTm9IX821iuavIJlrYwjDzy+4l4lFi2YPm9tw6jXR48NrxSX HMCOuqg76o+9I/PP8OGRrE/A/XkeAp3IXtvTE3Ohj4VCNtMNlYXq/EFTK+dxaA== Date: Mon, 17 Jul 2023 15:44:55 +0200 From: Alexandre Belloni To: soumya.sambu@windriver.com Cc: openembedded-core@lists.openembedded.org, steve@sakoman.com, Hari.GPillai@windriver.com Subject: Re: [oe-core][PATCH 1/1] perl: Fix CVE-2023-31486 Message-ID: <202307171344550a4050ec@mail.local> References: <20230714032510.1727852-1-soumya.sambu@windriver.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230714032510.1727852-1-soumya.sambu@windriver.com> X-GND-Sasl: alexandre.belloni@bootlin.com List-Id: X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Mon, 17 Jul 2023 13:45:07 -0000 X-Groupsio-URL: https://lists.openembedded.org/g/openembedded-core/message/184466 Hello, you pressed y instead of enter when git asked you what wharset to use, so the patch doesn't apply. Can you resend? On 14/07/2023 03:25:10+0000, Soumya via lists.openembedded.org wrote: > HTTP::Tiny before 0.083, a Perl core module since 5.13.9 and available > standalone on CPAN, has an insecure default TLS configuration where > users must opt in to verify certificates. > > References: > https://nvd.nist.gov/vuln/detail/CVE-2023-31486 > > Upstream patches: > https://github.com/chansen/p5-http-tiny/commit/77f557ef84698efeb6eed04e4a9704eaf85b741d > https://github.com/chansen/p5-http-tiny/commit/a22785783b17cbaa28afaee4a024d81a1903701d > > Signed-off-by: Soumya > --- > .../perl/files/CVE-2023-31486-0001.patch | 217 ++++++++++++++++++ > .../perl/files/CVE-2023-31486-0002.patch | 36 +++ > meta/recipes-devtools/perl/perl_5.36.1.bb | 2 + > 3 files changed, 255 insertions(+) > create mode 100644 meta/recipes-devtools/perl/files/CVE-2023-31486-0001.patch > create mode 100644 meta/recipes-devtools/perl/files/CVE-2023-31486-0002.patch > > diff --git a/meta/recipes-devtools/perl/files/CVE-2023-31486-0001.patch b/meta/recipes-devtools/perl/files/CVE-2023-31486-0001.patch > new file mode 100644 > index 0000000000..1074e0848d > --- /dev/null > +++ b/meta/recipes-devtools/perl/files/CVE-2023-31486-0001.patch > @@ -0,0 +1,217 @@ > +From 77f557ef84698efeb6eed04e4a9704eaf85b741d > +From: Stig Palmquist > +Date: Mon Jun 5 16:46:22 2023 +0200 > +Subject: [PATCH] Change verify_SSL default to 1, add ENV var to enable > + insecure default > + > +- Changes the `verify_SSL` default parameter from `0` to `1` > + > + Based on patch by Dominic Hargreaves: > + https://salsa.debian.org/perl-team/interpreter/perl/-/commit/1490431e40e22052f75a0b3449f1f53cbd27ba92 > + > + CVE: CVE-2023-31486 > + > +- Add check for `$ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT}` that > + enables the previous insecure default behaviour if set to `1`. > + > + This provides a workaround for users who encounter problems with the > + new `verify_SSL` default. > + > + Example to disable certificate checks: > + ``` > + $ PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1 ./script.pl > + ``` > + > +- Updates to documentation: > + - Describe changing the verify_SSL value > + - Describe the escape-hatch environment variable > + - Remove rationale for not enabling verify_SSL > + - Add missing certificate search paths > + - Replace "SSL" with "TLS/SSL" where appropriate > + - Use "machine-in-the-middle" instead of "man-in-the-middle" > + > +Upstream-Status: Backport [https://github.com/chansen/p5-http-tiny/commit/77f557ef84698efeb6eed04e4a9704eaf85b741d] > + > +Signed-off-by: Soumya > +--- > + cpan/HTTP-Tiny/lib/HTTP/Tiny.pm | 86 ++++++++++++++++++++++----------- > + 1 file changed, 57 insertions(+), 29 deletions(-) > + > +diff --git a/cpan/HTTP-Tiny/lib/HTTP/Tiny.pm b/cpan/HTTP-Tiny/lib/HTTP/Tiny.pm > +index 83ca06d..ebc34a1 100644 > +--- a/cpan/HTTP-Tiny/lib/HTTP/Tiny.pm > ++++ b/cpan/HTTP-Tiny/lib/HTTP/Tiny.pm > +@@ -40,10 +40,14 @@ sub _croak { require Carp; Carp::croak(@_) } > + #pod * C — Request timeout in seconds (default is 60) If a socket open, > + #pod read or write takes longer than the timeout, the request response status code > + #pod will be 599. > +-#pod * C — A boolean that indicates whether to validate the SSL > +-#pod certificate of an C — connection (default is false) > ++#pod * C — A boolean that indicates whether to validate the TLS/SSL > ++#pod certificate of an C — connection (default is true). Changed from false > ++#pod to true in version 0.083. > + #pod * C — A hashref of C — options to pass through to > + #pod L > ++#pod * C<$ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT}> - Changes the default > ++#pod certificate verification behavior to not check server identity if set to 1. > ++#pod Only effective if C is not set. Added in version 0.083. > + #pod > + #pod An accessor/mutator method exists for each attribute. > + #pod > +@@ -111,11 +115,17 @@ sub timeout { > + sub new { > + my($class, %args) = @_; > + > ++ # Support lower case verify_ssl argument, but only if verify_SSL is not > ++ # true. > ++ if ( exists $args{verify_ssl} ) { > ++ $args{verify_SSL} ||= $args{verify_ssl}; > ++ } > ++ > + my $self = { > + max_redirect => 5, > + timeout => defined $args{timeout} ? $args{timeout} : 60, > + keep_alive => 1, > +- verify_SSL => $args{verify_SSL} || $args{verify_ssl} || 0, # no verification by default > ++ verify_SSL => defined $args{verify_SSL} ? $args{verify_SSL} : _verify_SSL_default(), > + no_proxy => $ENV{no_proxy}, > + }; > + > +@@ -134,6 +144,13 @@ sub new { > + return $self; > + } > + > ++sub _verify_SSL_default { > ++ my ($self) = @_; > ++ # Check if insecure default certificate verification behaviour has been > ++ # changed by the user by setting PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1 > ++ return (($ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} || '') eq '1') ? 0 : 1; > ++} > ++ > + sub _set_proxies { > + my ($self) = @_; > + > +@@ -1055,7 +1072,7 @@ sub new { > + timeout => 60, > + max_line_size => 16384, > + max_header_lines => 64, > +- verify_SSL => 0, > ++ verify_SSL => HTTP::Tiny::_verify_SSL_default(), > + SSL_options => {}, > + %args > + }, $class; > +@@ -2043,11 +2060,11 @@ proxy > + timeout > + verify_SSL > + > +-=head1 SSL SUPPORT > ++=head1 TLS/SSL SUPPORT > + > + Direct C connections are supported only if L 1.56 or > + greater and L 1.49 or greater are installed. An error will occur > +-if new enough versions of these modules are not installed or if the SSL > ++if new enough versions of these modules are not installed or if the TLS > + encryption fails. You can also use C utility function > + that returns boolean to see if the required modules are installed. > + > +@@ -2055,7 +2072,7 @@ An C connection may be made via an C proxy that supports the CONNEC > + command (i.e. RFC 2817). You may not proxy C via a proxy that itself > + requires C to communicate. > + > +-SSL provides two distinct capabilities: > ++TLS/SSL provides two distinct capabilities: > + > + =over 4 > + > +@@ -2069,24 +2086,17 @@ Verification of server identity > + > + =back > + > +-B. > +- > +-Server identity verification is controversial and potentially tricky because it > +-depends on a (usually paid) third-party Certificate Authority (CA) trust model > +-to validate a certificate as legitimate. This discriminates against servers > +-with self-signed certificates or certificates signed by free, community-driven > +-CA's such as L. > ++B. > + > +-By default, HTTP::Tiny does not make any assumptions about your trust model, > +-threat level or risk tolerance. It just aims to give you an encrypted channel > +-when you need one. > ++This was changed in version 0.083 due to security concerns. The previous default > ++behavior can be enabled by setting C<$ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT}> > ++to 1. > + > +-Setting the C attribute to a true value will make HTTP::Tiny verify > +-that an SSL connection has a valid SSL certificate corresponding to the host > +-name of the connection and that the SSL certificate has been verified by a CA. > +-Assuming you trust the CA, this will protect against a L +-attack|http://en.wikipedia.org/wiki/Man-in-the-middle_attack>. If you are > +-concerned about security, you should enable this option. > ++Verification is done by checking that that the TLS/SSL connection has a valid > ++certificate corresponding to the host name of the connection and that the > ++certificate has been verified by a CA. Assuming you trust the CA, this will > ++protect against L ++attacks|http://en.wikipedia.org/wiki/Machine-in-the-middle_attack>. > + > + Certificate verification requires a file containing trusted CA certificates. > + > +@@ -2094,9 +2104,7 @@ If the environment variable C is present, HTTP::Tiny > + will try to find a CA certificate file in that location. > + > + If the L module is installed, HTTP::Tiny will use the CA file > +-included with it as a source of trusted CA's. (This means you trust Mozilla, > +-the author of Mozilla::CA, the CPAN mirror where you got Mozilla::CA, the > +-toolchain used to install it, and your operating system security, right?) > ++included with it as a source of trusted CA's. > + > + If that module is not available, then HTTP::Tiny will search several > + system-specific default locations for a CA certificate file: > +@@ -2115,13 +2123,33 @@ system-specific default locations for a CA certificate file: > + > + /etc/ssl/ca-bundle.pem > + > ++=item * > ++ > ++/etc/openssl/certs/ca-certificates.crt > ++ > ++=item * > ++ > ++/etc/ssl/cert.pem > ++ > ++=item * > ++ > ++/usr/local/share/certs/ca-root-nss.crt > ++ > ++=item * > ++ > ++/etc/pki/tls/cacert.pem > ++ > ++=item * > ++ > ++/etc/certs/ca-certificates.crt > ++ > + =back > + > + An error will be occur if C is true and no CA certificate file > + is available. > + > +-If you desire complete control over SSL connections, the C attribute > +-lets you provide a hash reference that will be passed through to > ++If you desire complete control over TLS/SSL connections, the C > ++attribute lets you provide a hash reference that will be passed through to > + C, overriding any options set by HTTP::Tiny. For > + example, to provide your own trusted CA file: > + > +@@ -2131,7 +2159,7 @@ example, to provide your own trusted CA file: > + > + The C attribute could also be used for such things as providing a > + client certificate for authentication to a server or controlling the choice of > +-cipher used for the SSL connection. See L documentation for > ++cipher used for the TLS/SSL connection. See L documentation for > + details. > + > + =head1 PROXY SUPPORT > +-- > +2.40.0 > diff --git a/meta/recipes-devtools/perl/files/CVE-2023-31486-0002.patch b/meta/recipes-devtools/perl/files/CVE-2023-31486-0002.patch > new file mode 100644 > index 0000000000..45452be389 > --- /dev/null > +++ b/meta/recipes-devtools/perl/files/CVE-2023-31486-0002.patch > @@ -0,0 +1,36 @@ > +From a22785783b17cbaa28afaee4a024d81a1903701d > +From: Stig Palmquist > +Date: Sun Jun 18 11:36:05 2023 +0200 > +Subject: [PATCH] Fix incorrect env var name for verify_SSL default > + > +The variable to override the verify_SSL default differed slightly in the > +documentation from what was checked for in the code. > + > +This commit makes the code use `PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT` > +as documented, instead of `PERL_HTTP_TINY_INSECURE_BY_DEFAULT` which was > +missing `SSL_` > + > +CVE: CVE-2023-31486 > + > +Upstream-Status: Backport [https://github.com/chansen/p5-http-tiny/commit/a22785783b17cbaa28afaee4a024d81a1903701d] > + > +Signed-off-by: Soumya > +--- > + cpan/HTTP-Tiny/lib/HTTP/Tiny.pm | 2 +- > + 1 file changed, 1 insertion(+), 1 deletion(-) > + > +diff --git a/cpan/HTTP-Tiny/lib/HTTP/Tiny.pm b/cpan/HTTP-Tiny/lib/HTTP/Tiny.pm > +index ebc34a1..65ac8ff 100644 > +--- a/cpan/HTTP-Tiny/lib/HTTP/Tiny.pm > ++++ b/cpan/HTTP-Tiny/lib/HTTP/Tiny.pm > +@@ -148,7 +148,7 @@ sub _verify_SSL_default { > + my ($self) = @_; > + # Check if insecure default certificate verification behaviour has been > + # changed by the user by setting PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT=1 > +- return (($ENV{PERL_HTTP_TINY_INSECURE_BY_DEFAULT} || '') eq '1') ? 0 : 1; > ++ return (($ENV{PERL_HTTP_TINY_SSL_INSECURE_BY_DEFAULT} || '') eq '1') ? 0 : 1; > + } > + > + sub _set_proxies { > +-- > +2.40.0 > diff --git a/meta/recipes-devtools/perl/perl_5.36.1.bb b/meta/recipes-devtools/perl/perl_5.36.1.bb > index 3db1d9c6ae..87768cc7f7 100644 > --- a/meta/recipes-devtools/perl/perl_5.36.1.bb > +++ b/meta/recipes-devtools/perl/perl_5.36.1.bb > @@ -18,6 +18,8 @@ SRC_URI = "https://www.cpan.org/src/5.0/perl-${PV}.tar.gz;name=perl \ > file://determinism.patch \ > file://0001-cpan-Sys-Syslog-Makefile.PL-Fix-_PATH_LOG-for-determ.patch \ > file://CVE-2023-31484.patch \ > + file://CVE-2023-31486-0001.patch \ > + file://CVE-2023-31486-0002.patch \ > " > SRC_URI:append:class-native = " \ > file://perl-configpm-switch.patch \ > -- > 2.40.0 > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#184250): https://lists.openembedded.org/g/openembedded-core/message/184250 > Mute This Topic: https://lists.openembedded.org/mt/100134685/3617179 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com] > -=-=-=-=-=-=-=-=-=-=-=- > -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com