diff options
| author | Ilias Apalodimas <[email protected]> | 2023-10-19 11:21:55 -0500 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2023-10-27 16:01:47 -0400 |
| commit | 946b311e47da22e409f2b0af5a3930db465e7eb8 (patch) | |
| tree | bf12cdd2a2644a42ae7fe960130bb2bd650d32b8 /lib/efi_loader | |
| parent | 526697d01cdfd5a9fdf505e934dd6f5131918c43 (diff) | |
efi_loader: fix EFI_ENTRY point on get_active_pcr_banks
efi_tcg2_get_active_pcr_banks doesn't immediately call the
EFI_ENTRY() wrapper once it enters the function. Move the call a
few lines above to cover the error cases properly as well.
Signed-off-by: Ilias Apalodimas <[email protected]>
Reviewed-by: Heinrich Schuchardt <[email protected]>
Diffstat (limited to 'lib/efi_loader')
| -rw-r--r-- | lib/efi_loader/efi_tcg2.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/efi_loader/efi_tcg2.c b/lib/efi_loader/efi_tcg2.c index 7b7926a0d46..626df12a377 100644 --- a/lib/efi_loader/efi_tcg2.c +++ b/lib/efi_loader/efi_tcg2.c @@ -1211,12 +1211,13 @@ efi_tcg2_get_active_pcr_banks(struct efi_tcg2_protocol *this, { efi_status_t ret; + EFI_ENTRY("%p, %p", this, active_pcr_banks); + if (!this || !active_pcr_banks) { ret = EFI_INVALID_PARAMETER; goto out; } - EFI_ENTRY("%p, %p", this, active_pcr_banks); ret = __get_active_pcr_banks(active_pcr_banks); out: |