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 2:
(11 comments)
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c File src/security/vboot/vboot_crtm.c:
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@104 PS2, Line 104: if (tpm_measure_region(&fmap, TPM_CRTM_PCR, "Read-only VPD") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@108 PS2, Line 108: if (tpm_measure_region(&fmap, TPM_CRTM_PCR, "Google Binary Blob") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@112 PS2, Line 112: if (tpm_measure_region(&fmap, TPM_CRTM_PCR, "Intel Flash Descriptor") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@116 PS2, Line 116: if (tpm_measure_region(&fmap, TPM_RUNTIME_DATA_PCR, "Intel ME") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@120 PS2, Line 120: if (tpm_measure_region(&fmap, TPM_RUNTIME_DATA_PCR, "EC firmware") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@124 PS2, Line 124: if (tpm_measure_region(&fmap, TPM_CRTM_PCR, "Intel GbE") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@128 PS2, Line 128: if (tpm_measure_region(&fmap, TPM_RUNTIME_DATA_PCR, "Platform Data") != TPM_SUCCESS) line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@193 PS2, Line 193: for (i = 0; i < sizeof(fmap_runtime_data) / sizeof(fmap_runtime_data[0]); i++) { line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@193 PS2, Line 193: for (i = 0; i < sizeof(fmap_runtime_data) / sizeof(fmap_runtime_data[0]); i++) { Prefer ARRAY_SIZE(fmap_runtime_data)
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@194 PS2, Line 194: if (!strncmp(fmap_runtime_data[i], name, sizeof(fmap_runtime_data[i]))) line over 80 characters
https://review.coreboot.org/#/c/31597/2/src/security/vboot/vboot_crtm.c@195 PS2, Line 195: return tpm_measure_region(rdev, TPM_RUNTIME_DATA_PCR, name); line over 80 characters