From cae6f1511ef81c645e1bf873b2ae975190ea5c4c Mon Sep 17 00:00:00 2001 From: Rich Megginson Date: Fri, 24 Jun 2011 19:44:05 -0600 Subject: memory leak in ipa_winsync_get_new_ds_user_dn_cb The new_dn_string passed into this function is malloc'd. It must be freed before we reassign the value. --- daemons/ipa-slapi-plugins/ipa-winsync/ipa-winsync.c | 1 + 1 file changed, 1 insertion(+) diff --git a/daemons/ipa-slapi-plugins/ipa-winsync/ipa-winsync.c b/daemons/ipa-slapi-plugins/ipa-winsync/ipa-winsync.c index 4b81a2ea..48c45c19 100644 --- a/daemons/ipa-slapi-plugins/ipa-winsync/ipa-winsync.c +++ b/daemons/ipa-slapi-plugins/ipa-winsync/ipa-winsync.c @@ -404,6 +404,7 @@ ipa_winsync_get_new_ds_user_dn_cb(void *cbdata, const Slapi_Entry *rawentry, } ldap_rdn2str(ldn[0], &rdn, LDAP_DN_FORMAT_LDAPV3); + slapi_ch_free_string(new_dn_string); *new_dn_string = slapi_ch_smprintf("%s,%s", rdn, slapi_sdn_get_dn(ds_suffix)); ldap_dnfree(ldn); ldap_memfree(rdn); -- cgit