* [PATCH] openssl: fix for CVE-2010-5298
@ 2014-05-19 7:00 rongqing.li
2014-05-22 1:42 ` Douglas Royds
0 siblings, 1 reply; 3+ messages in thread
From: rongqing.li @ 2014-05-19 7:00 UTC (permalink / raw)
To: openembedded-core
From: Yue Tao <Yue.Tao@windriver.com>
Race condition in the ssl3_read_bytes function in s3_pkt.c in OpenSSL
through 1.0.1g, when SSL_MODE_RELEASE_BUFFERS is enabled, allows remote
attackers to inject data across sessions or cause a denial of service
(use-after-free and parsing error) via an SSL connection in a
multithreaded environment.
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2010-5298
Signed-off-by: Yue Tao <Yue.Tao@windriver.com>
Signed-off-by: Roy Li <rongqing.li@windriver.com>
---
.../openssl-1.0.1e/openssl-CVE-2010-5298.patch | 24 ++++++++++++++++++++
1 file changed, 24 insertions(+)
create mode 100644 meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
diff --git a/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch b/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
new file mode 100644
index 0000000..417a774
--- /dev/null
+++ b/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
@@ -0,0 +1,24 @@
+openssl fix for CVE-2010-5298
+
+Upstream-Status: Backport
+
+Race condition in the ssl3_read_bytes function in s3_pkt.c in OpenSSL
+through 1.0.1g, when SSL_MODE_RELEASE_BUFFERS is enabled, allows remote
+attackers to inject data across sessions or cause a denial of service
+(use-after-free and parsing error) via an SSL connection in a
+multithreaded environment.
+
+http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2010-5298
+
+Signed-off-by: Yue Tao <Yue.Tao@windriver.com>
+--- a/ssl/s3_pkt.c
++++ b/ssl/s3_pkt.c
+@@ -1013,7 +1013,7 @@ start:
+ {
+ s->rstate=SSL_ST_READ_HEADER;
+ rr->off=0;
+- if (s->mode & SSL_MODE_RELEASE_BUFFERS)
++ if (s->mode & SSL_MODE_RELEASE_BUFFERS && s->s3->rbuf.left == 0)
+ ssl3_release_read_buffer(s);
+ }
+ }
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] openssl: fix for CVE-2010-5298
2014-05-19 7:00 [PATCH] openssl: fix for CVE-2010-5298 rongqing.li
@ 2014-05-22 1:42 ` Douglas Royds
2014-05-22 5:24 ` Rongqing Li
0 siblings, 1 reply; 3+ messages in thread
From: Douglas Royds @ 2014-05-22 1:42 UTC (permalink / raw)
To: openembedded-core
On 19/05/14 19:00, rongqing.li@windriver.com wrote:
> From: Yue Tao <Yue.Tao@windriver.com>
>
> Race condition in the ssl3_read_bytes function in s3_pkt.c in OpenSSL
> through 1.0.1g, when SSL_MODE_RELEASE_BUFFERS is enabled, allows remote
> attackers to inject data across sessions or cause a denial of service
> (use-after-free and parsing error) via an SSL connection in a
> multithreaded environment.
>
> http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2010-5298
>
> Signed-off-by: Yue Tao <Yue.Tao@windriver.com>
> Signed-off-by: Roy Li <rongqing.li@windriver.com>
> ---
> .../openssl-1.0.1e/openssl-CVE-2010-5298.patch | 24 ++++++++++++++++++++
> 1 file changed, 24 insertions(+)
> create mode 100644 meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
>
> diff --git a/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch b/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
> new file mode 100644
> index 0000000..417a774
> --- /dev/null
> +++ b/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
> @@ -0,0 +1,24 @@
> +openssl fix for CVE-2010-5298
> +
> +Upstream-Status: Backport
> +
> +Race condition in the ssl3_read_bytes function in s3_pkt.c in OpenSSL
> +through 1.0.1g, when SSL_MODE_RELEASE_BUFFERS is enabled, allows remote
> +attackers to inject data across sessions or cause a denial of service
> +(use-after-free and parsing error) via an SSL connection in a
> +multithreaded environment.
> +
> +http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2010-5298
> +
> +Signed-off-by: Yue Tao <Yue.Tao@windriver.com>
> +--- a/ssl/s3_pkt.c
> ++++ b/ssl/s3_pkt.c
> +@@ -1013,7 +1013,7 @@ start:
> + {
> + s->rstate=SSL_ST_READ_HEADER;
> + rr->off=0;
> +- if (s->mode & SSL_MODE_RELEASE_BUFFERS)
> ++ if (s->mode & SSL_MODE_RELEASE_BUFFERS && s->s3->rbuf.left == 0)
> + ssl3_release_read_buffer(s);
> + }
> + }
This patch file appears to be orphaned in openssl-1.0.1e/, when the
current version is 1.0.1g.
Also, there is no file://openssl-CVE-2010-5298.patch entry in the SRC_URI.
Was this deliberate? Am I missing something?
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] openssl: fix for CVE-2010-5298
2014-05-22 1:42 ` Douglas Royds
@ 2014-05-22 5:24 ` Rongqing Li
0 siblings, 0 replies; 3+ messages in thread
From: Rongqing Li @ 2014-05-22 5:24 UTC (permalink / raw)
To: Douglas Royds; +Cc: openembedded-core
On 05/22/2014 09:42 AM, Douglas Royds wrote:
> On 19/05/14 19:00, rongqing.li@windriver.com wrote:
>> From: Yue Tao <Yue.Tao@windriver.com>
>>
>> Race condition in the ssl3_read_bytes function in s3_pkt.c in OpenSSL
>> through 1.0.1g, when SSL_MODE_RELEASE_BUFFERS is enabled, allows remote
>> attackers to inject data across sessions or cause a denial of service
>> (use-after-free and parsing error) via an SSL connection in a
>> multithreaded environment.
>>
>> http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2010-5298
>>
>> Signed-off-by: Yue Tao <Yue.Tao@windriver.com>
>> Signed-off-by: Roy Li <rongqing.li@windriver.com>
>> ---
>> .../openssl-1.0.1e/openssl-CVE-2010-5298.patch | 24
>> ++++++++++++++++++++
>> 1 file changed, 24 insertions(+)
>> create mode 100644
>> meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
>>
>>
>> diff --git
>> a/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
>> b/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
>>
>> new file mode 100644
>> index 0000000..417a774
>> --- /dev/null
>> +++
>> b/meta/recipes-connectivity/openssl/openssl-1.0.1e/openssl-CVE-2010-5298.patch
>>
>> @@ -0,0 +1,24 @@
>> +openssl fix for CVE-2010-5298
>> +
>> +Upstream-Status: Backport
>> +
>> +Race condition in the ssl3_read_bytes function in s3_pkt.c in OpenSSL
>> +through 1.0.1g, when SSL_MODE_RELEASE_BUFFERS is enabled, allows remote
>> +attackers to inject data across sessions or cause a denial of service
>> +(use-after-free and parsing error) via an SSL connection in a
>> +multithreaded environment.
>> +
>> +http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2010-5298
>> +
>> +Signed-off-by: Yue Tao <Yue.Tao@windriver.com>
>> +--- a/ssl/s3_pkt.c
>> ++++ b/ssl/s3_pkt.c
>> +@@ -1013,7 +1013,7 @@ start:
>> + {
>> + s->rstate=SSL_ST_READ_HEADER;
>> + rr->off=0;
>> +- if (s->mode & SSL_MODE_RELEASE_BUFFERS)
>> ++ if (s->mode & SSL_MODE_RELEASE_BUFFERS &&
>> s->s3->rbuf.left == 0)
>> + ssl3_release_read_buffer(s);
>> + }
>> + }
>
> This patch file appears to be orphaned in openssl-1.0.1e/, when the
> current version is 1.0.1g.
>
> Also, there is no file://openssl-CVE-2010-5298.patch entry in the SRC_URI.
>
> Was this deliberate? Am I missing something?
thank, I will fix it
-Roy
--
Best Reagrds,
Roy | RongQing Li
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-05-22 5:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-19 7:00 [PATCH] openssl: fix for CVE-2010-5298 rongqing.li
2014-05-22 1:42 ` Douglas Royds
2014-05-22 5:24 ` Rongqing Li
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox