efi_loader: superfluous check in efi_remove_protocol()
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Fri, 10 May 2019 17:55:53 +0000 (19:55 +0200)
committerHeinrich Schuchardt <xypron.glpk@gmx.de>
Sun, 12 May 2019 18:54:22 +0000 (20:54 +0200)
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

index 50c5374e5fb4b9236e743e58a3ca1008495e3f9a..b8589b9531c0c87f47df7b45f1fd90853c41a8d4 100644 (file)
@@ -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);