[PATCH 08/13] boot: Move the architecture check into check_allowed()
Simon Glass
sjg at chromium.org
Wed Mar 25 17:54:17 CET 2026
We may as well have all the checks together, so move the call to
fit_image_check_target_arch() into check_allowed().
Signed-off-by: Simon Glass <sjg at chromium.org>
---
boot/image-fit.c | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/boot/image-fit.c b/boot/image-fit.c
index ba8a56a8ff4..0a880863063 100644
--- a/boot/image-fit.c
+++ b/boot/image-fit.c
@@ -2194,6 +2194,15 @@ static int check_allowed(const void *fit, int noffset,
bool type_ok, os_ok;
uint8_t os;
+ bootstage_mark(bootstage_id + BOOTSTAGE_SUB_CHECK_ARCH);
+ if (!tools_build() && IS_ENABLED(CONFIG_SANDBOX)) {
+ if (!fit_image_check_target_arch(fit, noffset)) {
+ puts("Unsupported Architecture\n");
+ bootstage_error(bootstage_id + BOOTSTAGE_SUB_CHECK_ARCH);
+ return -ENOEXEC;
+ }
+ }
+
bootstage_mark(bootstage_id + BOOTSTAGE_SUB_CHECK_ALL);
type_ok = fit_image_check_type(fit, noffset, image_type) ||
fit_image_check_type(fit, noffset, IH_TYPE_FIRMWARE) ||
@@ -2379,15 +2388,6 @@ int fit_image_load(struct bootm_headers *images, ulong addr,
if (noffset < 0)
return noffset;
- bootstage_mark(bootstage_id + BOOTSTAGE_SUB_CHECK_ARCH);
- if (!tools_build() && IS_ENABLED(CONFIG_SANDBOX)) {
- if (!fit_image_check_target_arch(fit, noffset)) {
- puts("Unsupported Architecture\n");
- bootstage_error(bootstage_id + BOOTSTAGE_SUB_CHECK_ARCH);
- return -ENOEXEC;
- }
- }
-
ret = check_allowed(fit, noffset, image_type, arch, bootstage_id);
if (ret)
return ret;
--
2.43.0
More information about the U-Boot
mailing list