[PATCH 1/1] efi_loader: don't export efi_remove_all_protocols

This function is only used inside efi_boottime.c and is not safe to use outside of this context.
Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com --- include/efi_loader.h | 2 -- 1 file changed, 2 deletions(-)
diff --git a/include/efi_loader.h b/include/efi_loader.h index 160f554ee9..1bac3f49a3 100644 --- a/include/efi_loader.h +++ b/include/efi_loader.h @@ -651,8 +651,6 @@ efi_status_t efi_protocol_open(struct efi_handler *handler, efi_status_t efi_remove_protocol(const efi_handle_t handle, const efi_guid_t *protocol, void *protocol_interface); -/* Delete all protocols from a handle */ -efi_status_t efi_remove_all_protocols(const efi_handle_t handle); /* Install multiple protocol interfaces */ efi_status_t EFIAPI efi_install_multiple_protocol_interfaces(efi_handle_t *handle, ...);

Hi Heinrich,
On Fri, 7 Oct 2022 at 17:57, Heinrich Schuchardt heinrich.schuchardt@canonical.com wrote:
This function is only used inside efi_boottime.c and is not safe to use outside of this context.
Yep, thanks for this. We also need to do the same with efi_delete_handle, once I finish up that cleanup
Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com
include/efi_loader.h | 2 -- 1 file changed, 2 deletions(-)
diff --git a/include/efi_loader.h b/include/efi_loader.h index 160f554ee9..1bac3f49a3 100644 --- a/include/efi_loader.h +++ b/include/efi_loader.h @@ -651,8 +651,6 @@ efi_status_t efi_protocol_open(struct efi_handler *handler, efi_status_t efi_remove_protocol(const efi_handle_t handle, const efi_guid_t *protocol, void *protocol_interface); -/* Delete all protocols from a handle */ -efi_status_t efi_remove_all_protocols(const efi_handle_t handle); /* Install multiple protocol interfaces */ efi_status_t EFIAPI efi_install_multiple_protocol_interfaces(efi_handle_t *handle, ...); -- 2.37.2
Reviewed-by: Ilias Apalodimas ilias.apalodimas@linaro.org
participants (2)
-
Heinrich Schuchardt
-
Ilias Apalodimas