diff options
author | Armin Kuster <akuster@mvista.com> | 2016-09-23 23:11:28 -0700 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-09-27 09:05:57 +0100 |
commit | 8e5e92193a0924d7019caaaa7c7afaec7c773726 (patch) | |
tree | 2445832017f6d4d59417370baff5f520f56ed4b8 /meta/recipes-connectivity | |
parent | 06ed5c5a1024131d2ff15394dd6fcaf55b599bbc (diff) | |
download | poky-8e5e92193a0924d7019caaaa7c7afaec7c773726.tar.gz |
openssl: Security fix CVE-2016-2182
affects openssl < 1.0.1i
(From OE-Core rev: 4be4162d5a03af6a20adc2314575e4d0baa5337a)
Signed-off-by: Armin Kuster <akuster@mvista.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-connectivity')
-rw-r--r-- | meta/recipes-connectivity/openssl/openssl/CVE-2016-2182.patch | 70 | ||||
-rw-r--r-- | meta/recipes-connectivity/openssl/openssl_1.0.2h.bb | 1 |
2 files changed, 71 insertions, 0 deletions
diff --git a/meta/recipes-connectivity/openssl/openssl/CVE-2016-2182.patch b/meta/recipes-connectivity/openssl/openssl/CVE-2016-2182.patch new file mode 100644 index 0000000000..5995cbea18 --- /dev/null +++ b/meta/recipes-connectivity/openssl/openssl/CVE-2016-2182.patch | |||
@@ -0,0 +1,70 @@ | |||
1 | From e36f27ddb80a48e579783bc29fb3758988342b71 Mon Sep 17 00:00:00 2001 | ||
2 | From: "Dr. Stephen Henson" <steve@openssl.org> | ||
3 | Date: Fri, 5 Aug 2016 14:26:03 +0100 | ||
4 | Subject: [PATCH] Check for errors in BN_bn2dec() | ||
5 | |||
6 | If an oversize BIGNUM is presented to BN_bn2dec() it can cause | ||
7 | BN_div_word() to fail and not reduce the value of 't' resulting | ||
8 | in OOB writes to the bn_data buffer and eventually crashing. | ||
9 | |||
10 | Fix by checking return value of BN_div_word() and checking writes | ||
11 | don't overflow buffer. | ||
12 | |||
13 | Thanks to Shi Lei for reporting this bug. | ||
14 | |||
15 | CVE-2016-2182 | ||
16 | |||
17 | Reviewed-by: Tim Hudson <tjh@openssl.org> | ||
18 | (cherry picked from commit 07bed46f332fce8c1d157689a2cdf915a982ae34) | ||
19 | |||
20 | Conflicts: | ||
21 | crypto/bn/bn_print.c | ||
22 | |||
23 | Upstream-Status: Backport | ||
24 | CVE: CVE-2016-2182 | ||
25 | Signed-off-by: Armin Kuster <akuster@mvista.com> | ||
26 | |||
27 | --- | ||
28 | crypto/bn/bn_print.c | 11 ++++++++--- | ||
29 | 1 file changed, 8 insertions(+), 3 deletions(-) | ||
30 | |||
31 | diff --git a/crypto/bn/bn_print.c b/crypto/bn/bn_print.c | ||
32 | index bfa31ef..b44403e 100644 | ||
33 | --- a/crypto/bn/bn_print.c | ||
34 | +++ b/crypto/bn/bn_print.c | ||
35 | @@ -111,6 +111,7 @@ char *BN_bn2dec(const BIGNUM *a) | ||
36 | char *p; | ||
37 | BIGNUM *t = NULL; | ||
38 | BN_ULONG *bn_data = NULL, *lp; | ||
39 | + int bn_data_num; | ||
40 | |||
41 | /*- | ||
42 | * get an upper bound for the length of the decimal integer | ||
43 | @@ -120,9 +121,9 @@ char *BN_bn2dec(const BIGNUM *a) | ||
44 | */ | ||
45 | i = BN_num_bits(a) * 3; | ||
46 | num = (i / 10 + i / 1000 + 1) + 1; | ||
47 | - bn_data = | ||
48 | - (BN_ULONG *)OPENSSL_malloc((num / BN_DEC_NUM + 1) * sizeof(BN_ULONG)); | ||
49 | - buf = (char *)OPENSSL_malloc(num + 3); | ||
50 | + bn_data_num = num / BN_DEC_NUM + 1; | ||
51 | + bn_data = OPENSSL_malloc(bn_data_num * sizeof(BN_ULONG)); | ||
52 | + buf = OPENSSL_malloc(num + 3); | ||
53 | if ((buf == NULL) || (bn_data == NULL)) { | ||
54 | BNerr(BN_F_BN_BN2DEC, ERR_R_MALLOC_FAILURE); | ||
55 | goto err; | ||
56 | @@ -143,7 +144,11 @@ char *BN_bn2dec(const BIGNUM *a) | ||
57 | i = 0; | ||
58 | while (!BN_is_zero(t)) { | ||
59 | *lp = BN_div_word(t, BN_DEC_CONV); | ||
60 | + if (*lp == (BN_ULONG)-1) | ||
61 | + goto err; | ||
62 | lp++; | ||
63 | + if (lp - bn_data >= bn_data_num) | ||
64 | + goto err; | ||
65 | } | ||
66 | lp--; | ||
67 | /* | ||
68 | -- | ||
69 | 2.7.4 | ||
70 | |||
diff --git a/meta/recipes-connectivity/openssl/openssl_1.0.2h.bb b/meta/recipes-connectivity/openssl/openssl_1.0.2h.bb index 4b0ad7e737..3d6474b1f7 100644 --- a/meta/recipes-connectivity/openssl/openssl_1.0.2h.bb +++ b/meta/recipes-connectivity/openssl/openssl_1.0.2h.bb | |||
@@ -44,6 +44,7 @@ SRC_URI += "file://find.pl;subdir=${BP}/util/ \ | |||
44 | file://CVE-2016-2181_p1.patch \ | 44 | file://CVE-2016-2181_p1.patch \ |
45 | file://CVE-2016-2181_p2.patch \ | 45 | file://CVE-2016-2181_p2.patch \ |
46 | file://CVE-2016-2181_p3.patch \ | 46 | file://CVE-2016-2181_p3.patch \ |
47 | file://CVE-2016-2182.patch \ | ||
47 | " | 48 | " |
48 | SRC_URI[md5sum] = "9392e65072ce4b614c1392eefc1f23d0" | 49 | SRC_URI[md5sum] = "9392e65072ce4b614c1392eefc1f23d0" |
49 | SRC_URI[sha256sum] = "1d4007e53aad94a5b2002fe045ee7bb0b3d98f1a47f8b2bc851dcd1c74332919" | 50 | SRC_URI[sha256sum] = "1d4007e53aad94a5b2002fe045ee7bb0b3d98f1a47f8b2bc851dcd1c74332919" |