sel_id is only assigned to in the case of EXPOACT_SELECT, anything
else will leave sel_id uninitialised. Initialise it on declaration
to ensure that it is always valid.

This issue was found with Smatch.

Signed-off-by: Andrew Goodbody <andrew.goodb...@linaro.org>
---
 boot/bootflow_menu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/boot/bootflow_menu.c b/boot/bootflow_menu.c
index 9d0dc352f97..58899a83295 100644
--- a/boot/bootflow_menu.c
+++ b/boot/bootflow_menu.c
@@ -182,7 +182,7 @@ int bootflow_menu_run(struct bootstd_priv *std, bool 
text_mode,
        struct bootflow *sel_bflow;
        struct udevice *dev;
        struct expo *exp;
-       uint sel_id;
+       uint sel_id = 0;
        bool done;
        int ret;
 

---
base-commit: 7027b445cc0bfb86204ecb1f1fe596f5895048d9
change-id: 20250630-bootflow_menu_fix-4ea403f596dd

Best regards,
-- 
Andrew Goodbody <andrew.goodb...@linaro.org>

Reply via email to