[coreboot-gerrit] [S] Change in coreboot[master]: security/tpm/tspi/log.c: fix strncpy() usage

[Sergii Dmytruk (Code Review)]

Attention is currently required from: Michał Żygowski, Christian Walter.

Sergii Dmytruk has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69708 )

Change subject: security/tpm/tspi/log.c: fix strncpy() usage
......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:
> This is still custom coreboot format right?

Yes.
> It is up to the parser to respect the format and be aware that the fields do not necessarily have to be NULL-terminated.

I don't think that was the original intent. If it was, the whole field would be used, but the code explicitly doesn't write last byte as if assuming it to be zero to guarantee string termination.