mirror of
				git://git.openwrt.org/openwrt/openwrt.git
				synced 2025-11-03 22:44:27 -05:00 
			
		
		
		
	uboot-mvebu: fix build ; use the build's tools/libressl
Since I have no openssl-dev on my machine, I first get this error: ``` tools/kwbimage.c:21:10: fatal error: openssl/bn.h: No such file or directory #include <openssl/bn.h> ``` After removing the UBOOT_MAKE_FLAGS the next error is: ``` tools/kwbimage.c:40:6: error: conflicting types for ‘EVP_MD_CTX_cleanup’ void EVP_MD_CTX_cleanup(EVP_MD_CTX *ctx) ``` After removing the OpenSSL patches the next error is: ``` HOSTLD tools/dumpimage /usr/bin/ld: cannot find -lssl /usr/bin/ld: cannot find -lcrypto collect2: error: ld returned 1 exit status scripts/Makefile.host:108: recipe for target 'tools/dumpimage' failed make[5]: *** [tools/dumpimage] Error 1 ``` So, the final part is to add the build system's HOST_LDFLAGS to the UBOOT_MAKE_FLAGS. (which was done in the previous commit) Then the image builds. Signed-off-by: Alexandru Ardelean <ardeleanalex@gmail.com>
This commit is contained in:
		
							parent
							
								
									c72e6582c0
								
							
						
					
					
						commit
						a6f79f5e5e
					
				@ -16,8 +16,6 @@ PKG_HASH:=f54baf3f9325bf444c7905f3a5b6f83680edb1e6e1a4d5f8a5ad80abe885113f
 | 
			
		||||
include $(INCLUDE_DIR)/u-boot.mk
 | 
			
		||||
include $(INCLUDE_DIR)/package.mk
 | 
			
		||||
 | 
			
		||||
UBOOT_MAKE_FLAGS:=
 | 
			
		||||
 | 
			
		||||
define U-Boot/Default
 | 
			
		||||
  BUILD_TARGET:=mvebu
 | 
			
		||||
  HIDDEN:=1
 | 
			
		||||
 | 
			
		||||
@ -1,153 +0,0 @@
 | 
			
		||||
From 59be82ef7e7ec4be6e1597d8aef65dd3d8c3a0d9 Mon Sep 17 00:00:00 2001
 | 
			
		||||
From: Jelle van der Waa <jelle@vdwaa.nl>
 | 
			
		||||
Date: Mon, 8 May 2017 21:31:19 +0200
 | 
			
		||||
Subject: [PATCH 1/2] rsa: Fix build with OpenSSL 1.1.x
 | 
			
		||||
 | 
			
		||||
The rsa_st struct has been made opaque in 1.1.x, add forward compatible
 | 
			
		||||
code to access the n, e, d members of rsa_struct.
 | 
			
		||||
 | 
			
		||||
EVP_MD_CTX_cleanup has been removed in 1.1.x and EVP_MD_CTX_reset should be
 | 
			
		||||
called to reinitialise an already created structure.
 | 
			
		||||
---
 | 
			
		||||
 lib/rsa/rsa-sign.c | 44 ++++++++++++++++++++++++++++++++++++++------
 | 
			
		||||
 1 file changed, 38 insertions(+), 6 deletions(-)
 | 
			
		||||
 | 
			
		||||
--- a/lib/rsa/rsa-sign.c
 | 
			
		||||
+++ b/lib/rsa/rsa-sign.c
 | 
			
		||||
@@ -9,6 +9,7 @@
 | 
			
		||||
 #include <string.h>
 | 
			
		||||
 #include <image.h>
 | 
			
		||||
 #include <time.h>
 | 
			
		||||
+#include <openssl/bn.h>
 | 
			
		||||
 #include <openssl/rsa.h>
 | 
			
		||||
 #include <openssl/pem.h>
 | 
			
		||||
 #include <openssl/err.h>
 | 
			
		||||
@@ -20,6 +21,19 @@
 | 
			
		||||
 #define HAVE_ERR_REMOVE_THREAD_STATE
 | 
			
		||||
 #endif
 | 
			
		||||
 
 | 
			
		||||
+#if OPENSSL_VERSION_NUMBER < 0x10100000L
 | 
			
		||||
+static void RSA_get0_key(const RSA *r,
 | 
			
		||||
+                 const BIGNUM **n, const BIGNUM **e, const BIGNUM **d)
 | 
			
		||||
