diff options
| author | nagendra modadugu <ngm@google.com> | 2016-12-13 13:31:21 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2016-12-14 06:03:07 -0800 |
| commit | c648430a6d3bd525becb523bf50703f16e147515 (patch) | |
| tree | e84285f54f049f879d06944ff6749d2a3eea922b /board/cr50/tpm2 | |
| parent | dba4c111137b1819538a314043760bf17092690f (diff) | |
| download | chrome-ec-c648430a6d3bd525becb523bf50703f16e147515.tar.gz | |
CR50: fix errors flagged by coverity
- Update SHA_DIGEST_MAX_BYTES to reflect SHA-512 support
- Fix unitialized variable error in tpm2/hash.c
BRANCH=none
BUG=none
CQ-DEPEND=CL:419327
TEST=make buildall succeeds
Change-Id: Ie002e5807d1e616da034dbb8896867572e148e00
Signed-off-by: nagendra modadugu <ngm@google.com>
Reviewed-on: https://chromium-review.googlesource.com/419698
Commit-Ready: Nagendra Modadugu <ngm@google.com>
Tested-by: Nagendra Modadugu <ngm@google.com>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'board/cr50/tpm2')
| -rw-r--r-- | board/cr50/tpm2/hash.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/board/cr50/tpm2/hash.c b/board/cr50/tpm2/hash.c index 8cd0dca41b..6c8ac91f3c 100644 --- a/board/cr50/tpm2/hash.c +++ b/board/cr50/tpm2/hash.c @@ -107,9 +107,11 @@ uint16_t _cpri__StartHash(TPM_ALG_ID alg, BOOL sequence, case TPM_ALG_SHA384: DCRYPTO_SHA384_init(ctx); + result = HASH_size(ctx); break; case TPM_ALG_SHA512: DCRYPTO_SHA512_init(ctx); + result = HASH_size(ctx); break; default: result = 0; |