[PATCH v2 2/4] efi_loader: correct return value of efi_bootmgr_run()

[Ilias Apalodimas]

On Tue, 21 Apr 2026 at 01:03, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> efi_bootmgr_run() is expected to return an efi_status_t value.
>
> Reported-by: Simon Glass <sjg at chromium.org>
> Fixes: 0bef4b0123f2 ("cmd: bootefi: move library interfaces under lib/efi_loader")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>

> ---
> v2:
>         new patch
> ---
>  lib/efi_loader/efi_bootmgr.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/efi_loader/efi_bootmgr.c b/lib/efi_loader/efi_bootmgr.c
> index a687f4d8e85..52ccebab19f 100644
> --- a/lib/efi_loader/efi_bootmgr.c
> +++ b/lib/efi_loader/efi_bootmgr.c
> @@ -1313,7 +1313,7 @@ efi_status_t efi_bootmgr_run(void *fdt)
>         if (ret != EFI_SUCCESS) {
>                 log_err("Error: Cannot initialize UEFI sub-system, r = %lu\n",
>                         ret & ~EFI_ERROR_MASK);
> -               return CMD_RET_FAILURE;
> +               return ret;
>         }
>
>         ret = efi_bootmgr_load(&handle, &load_options);
> --
> 2.53.0
>