[PATCH v6 1/3] efi_loader: Add check for event log passed from firmware

[Ilias Apalodimas]

Hi Heinrich,

> 
> > +
> > +	/*
> > +	 * initialize log area as 0xff so the OS can easily figure out the
> > +	 * last log entry
> 
> Where is this standardized?
> 
> EFI_TCG2_PROTOCOL.GetEventLog() returns EventLogLastEntry to indicate
> the last log entry.
> 

Indeed.  However while I was looking into EDK2 code I found a similar
comment [1].  Since EDK2 is used to boot windows as well I assumed that was
a good policy to keep.  


> > +	 */
> > +	memset(event_log.buffer, 0xff, TPM2_EVENT_LOG_SIZE);
> > +
> > +	/*
> > +	 * The log header is defined to be in SHA1 event log entry format.
> > +	 * Setup event header
> > +	 */

[...]

[1] https://github.com/tianocore/edk2/blob/master/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c#L1578

Cheers
/Ilias