Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmake: mcuboot: Set encrypted header flag when key is used #85573

Merged
merged 2 commits into from
Feb 13, 2025

Conversation

nordicjm
Copy link
Collaborator

@nordicjm nordicjm commented Feb 11, 2025

Uses the clear imgtool argument to set the encrypted flag in the
header of the signed hex output, without encrypting the data. This
addresses an issue whereby the first update would swap images and
leave the swapped output in the secondary slot without encryption

@zephyrbot
Copy link
Collaborator

zephyrbot commented Feb 11, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-mcuboot DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Feb 11, 2025
Uses the clear imgtool argument to set the encrypted flag in the
header of the signed hex output, without encrypting the data. This
addresses an issue whereby the first update would swap images and
leave the swapped output in the secondary slot without encryption

Signed-off-by: Jamie McCrae <[email protected]>
Adds a note about this change

Signed-off-by: Jamie McCrae <[email protected]>
@nordicjm nordicjm force-pushed the mcubootslot1encryptset branch from 3ee8642 to 268451b Compare February 11, 2025 11:45
@zephyrbot zephyrbot removed manifest manifest-mcuboot DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Feb 11, 2025
@kartben kartben merged commit f628add into zephyrproject-rtos:main Feb 13, 2025
23 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Build System Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants