build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31597 )
Change subject: security/vboot: Add fmap measurements ......................................................................
Patch Set 14:
(5 comments)
https://review.coreboot.org/#/c/31597/14/src/security/vboot/vboot_crtm.c File src/security/vboot/vboot_crtm.c:
https://review.coreboot.org/#/c/31597/14/src/security/vboot/vboot_crtm.c@53 PS14, Line 53: FMAP_PREFIX "SI_EC // EC firmware") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/14/src/security/vboot/vboot_crtm.c@58 PS14, Line 58: FMAP_PREFIX "SI_GBE // Intel GbE") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/14/src/security/vboot/vboot_crtm.c@63 PS14, Line 63: FMAP_PREFIX "SI_PDR // Platform Data") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/14/src/security/vboot/vboot_crtm.c@243 PS14, Line 243: strncpy(prefix_name, FMAP_PREFIX, TCPA_PCR_HASH_NAME - 1); line over 80 characters
https://review.coreboot.org/#/c/31597/14/src/security/vboot/vboot_crtm.c@245 PS14, Line 245: name, TCPA_PCR_HASH_NAME - sizeof(FMAP_PREFIX) - 1); line over 80 characters