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 authorization session handling. 3.2.x #2726

Conversation

JuergenReppSIT
Copy link
Member

If no policy is used the sessíon2 in FAPI context must be initialized with ESYS_TR_NONE in the authorization function.
The macro ENC_SESSION_IF_POLIY did produce an invalid ESYS handle if session2 was not initialized.

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

@JuergenReppSIT JuergenReppSIT added this to the 3.2.3 milestone Dec 5, 2023
@JuergenReppSIT JuergenReppSIT marked this pull request as draft December 7, 2023 08:46
@JuergenReppSIT JuergenReppSIT marked this pull request as ready for review December 7, 2023 11:10
If no policy is used the sessíon2 in FAPI context must be initialized
with ESYS_TR_NONE in the authorization function.
The macro ENC_SESSION_IF_POLIY did produce an invalid ESYS handle if
session2 was not initialized.

Signed-off-by: Juergen Repp <juergen_repp@web.de>
@JuergenReppSIT JuergenReppSIT force-pushed the fapi-fix-second-session-3.2.x branch from 6c8fb1f to b4d1da5 Compare December 10, 2023 18:05
Copy link

codecov bot commented Dec 10, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (fb92f8d) 83.64% compared to head (b4d1da5) 83.63%.

Additional details and impacted files
@@            Coverage Diff             @@
##            3.2.x    #2726      +/-   ##
==========================================
- Coverage   83.64%   83.63%   -0.02%     
==========================================
  Files         349      349              
  Lines       38693    38694       +1     
==========================================
- Hits        32366    32360       -6     
- Misses       6327     6334       +7     

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

@AndreasFuchsTPM AndreasFuchsTPM merged commit d0e8252 into tpm2-software:3.2.x Dec 11, 2023
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants