[coreboot-gerrit] [S] Change in coreboot[master]: commonlib/bsd/tpm_log_defs.h: Use C99 flexible arrays

[Maximilian Brune (Code Review)]

Attention is currently required from: Arthur Heymans, Elyes Haouas, Martin L Roth, Sergii Dmytruk.

Maximilian Brune has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76787?usp=email )

Change subject: commonlib/bsd/tpm_log_defs.h: Use C99 flexible arrays
......................................................................

Patch Set 5: Code-Review+1

(2 comments)

Patchset:

PS2:
> I have nothing against dropping the field and modifying `cbmem` and would prefer this solution over  […]

If we want to be ISO 99 compliant we have to remove the event property at the end of the tcpa_log_entry_header struct, since it is included in the tcpa_spec_entry struct. Therefore I would prefer the "ugly" conversion in cbmem (like it is implemented in the latest patchset)

File util/cbmem/cbmem.c:

https://review.coreboot.org/c/coreboot/+/76787/comment/3598eb82_ff4d6ea4 :
PS5, Line 915: 			uint8_t *ptr = (uint8_t *)log_entry + sizeof(*log_entry);
nit:
maybe use something like "event_ptr" instead of "ptr".