diff options
author | Khem Raj <raj.khem@gmail.com> | 2019-02-06 22:25:26 -0800 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-02-08 10:57:19 +0000 |
commit | be3745522dd60f8f63658d8ecebc2dd79209d70b (patch) | |
tree | a83e1a26e5dbc560a349214ad487ca2ce81c7553 /meta/recipes-connectivity | |
parent | c92e780c85523fe860602b947b029d7782afa47a (diff) | |
download | poky-be3745522dd60f8f63658d8ecebc2dd79209d70b.tar.gz |
openssl10: Fix mutliple include assumptions for bn.h in opensslconf.h
After adding #pragma once to wrapper header ( opensslconf.h ) this
latent issue got to bite us, where it expect bn.h to be including
openssl.h to define BN_* defines, which is fragile. This patch removes
the contraints for nested includes for bn.h
(From OE-Core rev: f787b0bb9b0626ddbf2ac94cb206c76716a3773d)
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-connectivity')
-rw-r--r-- | meta/recipes-connectivity/openssl/openssl10/0001-Fix-BN_LLONG-breakage.patch | 33 | ||||
-rw-r--r-- | meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb | 1 |
2 files changed, 34 insertions, 0 deletions
diff --git a/meta/recipes-connectivity/openssl/openssl10/0001-Fix-BN_LLONG-breakage.patch b/meta/recipes-connectivity/openssl/openssl10/0001-Fix-BN_LLONG-breakage.patch new file mode 100644 index 0000000000..13d39c918c --- /dev/null +++ b/meta/recipes-connectivity/openssl/openssl10/0001-Fix-BN_LLONG-breakage.patch | |||
@@ -0,0 +1,33 @@ | |||
1 | From 247b3188cde5f3347091cd54271127386d3aece0 Mon Sep 17 00:00:00 2001 | ||
2 | From: Khem Raj <raj.khem@gmail.com> | ||
3 | Date: Wed, 6 Feb 2019 22:10:33 -0800 | ||
4 | Subject: [PATCH] Fix BN_LLONG breakage | ||
5 | |||
6 | opensslconf.h is un-defining BN_LLONG only when included from bn.h which | ||
7 | is not robust at all, especially when include guards are used and | ||
8 | multiple inclusions of a given header is not allowed. so lets take out | ||
9 | the nesting constraint and add OPENSSL_SYS_UEFI constraint instead | ||
10 | |||
11 | Upstream-Status: Inappropriate [ fixed differently with OpenSSL 1.1+ ] | ||
12 | |||
13 | Signed-off-by: Khem Raj <raj.khem@gmail.com> | ||
14 | --- | ||
15 | crypto/opensslconf.h.in | 2 +- | ||
16 | 1 file changed, 1 insertion(+), 1 deletion(-) | ||
17 | |||
18 | diff --git a/crypto/opensslconf.h.in b/crypto/opensslconf.h.in | ||
19 | index 7a1c85d..a10c10f 100644 | ||
20 | --- a/crypto/opensslconf.h.in | ||
21 | +++ b/crypto/opensslconf.h.in | ||
22 | @@ -56,7 +56,7 @@ | ||
23 | #endif | ||
24 | #endif | ||
25 | |||
26 | -#if defined(HEADER_BN_H) && !defined(CONFIG_HEADER_BN_H) | ||
27 | +#if !defined(OPENSSL_SYS_UEFI) && !defined(CONFIG_HEADER_BN_H) | ||
28 | #define CONFIG_HEADER_BN_H | ||
29 | #undef BN_LLONG | ||
30 | |||
31 | -- | ||
32 | 2.20.1 | ||
33 | |||
diff --git a/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb b/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb index 809634f6c0..88aefdea4f 100644 --- a/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb +++ b/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb | |||
@@ -40,6 +40,7 @@ SRC_URI = "http://www.openssl.org/source/openssl-${PV}.tar.gz \ | |||
40 | file://0001-Fix-build-with-clang-using-external-assembler.patch \ | 40 | file://0001-Fix-build-with-clang-using-external-assembler.patch \ |
41 | file://0001-openssl-force-soft-link-to-avoid-rare-race.patch \ | 41 | file://0001-openssl-force-soft-link-to-avoid-rare-race.patch \ |
42 | file://0001-allow-manpages-to-be-disabled.patch \ | 42 | file://0001-allow-manpages-to-be-disabled.patch \ |
43 | file://0001-Fix-BN_LLONG-breakage.patch \ | ||
43 | " | 44 | " |
44 | 45 | ||
45 | SRC_URI_append_class-target = " \ | 46 | SRC_URI_append_class-target = " \ |