Skip to content

Commit

Permalink
[nrf noup] Fix KMU breaking non-KMU builds of MCUboot
Browse files Browse the repository at this point in the history
The commit fixes problem where usage of OR instead of AND
in conditional expression broke non-KMU signature builds
of MCUboot.
The commit also removes BOOT_SIGNATURE_TYPE_PURE_ALLOW
on NRF_SECURITY, which breaks build for non-PSA builds.

fixup! [nrf noup] bootutil: Add support for KMU stored ED25519 signature key

Signed-off-by: Dominik Ermel <[email protected]>
  • Loading branch information
de-nordic authored and rlubos committed Oct 29, 2024
1 parent 0a70e49 commit 352a241
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion boot/zephyr/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,7 @@ if(CONFIG_MCUBOOT_SERIAL)
endif()
endif()

if(NOT CONFIG_BOOT_SIGNATURE_USING_KMU OR NOT CONFIG_BOOT_SIGNATURE_KEY_FILE STREQUAL "")
if(NOT CONFIG_BOOT_SIGNATURE_USING_KMU AND NOT CONFIG_BOOT_SIGNATURE_KEY_FILE STREQUAL "")
# CONF_FILE points to the KConfig configuration files of the bootloader.
foreach (filepath ${CONF_FILE})
file(READ ${filepath} temp_text)
Expand Down
1 change: 0 additions & 1 deletion boot/zephyr/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,6 @@ endchoice # BOOT_IMG_HASH_ALG

config BOOT_SIGNATURE_TYPE_PURE_ALLOW
bool
depends on NRF_SECURITY
help
Hidden option set by configurations that allow Pure variant,
for example ed25519. The pure variant means that image
Expand Down

0 comments on commit 352a241

Please sign in to comment.