Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pcrauth status #3337

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

idesai
Copy link
Member

@idesai idesai commented Jan 11, 2024

Fixes #3330
Fixes #3333

@idesai
Copy link
Member Author

idesai commented Jan 11, 2024

TODO:

@idesai idesai force-pushed the add_pcrauth_status branch 3 times, most recently from 377c8db to 313e8f7 Compare January 12, 2024 17:40
… set

Signed-off-by: Imran Desai <imran.desai@intel.com>
Copy link

codecov bot commented Jan 12, 2024

Codecov Report

Attention: 141 lines in your changes are missing coverage. Please review.

Comparison is base (b7e1d2f) 74.38% compared to head (0ae8cee) 73.97%.

Files Patch % Lines
lib/tpm2_util.c 7.04% 66 Missing ⚠️
tools/tpm2_pcrsetauthvalue.c 1.53% 64 Missing ⚠️
lib/tpm2.c 0.00% 11 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3337      +/-   ##
==========================================
- Coverage   74.38%   73.97%   -0.42%     
==========================================
  Files         173      174       +1     
  Lines       23673    23835     +162     
==========================================
+ Hits        17610    17631      +21     
- Misses       6063     6204     +141     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AndreasFuchsTPM
Copy link
Member

Hmmmmm... you know that pcrsetauthvalue is forbidden for all PTP compatible TPMs, right ?
@idesai What's the use case ?

@idesai
Copy link
Member Author

idesai commented Jan 25, 2024

Hmmmmm... you know that pcrsetauthvalue is forbidden for all PTP compatible TPMs, right ? @idesai What's the use case ?

This is implementing the command spec for TPMs that implement the command.

@AndreasFuchsTPM
Copy link
Member

Please add big warnings to the manpage and also to the error printf's that this is not included in PC-Client compatible TPM; indicating that this is actually forbidden for PTP compliant TPMs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement tpm2_pcrsetauthvalue Getting PCR properties with tpm2_getcap
2 participants