]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
efi_selftest: ReinstallProtocolInterface test
authorHeinrich Schuchardt <heinrich.schuchardt@canonical.com>
Sat, 10 Jun 2023 19:25:18 +0000 (21:25 +0200)
committerHeinrich Schuchardt <heinrich.schuchardt@canonical.com>
Fri, 16 Jun 2023 04:45:19 +0000 (06:45 +0200)
Test ReinstallProtocolInterface() more rigorously.

Replacing the sole installed protocol interface must not result in deleting
the handle and creating a new one.

Check which interface is actually installed before and after
ReinstallProtocolInterface().

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
lib/efi_selftest/efi_selftest_register_notify.c

index ad4bcce1a1036de34418e5fc8c0fb3f2ba5b3769..adf5dd00a1e7aee2bb972e11d9750134cbc979ac 100644 (file)
@@ -124,6 +124,7 @@ static int execute(void)
 {
        efi_status_t ret;
        efi_handle_t handle1 = NULL, handle2 = NULL;
+       struct interface *interface;
        struct interface interface1, interface2;
 
        ret = boottime->install_protocol_interface(&handle1, &guid1,
@@ -145,6 +146,18 @@ static int execute(void)
                efi_st_error("LocateHandle failed\n");
                return EFI_ST_FAILURE;
        }
+       interface = NULL;
+       ret = boottime->open_protocol(handle1, &guid1, (void**)&interface,
+                                     NULL, NULL,
+                                     EFI_OPEN_PROTOCOL_GET_PROTOCOL);
+       if (ret != EFI_SUCCESS) {
+               efi_st_error("Cannot find installed protocol on handle\n");
+               return EFI_ST_FAILURE;
+       }
+       if (interface != &interface1) {
+               efi_st_error("Wrong interface after install\n");
+               return EFI_ST_FAILURE;
+       }
        ret = boottime->free_pool(context.handles);
        if (ret != EFI_SUCCESS) {
                efi_st_error("FreePool failed\n");
@@ -186,6 +199,18 @@ static int execute(void)
                efi_st_error("FreePool failed\n");
                return EFI_ST_FAILURE;
        }
+       interface = NULL;
+       ret = boottime->open_protocol(handle1, &guid1, (void**)&interface,
+                                     NULL, NULL,
+                                     EFI_OPEN_PROTOCOL_GET_PROTOCOL);
+       if (ret != EFI_SUCCESS) {
+               efi_st_error("Cannot find reinstalled protocol on handle\n");
+               return EFI_ST_FAILURE;
+       }
+       if (interface != &interface2) {
+               efi_st_error("Wrong interface after reinstall\n");
+               return EFI_ST_FAILURE;
+       }
        context.notify_count = 0;
        ret = boottime->install_protocol_interface(&handle2, &guid1,
                                                   EFI_NATIVE_INTERFACE,