Patch Set 70:

Now we are back at tcpa log replay. Not gonna happen. It impacts the security of a measured boot

So I've read through the discussion, and while there are a few potential security issues that I could imagine with TCPA log replay, I don't see how the very same issues wouldn't apply to the retroactive measuring that we have now.

So how is the replay approach worse than what we have right now? And, if it is a problem, isn't the proper solution to push tpm init earlier - which, in the replay case, seems to automatically solve any issues because any further measurement is done at the time the stage is loaded? Which is the same approach I'd take to close any gaps in the current retroactive measuring scheme.

From my point of view these two approaches seem to be equivalent in impact, with the major difference being a code design improvement by the change (as claimed by Julius, I haven't looked at the big design picture yet), so what's the problem with this change?

It's possible that I missed something big (as said I haven't gone through the entire design here, just the question of when measurements happen and from which data source), but if the TCPA log stored in (CAR) memory isn't trustworthy from bootblock to romstage, any assurance that measurements are worth anything is bound to go down the drain, no?

One thing that could be negatively affected is compliance to some bullshit firmware security standard, but even NIST SP 800-147 and SP 800-155 don't seem to say anything about how measuring is supposed to happen. 155 mentions the word "replay", but in a very specific context that doesn't apply here ("not just a replay of an earlier good response"). If it helps, we could probably call the early TCPA log a "PCR cache"?

View Change

To view, visit change 35077. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1fb376b4a8b98baffaee4d574937797bba1f8aee
Gerrit-Change-Number: 35077
Gerrit-PatchSet: 70
Gerrit-Owner: Bill XIE <persmule@hardenedlinux.org>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Bill XIE <persmule@hardenedlinux.org>
Gerrit-Reviewer: David Guckian <david.guckian@intel.com>
Gerrit-Reviewer: Frans Hendriks <fhendriks@eltan.com>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki@gmail.com>
Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@intel.com>
Gerrit-Reviewer: Werner Zeh <werner.zeh@siemens.com>
Gerrit-Reviewer: Wim Vervoorn <wvervoorn@eltan.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Joel Kitching <kitching@google.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-CC: Shawn C <citypw@hardenedlinux.org>
Gerrit-Comment-Date: Wed, 25 Mar 2020 22:06:04 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment