diff options
author | Heinrich Schuchardt <xypron.glpk@gmx.de> | 2018-01-11 08:15:56 +0100 |
---|---|---|
committer | Alexander Graf <agraf@suse.de> | 2018-01-22 23:09:12 +0100 |
commit | 5be9744ae78a399b93e89d19777619714d178764 (patch) | |
tree | 12a0afc7edf8436cd8e38f81bd941b0fcd4801a0 /lib/efi_selftest/efi_selftest_devicepath.c | |
parent | 32e6fed6e93fad0b38aa5bdec085a3ae316aa97b (diff) |
efi_selftest: do not try to close device path protocol
CloseProtocol cannot be called without agent handle.
There is no need to close the device path protocol if
it has been opened without agent handle.
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Alexander Graf <agraf@suse.de>
Diffstat (limited to 'lib/efi_selftest/efi_selftest_devicepath.c')
-rw-r--r-- | lib/efi_selftest/efi_selftest_devicepath.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/efi_selftest/efi_selftest_devicepath.c b/lib/efi_selftest/efi_selftest_devicepath.c index 1ab54ebb37..d9a6697892 100644 --- a/lib/efi_selftest/efi_selftest_devicepath.c +++ b/lib/efi_selftest/efi_selftest_devicepath.c @@ -299,10 +299,10 @@ static int execute(void) efi_st_error("FreePool failed\n"); return EFI_ST_FAILURE; } - ret = boottime->close_protocol(handles[i], &guid_device_path, - NULL, NULL); - if (ret != EFI_SUCCESS) - efi_st_todo("Cannot close device path protocol.\n"); + /* + * CloseProtocol cannot be called without agent handle. + * There is no need to close the device path protocol. + */ } ret = boottime->free_pool(handles); if (ret != EFI_SUCCESS) { |