diff options
author | Randall Spangler <rspangler@chromium.org> | 2016-10-14 10:41:44 -0700 |
---|---|---|
committer | Randall Spangler <rspangler@chromium.org> | 2016-10-29 00:57:07 +0000 |
commit | 49e517d818fbf4a78fe452a8f7f38a293f945804 (patch) | |
tree | f2be8bb69e85acc6b19a8d04da1f667a67cd6266 /firmware/lib/vboot_kernel.c | |
parent | bf6263d52994937033b397833be836d081b61958 (diff) | |
download | vboot-49e517d818fbf4a78fe452a8f7f38a293f945804.tar.gz |
vboot: use vb2_safe_memcmp instead of SafeMemcmp
No need to have two implementations of this now.
BUG=chromium:611535
BRANCH=none
TEST=make runtests; emerge-kevin coreboot depthcharge
Change-Id: I18bac928eb09971c37f3e1d7cbfd2009999b1f31
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/400899
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'firmware/lib/vboot_kernel.c')
-rw-r--r-- | firmware/lib/vboot_kernel.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/firmware/lib/vboot_kernel.c b/firmware/lib/vboot_kernel.c index 1e7b7d0d..be5389a4 100644 --- a/firmware/lib/vboot_kernel.c +++ b/firmware/lib/vboot_kernel.c @@ -288,15 +288,17 @@ VbError_t LoadKernel(LoadKernelParams *params, VbCommonParams *cparams) VBDEBUG(("Checking developer key hash.\n")); vb2_digest_buffer(buf, buflen, VB2_HASH_SHA256, digest, sizeof(digest)); - if (0 != SafeMemcmp(digest, params->fwmp->dev_key_hash, - VB2_SHA256_DIGEST_SIZE)) { + if (0 != vb2_safe_memcmp(digest, + params->fwmp->dev_key_hash, + VB2_SHA256_DIGEST_SIZE)) { int i; VBDEBUG(("Wrong developer key hash.\n")); VBDEBUG(("Want: ")); for (i = 0; i < VB2_SHA256_DIGEST_SIZE; i++) VBDEBUG(("%02x", - params->fwmp->dev_key_hash[i])); + params-> + fwmp->dev_key_hash[i])); VBDEBUG(("\nGot: ")); for (i = 0; i < VB2_SHA256_DIGEST_SIZE; i++) VBDEBUG(("%02x", digest[i])); |