diff options
author | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2021-08-19 01:49:02 +0300 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2021-09-14 18:46:48 +0300 |
commit | de41f0ee0d68bcdf7d97018fdfe4bfe9fe3e53a2 (patch) | |
tree | 5a61299f2b0786ddf20627a5ea83ee90d10f674a /include | |
parent | 423e324de2a6ada4e24d97cc7aed1c8a7cd15e5f (diff) | |
download | u-boot-de41f0ee0d68bcdf7d97018fdfe4bfe9fe3e53a2.tar.xz |
image: rsa: Move padding_algos to linker lists
We are not guaranteed to have the padding_pkcs_15_verify symbol since
commit 92c960bc1d ("lib: rsa: Remove #ifdefs from rsa.h"), and
commit 61416fe9df ("Kconfig: FIT_SIGNATURE should not select RSA_VERIFY")
The padding_algos only make sense with RSA verification, which can now
be disabled in lieu of ECDSA. In fact this will lead to build failures
because of the missing symbol mentioned earlier.
To resolve this, move the padding_algos to a linker list, with
declarations moved to rsa_verify.c. This is consistent with commit
6909edb4ce ("image: rsa: Move verification algorithm to a linker list")
One could argue that the added #ifdef USE_HOSTCC is ugly, and should
be hidden within the U_BOOT_PADDING_ALGO() macro. However, this would
be inconsistent with the "cryptos" list. This logic for was not
previously explored:
Without knowledge of the U_BOOT_PADDING_ALGO() macro, its use is
similar to something being declared. However, should #ifndef
USE_HOSTCC be part of the macro, it would not be obvious that it
behaves differently on host code and target code. Having the #ifndef
outside the macro makes this obvious.
Also, the #ifdef is not always necessary. For example ecda-verify
makes use of U_BOOT_CRYPTO_ALGO() without any accompanying #ifdefs.
The fundamental issue is a lack of separation of host and target code
in rsa_verify. Therefore, the declaration of a padding algo with the
external #ifdef is more readable and consistent.
Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/image.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/include/image.h b/include/image.h index 98b33d0629..73a763a693 100644 --- a/include/image.h +++ b/include/image.h @@ -1312,6 +1312,10 @@ struct padding_algo { const uint8_t *hash, int hash_len); }; +/* Declare a new U-Boot padding algorithm handler */ +#define U_BOOT_PADDING_ALGO(__name) \ +ll_entry_declare(struct padding_algo, __name, paddings) + /** * image_get_checksum_algo() - Look up a checksum algorithm * |