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 39:
(13 comments)
https://review.coreboot.org/#/c/29563/39/src/include/memlayout.h File src/include/memlayout.h:
https://review.coreboot.org/#/c/29563/39/src/include/memlayout.h@167 PS39, 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/39/src/include/memlayout.h@167 PS39, Line 167: #define VBOOT2_TPM_LOG(addr, size) \ macros should not use a trailing semicolon
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi.h File src/security/tpm/tspi.h:
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi.h@50 PS39, Line 50: uint32_t tpm_extend_pcr(int pcr, const char *digest_type, uint8_t *digest, size_t digest_len, line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c File src/security/tpm/tspi/log.c:
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@37 PS39, Line 37: size_t tcpa_log_len = sizeof(struct tcpa_table) + MAX_TCPA_LOG_ENTRIES * sizeof(struct tcpa_entry); line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@40 PS39, Line 40: if (!tclt && IS_ENABLED(CONFIG_VBOOT_TCPA_LOG_RECOVERY)) { line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@43 PS39, Line 43: if (strncmp(TCPA_LOG_HEADER, (const char *)tclt->header, sizeof(tclt->header))) { line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@44 PS39, Line 44: tclt->max_entries = MAX_PREMEM_TCPA_LOG_ENTRIES; line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@46 PS39, Line 46: strncpy((char *)tclt->header, TCPA_LOG_HEADER, sizeof(tclt->header)); line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@54 PS39, Line 54: strncpy((char *)tclt->header, TCPA_LOG_HEADER, sizeof(tclt->header)); line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@60 PS39, Line 60: if (strncmp(TCPA_LOG_HEADER, (const char *)tclt->header, sizeof(tclt->header))) { line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@63 PS39, Line 63: strncpy((char *)tclt->header, TCPA_LOG_HEADER, sizeof(tclt->header)); line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/tpm/tspi/log.c@99 PS39, Line 99: void tcpa_log_add_table_entry(const char *name, const uint32_t pcr, const char *digest_type, line over 80 characters
https://review.coreboot.org/#/c/29563/39/src/security/vboot/secdata_tpm.c File src/security/vboot/secdata_tpm.c:
https://review.coreboot.org/#/c/29563/39/src/security/vboot/secdata_tpm.c@86 PS39, Line 86: return tpm_extend_pcr(pcr, "SHA256", buffer, size, TPM_PCR_GBB_HWID_NAME); line over 80 characters
-
build bot (Jenkins) (Code Review)