[PATCH] bootstd: Fix bootflow info for efi_mgr
Sam Protsenko
semen.protsenko at linaro.org
Sat Oct 4 00:27:14 CEST 2025
A "Synchronous Abort" CPU exception happens on an attempt to run the
"bootflow info" command for a global boot method (e.g. efi_mgr):
=> bootflow select 0
=> bootflow info
"Synchronous Abort" handler, esr 0x96000006, far 0x8
It happens because do_bootflow_info() tries to dereference bflow->dev,
which is NULL in case of efi_mgr. Add the corresponding check to prevent
this NULL pointer dereference and make "bootflow info" command work
properly for global boot methods.
Fixes: 2d653f686b6d ("bootstd: Add a bootflow command")
Signed-off-by: Sam Protsenko <semen.protsenko at linaro.org>
---
cmd/bootflow.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cmd/bootflow.c b/cmd/bootflow.c
index 551dffbb8b89..081f64e12ce9 100644
--- a/cmd/bootflow.c
+++ b/cmd/bootflow.c
@@ -404,7 +404,7 @@ static int do_bootflow_info(struct cmd_tbl *cmdtp, int flag, int argc,
}
printf("Name: %s\n", bflow->name);
- printf("Device: %s\n", bflow->dev->name);
+ printf("Device: %s\n", bflow->dev ? bflow->dev->name : "(none)");
printf("Block dev: %s\n", bflow->blk ? bflow->blk->name : "(none)");
printf("Method: %s\n", bflow->method->name);
printf("State: %s\n", bootflow_state_get_name(bflow->state));
base-commit: d33b21b7e261691e8d6613a24cc9b0ececba3b01
--
2.39.5
More information about the U-Boot
mailing list