diff options
| author | sbose <sbose@ipa18-devel.ipa18.devel> | 2013-02-11 15:39:22 +0100 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2013-02-14 10:11:26 +0100 |
| commit | 3f8778890e1a62d251f7069ead981d088c014b16 (patch) | |
| tree | a11961f4c34e2999093ae6d8e2db521fd7e6c830 /daemons/ipa-kdb | |
| parent | dfad4396ffa9d47dc9effb832f05f1b9f7c36582 (diff) | |
| download | freeipa-3f8778890e1a62d251f7069ead981d088c014b16.tar.gz freeipa-3f8778890e1a62d251f7069ead981d088c014b16.tar.xz freeipa-3f8778890e1a62d251f7069ead981d088c014b16.zip | |
ipa-kdb: Free talloc autofree context when module is closed
Currently kdb5kdc crashes on exit if the ipadb KDB modules is loaded and trusts
are configured. The reason is the talloc autofree context which get initialised
during the ndr_push_union_blob() call. On exit the KDC module is unloaded an
later on atexit() tries to free the context, but all related symbols are
already unloaded with the module.
This patch frees the talloc autofree context during the cleanup routine of the
module. Since this is called only at exit and not during normal operations this
is safe even if other KDC plugins use the talloc autofree context, e.g. via
some Samba libraries, as well.
Fixes https://fedorahosted.org/freeipa/ticket/3410
Diffstat (limited to 'daemons/ipa-kdb')
| -rw-r--r-- | daemons/ipa-kdb/ipa_kdb.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/daemons/ipa-kdb/ipa_kdb.c b/daemons/ipa-kdb/ipa_kdb.c index 55a932abd..2a344dc69 100644 --- a/daemons/ipa-kdb/ipa_kdb.c +++ b/daemons/ipa-kdb/ipa_kdb.c @@ -20,6 +20,8 @@ * along with this program. If not, see <http://www.gnu.org/licenses/>. */ +#include <talloc.h> + #include "ipa_kdb.h" struct ipadb_context *ipadb_get_context(krb5_context kcontext) @@ -456,6 +458,7 @@ static krb5_error_code ipadb_fini_module(krb5_context kcontext) ipactx = ipadb_get_context(kcontext); ipadb_context_free(kcontext, &ipactx); + talloc_free(talloc_autofree_context()); return 0; } |