[PATCH v3 04/11] efi_loader: fix void* check in open_file_system()

[Simon Glass]

On 2026-06-28T08:27:58, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
> efi_loader: fix void* check in open_file_system()
>
> The UEFI specification requires FreePool() to return EFI_INVALID_PARAMETER
> when the passed buffer is invalid. But it does not well define what
> checks are required. Therefore we try to avoid passing NULL to
> FreePool() in open_file_system() used by dtbdump.c and smbiosdump.c.
> But we check the wrong variable.
>
> Check the pointer to the handle buffer, not the handle.
>
> Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
>
> lib/efi_loader/dtbdump.c    | 2 +-
>  lib/efi_loader/smbiosdump.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>