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]>