+{
 | 
			
		||||
+   if (n != NULL)
 | 
			
		||||
+       *n = r->n;
 | 
			
		||||
+   if (e != NULL)
 | 
			
		||||
+       *e = r->e;
 | 
			
		||||
+   if (d != NULL)
 | 
			
		||||
+       *d = r->d;
 | 
			
		||||
+}
 | 
			
		||||
+#endif
 | 
			
		||||
+
 | 
			
		||||
 static int rsa_err(const char *msg)
 | 
			
		||||
 {
 | 
			
		||||
 	unsigned long sslErr = ERR_get_error();
 | 
			
		||||
@@ -286,16 +300,22 @@ static int rsa_init(void)
 | 
			
		||||
 {
 | 
			
		||||
 	int ret;
 | 
			
		||||
 
 | 
			
		||||
+#if OPENSSL_VERSION_NUMBER < 0x10100000L
 | 
			
		||||
 	ret = SSL_library_init();
 | 
			
		||||
+#else
 | 
			
		||||
+	ret = OPENSSL_init_ssl(0, NULL);
 | 
			
		||||
+#endif
 | 
			
		||||
 	if (!ret) {
 | 
			
		||||
 		fprintf(stderr, "Failure to init SSL library\n");
 | 
			
		||||
 		return -1;
 | 
			
		||||
 	}
 | 
			
		||||
+#if OPENSSL_VERSION_NUMBER < 0x10100000L
 | 
			
		||||
 	SSL_load_error_strings();
 | 
			
		||||
 
 | 
			
		||||
 	OpenSSL_add_all_algorithms();
 | 
			
		||||
 	OpenSSL_add_all_digests();
 | 
			
		||||
 	OpenSSL_add_all_ciphers();
 | 
			
		||||
+#endif
 | 
			
		||||
 
 | 
			
		||||
 	return 0;
 | 
			
		||||
 }
 | 
			
		||||
@@ -335,12 +355,15 @@ err_set_rsa:
 | 
			
		||||
 err_engine_init:
 | 
			
		||||
 	ENGINE_free(e);
 | 
			
		||||
 err_engine_by_id:
 | 
			
		||||
+#if OPENSSL_VERSION_NUMBER < 0x10100000L
 | 
			
		||||
 	ENGINE_cleanup();
 | 
			
		||||
+#endif
 | 
			
		||||
 	return ret;
 | 
			
		||||
 }
 | 
			
		||||
 
 | 
			
		||||
 static void rsa_remove(void)
 | 
			
		||||
 {
 | 
			
		||||
+#if OPENSSL_VERSION_NUMBER < 0x10100000L
 | 
			
		||||
 	CRYPTO_cleanup_all_ex_data();
 | 
			
		||||
 	ERR_free_strings();
 | 
			
		||||
 #ifdef HAVE_ERR_REMOVE_THREAD_STATE
 | 
			
		||||
@@ -349,6 +372,7 @@ static void rsa_remove(void)
 | 
			
		||||
 	ERR_remove_state(0);
 | 
			
		||||
 #endif
 | 
			
		||||
 	EVP_cleanup();
 | 
			
		||||
+#endif
 | 
			
		||||
 }
 | 
			
		||||
 
 | 
			
		||||
 static void rsa_engine_remove(ENGINE *e)
 | 
			
		||||
@@ -409,7 +433,11 @@ static int rsa_sign_with_key(RSA *rsa, s
 | 
			
		||||
 		ret = rsa_err("Could not obtain signature");
 | 
			
		||||
 		goto err_sign;
 | 
			
		||||
 	}
 | 
			
		||||
-	EVP_MD_CTX_cleanup(context);
 | 
			
		||||
+	#if OPENSSL_VERSION_NUMBER < 0x10100000L
 | 
			
		||||
+		EVP_MD_CTX_cleanup(context);
 | 
			
		||||
+	#else
 | 
			
		||||
+		EVP_MD_CTX_reset(context);
 | 
			
		||||
+	#endif
 | 
			
		||||
 	EVP_MD_CTX_destroy(context);
 | 
			
		||||
 	EVP_PKEY_free(key);
 | 
			
		||||
 
 | 
			
		||||
