build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29563 )
Change subject: security/tpm: Fix TCPA log feature ......................................................................
Patch Set 61:
(9 comments)
https://review.coreboot.org/#/c/29563/61/src/include/memlayout.h File src/include/memlayout.h:
https://review.coreboot.org/#/c/29563/61/src/include/memlayout.h@167 PS61, Line 167: #define VBOOT2_TPM_LOG(addr, size) \ Macros with multiple statements should be enclosed in a do - while loop
https://review.coreboot.org/#/c/29563/61/src/include/memlayout.h@167 PS61, Line 167: #define VBOOT2_TPM_LOG(addr, size) \ macros should not use a trailing semicolon
https://review.coreboot.org/#/c/29563/61/src/security/tpm/tspi/log.c File src/security/tpm/tspi/log.c:
https://review.coreboot.org/#/c/29563/61/src/security/tpm/tspi/log.c@94 PS61, Line 94: enum vb2_hash_algorithm digest_algo, const uint8_t *digest, line over 80 characters
https://review.coreboot.org/#/c/29563/61/src/security/tpm/tspi/log.c@156 PS61, Line 156: struct tcpa_entry *tce = &ram_log->entries[ram_log->num_entries++]; line over 80 characters
https://review.coreboot.org/#/c/29563/61/src/security/tpm/tspi/log.c@157 PS61, Line 157: strncpy(tce->name, preram_log->entries[i].name, TCPA_PCR_HASH_NAME - 1); line over 80 characters
https://review.coreboot.org/#/c/29563/61/src/security/tpm/tspi/log.c@160 PS61, Line 160: if (preram_log->entries[i].digest_length > TCPA_DIGEST_MAX_LENGTH) { line over 80 characters
https://review.coreboot.org/#/c/29563/61/src/security/tpm/tspi/log.c@165 PS61, Line 165: strncpy(tce->digest_type, preram_log->entries[i].digest_type, TCPA_PCR_HASH_LEN - 1); line over 80 characters
https://review.coreboot.org/#/c/29563/61/src/security/tpm/tspi/log.c@166 PS61, Line 166: tce->digest_length = MIN(preram_log->entries[i].digest_length, TCPA_DIGEST_MAX_LENGTH); line over 80 characters
https://review.coreboot.org/#/c/29563/61/src/security/tpm/tspi/log.c@167 PS61, Line 167: memcpy(tce->digest, preram_log->entries[i].digest, tce->digest_length); line over 80 characters
-
build bot (Jenkins) (Code Review)