Hi Pali,
On Mon, 2 Aug 2021 at 23:04, Heinrich Schuchardt xypron.glpk@gmx.de wrote:
+cc Ilias
On 8/2/21 3:18 PM, Pali Rohár wrote:
Macro U_BOOT_VERSION_STRING is already stored in variable version_string. So use directly this variable instead of storing U_BOOT_VERSION_STRING into temporary variable.
Looks ok Acked-by: Ilias Apalodimas ilias.apalodimas@linaro.org
Signed-off-by: Pali Rohár pali@kernel.org
lib/efi_loader/efi_tcg2.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/lib/efi_loader/efi_tcg2.c b/lib/efi_loader/efi_tcg2.c index 1319a8b37868..14353ae71c9b 100644 --- a/lib/efi_loader/efi_tcg2.c +++ b/lib/efi_loader/efi_tcg2.c @@ -1275,11 +1275,10 @@ free_pool: static efi_status_t efi_append_scrtm_version(struct udevice *dev) { struct tpml_digest_values digest_list;
u8 ver[] = U_BOOT_VERSION_STRING; const int pcr_index = 0; efi_status_t ret;ret = tcg2_create_digest(ver, sizeof(ver), &digest_list);
ret = tcg2_create_digest(version_string, strlen(version_string) + 1, &digest_list); if (ret != EFI_SUCCESS) goto out;@@ -1288,7 +1287,7 @@ static efi_status_t efi_append_scrtm_version(struct udevice *dev) goto out;
ret = tcg2_agile_log_append(pcr_index, EV_S_CRTM_VERSION, &digest_list,
sizeof(ver), ver);
strlen(version_string) + 1, version_string);out: return ret;