Skip to content

[nrf noup] boot: zephyr: Disable self RWX#491

Merged
nordicjm merged 1 commit into
nrfconnect:mainfrom
michalek-no:mb-self-lock
Aug 27, 2025
Merged

[nrf noup] boot: zephyr: Disable self RWX#491
nordicjm merged 1 commit into
nrfconnect:mainfrom
michalek-no:mb-self-lock

Conversation

@michalek-no
Copy link
Copy Markdown
Contributor

Disables read write and execute on mcuboots NVM
at the end of execution.

Comment thread boot/zephyr/Kconfig Outdated
Comment thread boot/zephyr/main.c
Comment thread boot/zephyr/main.c
Comment thread boot/zephyr/main.c
uint32_t reset;
};

static void __ramfunc jump_in(uint32_t reset)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we have better name for this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, however this is the hardest part. Any suggestions?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you seen names I have invented?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope. Bring it on.

Comment thread boot/zephyr/main.c Outdated
@michalek-no michalek-no force-pushed the mb-self-lock branch 2 times, most recently from c02faf0 to 3c510f1 Compare August 22, 2025 12:22
Copy link
Copy Markdown
Collaborator

@nvlsianpu nvlsianpu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just one nit

Comment thread boot/zephyr/Kconfig Outdated
Disables read write and execute on mcuboots NVM
at the end of execution.

Signed-off-by: Mateusz Michalek <mateusz.michalek@nordicsemi.no>
@sonarqubecloud
Copy link
Copy Markdown

@michalek-no michalek-no requested a review from nvlsianpu August 26, 2025 19:50
@nordicjm nordicjm merged commit 285fd59 into nrfconnect:main Aug 27, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants