diff options
author | Tom Rini <trini@konsulko.com> | 2023-04-06 10:03:33 -0400 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2023-04-28 11:30:17 -0600 |
commit | e0dda26c2e9fb4150a5ab0b8b5ad58da5dd355cd (patch) | |
tree | 1a3bfd219caae4e733584a9681cefc3a6685087a /test/py/u_boot_utils.py | |
parent | 86592af5369ae14fb3b273479beb430d7e88de10 (diff) |
bootflow: Rework do_bootflow_menu() slightly
When building this with clang, we get a warning such as:
cmd/bootflow.c:412:27: warning: variable 'bflow' is uninitialized when used here [-Wuninitialized]
printf("Selected: %s\n", bflow->os_name ? bflow->os_name : bflow->name);
^~~~~
And a suggestion to just initialize bflow to NULL. This would however
would be ensuring a bad dereference. Instead, looking at the function we
rework things so that when CONFIG_EXPO is not enabled (and so, no UI) we
error early and would never reach this point in the code. Simplify the
rest slightly as well while at this.
Signed-off-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'test/py/u_boot_utils.py')
0 files changed, 0 insertions, 0 deletions