@@ -479,6 +507,7 @@ static int rsa_get_exponent(RSA *key, ui
 | 
			
		||||
 {
 | 
			
		||||
 	int ret;
 | 
			
		||||
 	BIGNUM *bn_te;
 | 
			
		||||
+	const BIGNUM *key_e;
 | 
			
		||||
 	uint64_t te;
 | 
			
		||||
 
 | 
			
		||||
 	ret = -EINVAL;
 | 
			
		||||
@@ -487,17 +516,18 @@ static int rsa_get_exponent(RSA *key, ui
 | 
			
		||||
 	if (!e)
 | 
			
		||||
 		goto cleanup;
 | 
			
		||||
 
 | 
			
		||||
-	if (BN_num_bits(key->e) > 64)
 | 
			
		||||
+	RSA_get0_key(key, NULL, &key_e, NULL);
 | 
			
		||||
+	if (BN_num_bits(key_e) > 64)
 | 
			
		||||
 		goto cleanup;
 | 
			
		||||
 
 | 
			
		||||
-	*e = BN_get_word(key->e);
 | 
			
		||||
+	*e = BN_get_word(key_e);
 | 
			
		||||
 
 | 
			
		||||
-	if (BN_num_bits(key->e) < 33) {
 | 
			
		||||
+	if (BN_num_bits(key_e) < 33) {
 | 
			
		||||
 		ret = 0;
 | 
			
		||||
 		goto cleanup;
 | 
			
		||||
 	}
 | 
			
		||||
 
 | 
			
		||||
-	bn_te = BN_dup(key->e);
 | 
			
		||||
+	bn_te = BN_dup(key_e);
 | 
			
		||||
 	if (!bn_te)
 | 
			
		||||
 		goto cleanup;
 | 
			
		||||
 
 | 
			
		||||
@@ -527,6 +557,7 @@ int rsa_get_params(RSA *key, uint64_t *e
 | 
			
		||||
 {
 | 
			
		||||
 	BIGNUM *big1, *big2, *big32, *big2_32;
 | 
			
		||||
 	BIGNUM *n, *r, *r_squared, *tmp;
 | 
			
		||||
+	const BIGNUM *key_n;
 | 
			
		||||
 	BN_CTX *bn_ctx = BN_CTX_new();
 | 
			
		||||
 	int ret = 0;
 | 
			
		||||
 
 | 
			
		||||
@@ -548,7 +579,8 @@ int rsa_get_params(RSA *key, uint64_t *e
 | 
			
		||||
 	if (0 != rsa_get_exponent(key, exponent))
 | 
			
		||||
 		ret = -1;
 | 
			
		||||
 
 | 
			
		||||
-	if (!BN_copy(n, key->n) || !BN_set_word(big1, 1L) ||
 | 
			
		||||
+	RSA_get0_key(key, &key_n, NULL, NULL);
 | 
			
		||||
+	if (!BN_copy(n, key_n) || !BN_set_word(big1, 1L) ||
 | 
			
		||||
 	    !BN_set_word(big2, 2L) || !BN_set_word(big32, 32L))
 | 
			
		||||
 		ret = -1;
 | 
			
		||||
 
 | 
			
		||||
@ -1,96 +0,0 @@
 | 
			
		||||
From 65030804dc57f3488e4ffe21e72fc65cd245cb98 Mon Sep 17 00:00:00 2001
 | 
			
		||||
From: Jelle van der Waa <jelle@vdwaa.nl>
 | 
			
		||||
Date: Mon, 8 May 2017 21:31:20 +0200
 | 
			
		||||
Subject: [PATCH 2/2] tools: kwbimage fix build with OpenSSL 1.1.x
 | 
			
		||||
 | 
			
		||||
The rsa_st struct has been made opaque in 1.1.x, add forward compatible
 | 
			
		||||
code to access the n, e, d members of rsa_struct.
 | 
			
		||||
 | 
			
		||||
EVP_MD_CTX_cleanup has been removed in 1.1.x and EVP_MD_CTX_reset should be
 | 
			
		||||
called to reinitialise an already created structure.
 | 
			
		||||
 | 
			
		||||
Signed-off-by: Jelle van der Waa <jelle@vdwaa.nl>
 | 
			
		||||
---
 | 
			
		||||
 tools/kwbimage.c | 36 ++++++++++++++++++++++++++++++------
 | 
			
		||||
 1 file changed, 30 insertions(+), 6 deletions(-)
 | 
			
		||||
 | 
			
		||||
--- a/tools/kwbimage.c
 | 
			
		||||
+++ b/tools/kwbimage.c
 | 
			
		||||
@@ -18,10 +18,30 @@
 | 
			
		||||
 #include "kwbimage.h"
 | 
			
		||||
 
 | 
			
		||||
 #ifdef CONFIG_KWB_SECURE
 | 
			
		||||
+#include <openssl/bn.h>
 | 
			
		||||
 #include <openssl/rsa.h>
 | 
			
		||||
 #include <openssl/pem.h>
 | 
			
		||||
 #include <openssl/err.h>
 | 
			
		||||
 #include <openssl/evp.h>
 | 
			
		||||
+
 | 
			
		||||
+#if OPENSSL_VERSION_NUMBER < 0x10100000L
 | 
			
		||||
+static void RSA_get0_key(const RSA *r,
 | 
			
		||||
+                 const BIGNUM **n, const BIGNUM **e, const BIGNUM **d)
 | 
			
		||||
+{
 | 
			
		||||
+   if (n != NULL)
 | 
			
		||||
+       *n = r->n;
 | 
			
		||||
+   if (e != NULL)
 | 
			
		||||
+       *e = r->e;
 | 
			
		||||
+   if (d != NULL)
 | 
			
		||||
+       *d = r->d;
 | 
			
		||||
+}
 | 
			
		||||
+
 | 
			
		||||
+#else
 | 
			
		||||
+void EVP_MD_CTX_cleanup(EVP_MD_CTX *ctx)
 | 
			
		||||
+{
 | 
			
		||||
+	EVP_MD_CTX_reset(ctx);
 | 
			
		||||
+}
 | 
			
		||||
+#endif
 | 
			
		||||
 #endif
 | 
			
		||||
 
 | 
			
		||||
 static struct image_cfg_element *image_cfg;
 | 
			
		||||
@@ -470,12 +490,16 @@ static int kwb_export_pubkey(RSA *key, s
 | 
			
		||||
 			     char *keyname)
 | 
			
		||||
 {
 | 
			
		||||
 	int size_exp, size_mod, size_seq;
 | 
			
		||||
+	const BIGNUM *key_e, *key_n;
 | 
			
		||||
 	uint8_t *cur;
 | 
			
		||||
 	char *errmsg = "Failed to encode %s\n";
 | 
			
		||||
 
 | 
			
		||||
-	if (!key || !key->e || !key->n || !dst) {
 | 
			
		||||
+	RSA_get0_key(key, NULL, &key_e, NULL);
 | 
			
		||||
+	RSA_get0_key(key, &key_n, NULL, NULL);
 | 
			
		||||
+
 | 
			
		||||
+	if (!key || !key_e || !key_n || !dst) {
 | 
			
		||||
 		fprintf(stderr, "export pk failed: (%p, %p, %p, %p)",
 | 
			
		||||
-			key, key->e, key->n, dst);
 | 
			
		||||
+			key, key_e, key_n, dst);
 | 
			
		||||
 		fprintf(stderr, errmsg, keyname);
 | 
			
		||||
 		return -EINVAL;
 | 
			
		||||
 	}
 | 
			
		||||
@@ -490,8 +514,8 @@ static int kwb_export_pubkey(RSA *key, s
 | 
			
		||||
 	 * do the encoding manually.
 | 
			
		||||
 	 */
 | 
			
		||||
 
 | 
			
		||||
-	size_exp = BN_num_bytes(key->e);
 | 
			
		||||
-	size_mod = BN_num_bytes(key->n);
 | 
			
		||||
+	size_exp = BN_num_bytes(key_e);
 | 
			
		||||
+	size_mod = BN_num_bytes(key_n);
 | 
			
		||||
 	size_seq = 4 + size_mod + 4 + size_exp;
 | 
			
		||||
 
 | 
			
		||||
 	if (size_mod > 256) {
 | 
			
		||||
@@ -520,14 +544,14 @@ static int kwb_export_pubkey(RSA *key, s
 | 
			
		||||
 	*cur++ = 0x82;
 | 
			
		||||
 	*cur++ = (size_mod >> 8) & 0xFF;
 | 
			
		||||
 	*cur++ = size_mod & 0xFF;
 | 
			
		||||
-	BN_bn2bin(key->n, cur);
 | 
			
		||||
+	BN_bn2bin(key_n, cur);
 | 
			
		||||
 	cur += size_mod;
 | 
			
		||||
 	/* Exponent */
 | 
			
		||||
 	*cur++ = 0x02;		/* INTEGER */
 | 
			
		||||
 	*cur++ = 0x82;
 | 
			
		||||
 	*cur++ = (size_exp >> 8) & 0xFF;
 | 
			
		||||
 	*cur++ = size_exp & 0xFF;
 | 
			
		||||
-	BN_bn2bin(key->e, cur);
 | 
			
		||||
+	BN_bn2bin(key_e, cur);
 | 
			
		||||
 
 | 
			
		||||
 	if (hashf) {
 | 
			
		||||
 		struct hash_v1 pk_hash;
 | 
			
		||||
		Loading…
	
		Reference in New Issue
	
	Block a user