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 3.2.x #2684

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 #2684 (3b813b0) into 3.2.x (db6d974) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            3.2.x    #2684      +/-   ##
==========================================
- Coverage   83.67%   83.67%   -0.01%     
==========================================
  Files         349      349              
  Lines       38687    38683       -4     
==========================================
- Hits        32373    32369       -4     
  Misses       6314     6314              
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.36% <ø> (-0.03%) ⬇️
src/tss2-fapi/fapi_util.c 82.15% <ø> (-0.01%) ⬇️

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

@JuergenReppSIT JuergenReppSIT merged commit 8fe6bcf into tpm2-software:3.2.x Aug 16, 2023
18 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