diff options
author | Mattias Nissler <mnissler@chromium.org> | 2019-02-21 10:25:02 +0100 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-02-27 04:59:14 -0800 |
commit | 0e9bab362524168cea66476cd418322b9587a5f0 (patch) | |
tree | 9de3fd5fb10dbc886b01e9f7a3508223f41b074c /futility | |
parent | 348132bf4746058d87ec6d6c6cc7bd90c8ca3738 (diff) | |
download | vboot-0e9bab362524168cea66476cd418322b9587a5f0.tar.gz |
tpm_lite: Fix delegation table parsing
The computation to determine the number of table entries was incorrect
due to sizeof(TPM_FAMILY_TABLE_ENTRY) producing a wrong value for the
size of a single entry. TPM_FAMILY_TABLE_ENTRY actually doesn't match
the encoding of the entries in the TPM's response due to alignment.
Fix this by using a constant that reflects the correct entry size.
Relatedly, I found that ReadDelegationFamilyTableTest would have
caught the bug, but was actually not being invoked. Fix this as well.
BUG=chromium:934193
TEST=Unit tests, manual per instructions in bug.
BRANCH=none
Change-Id: Ic72ad110dc0dbf15cc3cc25b438c4bc2bd2d6015
Reviewed-on: https://chromium-review.googlesource.com/1480519
Commit-Ready: Mattias Nissler <mnissler@chromium.org>
Tested-by: Mattias Nissler <mnissler@chromium.org>
Tested-by: Andreea-Elena Costinas <acostinas@google.com>
Reviewed-by: Andrey Pronin <apronin@chromium.org>
Diffstat (limited to 'futility')
0 files changed, 0 insertions, 0 deletions