[PATCH 1/1] efi_loader: simplify efi_tcg2_hash_log_extend_event()

[Ilias Apalodimas]

On Tue, 5 Nov 2024 at 04:20, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> The value of variable nt is never used. Just use NULL when calling
> efi_check_pe().
>
> The API function is not expected to write to the console. Such output might
> have unwanted side effects on the screen layout of an EFI application.
>
> Leave error handling to the caller.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>  lib/efi_loader/efi_tcg2.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/lib/efi_loader/efi_tcg2.c b/lib/efi_loader/efi_tcg2.c
> index 866a529857e..572c6b5bf63 100644
> --- a/lib/efi_loader/efi_tcg2.c
> +++ b/lib/efi_loader/efi_tcg2.c
> @@ -607,12 +607,9 @@ efi_tcg2_hash_log_extend_event(struct efi_tcg2_protocol *this, u64 flags,
>          * Format"
>          */
>         if (flags & PE_COFF_IMAGE) {
> -               IMAGE_NT_HEADERS32 *nt;
> -
>                 ret = efi_check_pe((void *)(uintptr_t)data_to_hash,
> -                                  data_to_hash_len, (void **)&nt);
> +                                  data_to_hash_len, NULL);
>                 if (ret != EFI_SUCCESS) {
> -                       log_err("Not a valid PE-COFF file\n");
>                         ret = EFI_UNSUPPORTED;
>                         goto out;
>                 }
> --
> 2.45.2
>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>