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

FAPI: Fix unneeded encryption. #2683

Conversation

JuergenReppSIT
Copy link
Member

A second session for encryption can only be used if the command has TPM2B parameters which can be encrypted. If no such parameter exists the TPM produces an error.
The call of the macro ENC_SESSION_IF_POLICY is replaces with ESYS_TR_NONE in these cases.

Signed-off-by: Juergen Repp juergen_repp@web.de

A second session for encryption can only be used if the command
has TPM2B parameters which can be encrypted. If no such parameter
exists the TPM produces an error.
The call of the macro ENC_SESSION_IF_POLICY is replaces with
ESYS_TR_NONE in these cases.

Signed-off-by: Juergen Repp <juergen_repp@web.de>
@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #2683 (550757c) into master (fe83020) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2683   +/-   ##
=======================================
  Coverage   82.57%   82.57%           
=======================================
  Files         368      368           
  Lines       42985    42980    -5     
=======================================
- Hits        35495    35491    -4     
+ Misses       7490     7489    -1     
Files Changed Coverage Δ
src/tss2-fapi/api/Fapi_Delete.c 89.51% <ø> (-0.04%) ⬇️
src/tss2-fapi/api/Fapi_NvIncrement.c 91.58% <ø> (-0.08%) ⬇️
src/tss2-fapi/api/Fapi_Provision.c 84.53% <ø> (-0.02%) ⬇️
src/tss2-fapi/fapi_util.c 82.90% <ø> (+0.03%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@JuergenReppSIT JuergenReppSIT merged commit b7bad34 into tpm2-software:master Aug 16, 2023
27 checks passed
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.

2 participants