From 3b278dd9e6643a9f75ef108d229c619e197067d0 Mon Sep 17 00:00:00 2001 From: Sam Protsenko Date: Fri, 3 Oct 2025 17:27:14 -0500 Subject: [PATCH] bootstd: Fix bootflow info for efi_mgr 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 Reviewed-by: Simon Glass Reviewed-by: Mattijs Korpershoek --- cmd/bootflow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/bootflow.c b/cmd/bootflow.c index 551dffbb8b8..081f64e12ce 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)); -- 2.47.3