Commit c0a62ead authored by Eric Biggers's avatar Eric Biggers Committed by Borislav Petkov (AMD)
Browse files

x86/microcode/AMD: Use sha256() instead of init/update/final 



Just call sha256() instead of doing the init/update/final sequence.

No functional changes.

Signed-off-by: default avatarEric Biggers <ebiggers@google.com>
Signed-off-by: default avatarBorislav Petkov (AMD) <bp@alien8.de>
Link: https://lore.kernel.org/20250428183006.782501-1-ebiggers@kernel.org
parent 4e2c7197

arch/x86/kernel/cpu/microcode/amd.c

+1 −4
Original line number Diff line number Diff line
@@ -211,7 +211,6 @@ static bool verify_sha256_digest(u32 patch_id, u32 cur_rev, const u8 *data, unsi 
{

	struct patch_digest *pd = NULL;

	u8 digest[SHA256_DIGEST_SIZE];

	struct sha256_state s;

	int i;



	if (x86_family(bsp_cpuid_1_eax) < 0x17 ||
@@ -230,9 +229,7 @@ static bool verify_sha256_digest(u32 patch_id, u32 cur_rev, const u8 *data, unsi 
		return false;

	}



	sha256_init(&s);

	sha256_update(&s, data, len);

	sha256_final(&s, digest);

	sha256(data, len, digest);



	if (memcmp(digest, pd->sha256, sizeof(digest))) {

		pr_err("Patch 0x%x SHA256 digest mismatch!\n", patch_id);