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.

Sign up for GitHub

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

Jump to bottom

FAPI: Fix usage of external PEM keys for PolicyAuthorize. 4.1.x #2857

Open
wants to merge 1 commit into
base: 4.1.x
Choose a base branch
from
Open

FAPI: Fix usage of external PEM keys for PolicyAuthorize. 4.1.x #2857

wants to merge 1 commit into from

Conversation

JuergenReppSIT
Copy link
Member

  • PolicyAuthorize with an external imported PEM key did not work if the default nameAlg for the imported key (sha1) was not equal to the default nameAlg in the current fapi profile. The nameAlg from the profile is now used for the imported key.
  • To prevent a possible double free after cleanup of a policy the NULL pointer is set for the reference to this policy.

@JuergenReppSIT
FAPI: Fix usage of external PEM keys for PolicyAuthorize. 4.1.x
c3b2bb9 
* PolicyAuthorize with an external imported PEM key did not work if
  the default nameAlg for the imported key (sha1) was not equal to
  the default nameAlg in the current fapi profile.
  The nameAlg from the profile is now used for the imported key.
* To prevent a possible double free after cleanup of a policy
  the NULL pointer is set for the reference to this policy.

Signed-off-by: Juergen Repp <juergen_repp@web.de>
@AndreasFuchsTPM AndreasFuchsTPM added this to the 4.1.4 milestone Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.1.4
Development

Successfully merging this pull request may close these issues.
2 participants
@JuergenReppSIT @AndreasFuchsTPM