Freeing a NULL pointer is an error in EFI, so check the pointer first,
before freeing it.

Signed-off-by: Simon Glass <s...@chromium.org>
---

(no changes since v1)

 lib/efi_loader/efi_bootmgr.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/efi_loader/efi_bootmgr.c b/lib/efi_loader/efi_bootmgr.c
index e3b8dfb6013..cb5ea0e4040 100644
--- a/lib/efi_loader/efi_bootmgr.c
+++ b/lib/efi_loader/efi_bootmgr.c
@@ -1186,7 +1186,8 @@ out:
                        free(opt[i].lo);
        }
        free(opt);
-       efi_free_pool(handles);
+       if (handles)
+               efi_free_pool(handles);
 
        if (ret == EFI_NOT_FOUND)
                return EFI_SUCCESS;
-- 
2.34.1

Reply via email to