[coreboot-gerrit] [S] Change in coreboot[main]: util/cbmem/cbmem.c: Avoid overflows when parsing TCG TPM logs

[Paul Menzel (Code Review)]

Attention is currently required from: Michał Żygowski.

Paul Menzel has posted comments on this change by Michał Żygowski. ( https://review.coreboot.org/c/coreboot/+/84926?usp=email )

Change subject: util/cbmem/cbmem.c: Avoid overflows when parsing TCG TPM logs
......................................................................

Patch Set 2:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84926/comment/1625f5cb_f807c396?usp... :
PS2, Line 9: utilit yassumed
utilit*y* assumed

https://review.coreboot.org/c/coreboot/+/84926/comment/79f183fd_d64208be?usp... :
PS2, Line 15: TEST=Dump TCG TPM1.2 event log on Dell OptiPlex 7010 and see
Please add the exact command you used.

File util/cbmem/cbmem.c:

https://review.coreboot.org/c/coreboot/+/84926/comment/84a61fb0_b4371ad7?usp... :
PS2, Line 890: 			fprintf(stderr, "Invalid TPM1.2 log entry overflowing cbmem area\n");
Would it be helpful to add the length/size numbers?

What should users do, if they see this message?