diff options
author | Ilias Apalodimas <ilias.apalodimas@linaro.org> | 2023-10-19 11:21:55 -0500 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2023-10-27 16:01:47 -0400 |
commit | 946b311e47da22e409f2b0af5a3930db465e7eb8 (patch) | |
tree | bf12cdd2a2644a42ae7fe960130bb2bd650d32b8 /lib | |
parent | 526697d01cdfd5a9fdf505e934dd6f5131918c43 (diff) | |
download | u-boot-946b311e47da22e409f2b0af5a3930db465e7eb8.tar.gz |
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 <ilias.apalodimas@linaro.org>
Reviewed-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Diffstat (limited to 'lib')
-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 7b7926a0d4..626df12a37 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: |