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 13:
(5 comments)
https://review.coreboot.org/#/c/31597/13/src/security/vboot/vboot_crtm.c File src/security/vboot/vboot_crtm.c:
https://review.coreboot.org/#/c/31597/13/src/security/vboot/vboot_crtm.c@51 PS13, Line 51: FMAP_PREFIX "SI_EC // EC firmware") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/13/src/security/vboot/vboot_crtm.c@56 PS13, Line 56: FMAP_PREFIX "SI_GBE // Intel GbE") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/13/src/security/vboot/vboot_crtm.c@61 PS13, Line 61: FMAP_PREFIX "SI_PDR // Platform Data") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/13/src/security/vboot/vboot_crtm.c@241 PS13, Line 241: strncpy(prefix_name, FMAP_PREFIX, TCPA_PCR_HASH_NAME - 1); line over 80 characters
https://review.coreboot.org/#/c/31597/13/src/security/vboot/vboot_crtm.c@243 PS13, Line 243: name, TCPA_PCR_HASH_NAME - sizeof(FMAP_PREFIX) - 1); line over 80 characters