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. #2725

Merged

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 bug backport Issues to be backported to old-stable labels Dec 5, 2023
@JuergenReppSIT JuergenReppSIT added this to the 4.0.2 milestone Dec 5, 2023
@JuergenReppSIT JuergenReppSIT marked this pull request as draft December 7, 2023 08:47
@JuergenReppSIT JuergenReppSIT marked this pull request as ready for review December 7, 2023 11:11
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 branch from f13cf17 to 0f63d1e Compare December 10, 2023 18:04
Copy link

codecov bot commented Dec 10, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (182b027) 82.59% compared to head (0f63d1e) 82.59%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2725      +/-   ##
==========================================
- Coverage   82.59%   82.59%   -0.01%     
==========================================
  Files         369      369              
  Lines       43124    43125       +1     
==========================================
  Hits        35619    35619              
- Misses       7505     7506       +1     

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

@AndreasFuchsTPM AndreasFuchsTPM merged commit a55983e into tpm2-software:master Dec 11, 2023
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Issues to be backported to old-stable bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants