Re: [PATCH] tpm: Require that all digests are present in TCG_PCR_EVENT2 structures

Jarkko Sakkinen
2020-07-02T23:57:27Z

On Tue, Jun 30, 2020 at 01:33:21PM -0500, Tyler Hicks wrote:
> Jarkko, is this an ack from you?
>
> Is there anything I can do to help along this fix?
>
> I've spoke with two others that have poured through these specs to
> implement firmware event log parsers and they thought the change made
> sense.
>
> Tyler

I revisited the original patch and this stroke into my eye:

"This is true, for example, when firmware passes the event log to the
kernel via a reserved memory region described in device tree."

During this discussion you gave an explanation what can trigger the bug
but in the commit message nothing anchors to anything.

Please give a concrete example what can trigger the issue directly in
the commit message instead. It's obviously needed.

In addition, you could also rewrite the existing inline comment to be
something more reasonable to the context.

/Jarkko