Skip to content
This repository was archived by the owner on Aug 4, 2025. It is now read-only.

Conversation

@Hulongyin
Copy link
Contributor

This PR is to add event log replay and verify RTMR value with measurement.

@Hulongyin Hulongyin self-assigned this Jan 29, 2024
@Hulongyin Hulongyin added the feature new feature development label Jan 29, 2024
@Hulongyin Hulongyin force-pushed the add_eventlog branch 3 times, most recently from 4758f63 to d8c46d6 Compare January 30, 2024 01:49
@Hulongyin Hulongyin force-pushed the add_eventlog branch 12 times, most recently from 1f875ee to bf3615f Compare February 4, 2024 05:44
key (SWK) to encrypt the user key (wrapped_key).
- Encrypt the SWK by the public key from client (wrapped_swk).
For a key broker client, here is an example flow to get a key from KBS:
- Verify Event Logs.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

=> Get and replay all event logs, and verify by the measurement register.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, fixed.

if index not in measurement_dict or measurement_dict[index] == {}:
LOG.error("IMR[%d] verify failed, the replayed value from event logs doesn't exist",
index)
flag = False
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

return False

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, fixed.

LOG.info("IMR[%d] passed the verification.", index)
else:
LOG.error("IMR[%d] did not pass the verification.", index)
flag = False
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

return False

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, fixed.

Signed-off-by: Longyin Hu <longyin.hu@intel.com>
Copy link
Contributor

@dongx1x dongx1x left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@dongx1x dongx1x merged commit 1aa8f27 into intel:main Feb 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

feature new feature development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants