[coreboot-gerrit] Patch set updated for coreboot: tpm2: remove unused buffer size definition

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Tue Jul 12 19:25:39 CEST 2016


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15609

-gerrit

commit 27966f75d3f759ec446589f1402648984ed0b5a9
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Thu Jul 7 11:15:47 2016 -0700

    tpm2: remove unused buffer size definition
    
    TPM2 structure definitions use pointers instead of buffers where
    possible. One structure was left behind. Replace that buffer definition
    with a pointer to be consistent.
    
    BRANCH=none
    BUG=chrome-os-partner:50645
    TEST=compilation succeeds, the code using the changed structure in the
         upcoming patches allows to successfully boot chrome OS on Kevin
    
    Change-Id: Iea59943aa0ad6e42fcd479765a9ded0d7a1680d7
    Signed-off-by: Martin Roth <martinroth at chromium.org>
    Original-Commit-Id: 02b2909b1875ba65cd56cf8e3697a2b67ddaea07
    Original-Change-Id: I9856ac516be13f5892ba8af0526708409a297033
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/358771
    Original-Reviewed-by: Douglas Anderson <dianders at chromium.org>
---
 src/lib/tpm2_tlcl_structures.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/tpm2_tlcl_structures.h b/src/lib/tpm2_tlcl_structures.h
index fad9c0d..1bf3abc 100644
--- a/src/lib/tpm2_tlcl_structures.h
+++ b/src/lib/tpm2_tlcl_structures.h
@@ -198,7 +198,7 @@ typedef struct {
 typedef union {
 	struct {
 		uint16_t  size;
-		uint8_t   buffer[sizeof(TPMU_HA)];
+		const uint8_t   *buffer;
 	} t;
 	TPM2B b;
 } TPM2B_DIGEST;



More information about the coreboot-gerrit mailing list