diff mbox series

[U-Boot,1/1] efi_loader: superfluous check inefi_remove_protocol()

Message ID 20190510180200.11023-1-xypron.glpk@gmx.de
State Accepted, archived
Commit 61c63db7fe8af569229cb598b018c65f9ad9c95f
Delegated to: Heinrich Schuchardt
Headers show
Series [U-Boot,1/1] efi_loader: superfluous check inefi_remove_protocol() | expand

Commit Message

Heinrich Schuchardt May 10, 2019, 6:02 p.m. UTC
efi_search_protocol() already checks that the GUID matches. Don't check
a second time.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 lib/efi_loader/efi_boottime.c | 2 --
 1 file changed, 2 deletions(-)

--
2.20.1
diff mbox series

Patch

diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c
index a1fac60ee1..7263b2e0a7 100644
--- a/lib/efi_loader/efi_boottime.c
+++ b/lib/efi_loader/efi_boottime.c
@@ -513,8 +513,6 @@  efi_status_t efi_remove_protocol(const efi_handle_t handle,
 	ret = efi_search_protocol(handle, protocol, &handler);
 	if (ret != EFI_SUCCESS)
 		return ret;
-	if (guidcmp(handler->guid, protocol))
-		return EFI_INVALID_PARAMETER;
 	if (handler->protocol_interface != protocol_interface)
 		return EFI_INVALID_PARAMETER;
 	list_del(&handler->link);