On 2026-06-28T08:27:58, Heinrich Schuchardt
<[email protected]> 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 <[email protected]>
> Signed-off-by: Heinrich Schuchardt <[email protected]>
>
> lib/efi_loader/dtbdump.c    | 2 +-
>  lib/efi_loader/smbiosdump.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass <[email protected]>

Reply via email to