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

[ipa-4-4] ipa-kdb: support KDB DAL version 6.1 #606

Closed
wants to merge 1 commit into from

Conversation

tkrizek
Copy link
Contributor

@tkrizek tkrizek commented Mar 16, 2017

Rebased patch for ipa-4-4. It's already in F26/rawhide.

Oiginal PR: #410

@martbab
Copy link
Contributor

martbab commented Mar 17, 2017

Since the original ticket is in already closed milestone open a separate one for backport to 4-4 branch please.

DAL version 6.0 removed support for a callback to free principal.
This broke KDB drivers which had complex e_data structure within
the principal structure. As result, FreeIPA KDB driver was leaking
memory with DAL version 6.0 (krb5 1.15).

DAL version 6.1 added a special callback for freeing e_data structure.
See details at krb5/krb5#596

Restructure KDB driver code to provide this callback in case
we are built against DAL version that supports it. For DAL version
prior to 6.0 use this callback in the free_principal callback to
tidy the code.

Use explicit KDB version dependency in Fedora 26+ via BuildRequires.

With new DAL version, freeipa package will fail to build and
we'll have to add a support for new DAL version explicitly.

https://pagure.io/freeipa/issue/6776
@tkrizek tkrizek force-pushed the ipa-4-4-support-kdb-dal-6.1 branch from 6997574 to b436e0d Compare March 17, 2017 09:04
@tkrizek
Copy link
Contributor Author

tkrizek commented Mar 17, 2017

Created the ticket and linked it in the commit message.

@MartinBasti MartinBasti self-assigned this Mar 20, 2017
@MartinBasti MartinBasti added the ack Pull Request approved, can be merged label Mar 20, 2017
@MartinBasti
Copy link
Contributor

works for me

@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Mar 20, 2017
@MartinBasti
Copy link
Contributor

ipa-4-4:

  • 95daecb ipa-kdb: support KDB DAL version 6.1

@tkrizek tkrizek deleted the ipa-4-4-support-kdb-dal-6.1 branch September 15, 2017 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
5 participants