summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNathan Kinder <nkinder@redhat.com>2010-09-17 14:14:53 -0700
committerNathan Kinder <nkinder@redhat.com>2010-09-17 14:39:45 -0700
commit9433fc73f04520ce7f309fef6bcc4052146d34fe (patch)
tree0dd5d62472aa591895beb4192098ad0f72cf3851
parent6b3b9009af5c85b0be9dea36aca86c93972a20a4 (diff)
downloadds-9433fc73f04520ce7f309fef6bcc4052146d34fe.tar.gz
ds-9433fc73f04520ce7f309fef6bcc4052146d34fe.tar.xz
ds-9433fc73f04520ce7f309fef6bcc4052146d34fe.zip
Bug 630092 - (cov#12068) Resource leak in certmap code
The ldapu_propval_list_free() function was freeing the nodes in the list, but not the list itself. We need to free the list itself after all of the nodes have been freed.
-rw-r--r--lib/ldaputil/certmap.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/ldaputil/certmap.c b/lib/ldaputil/certmap.c
index 9c6b2bad..f3da425e 100644
--- a/lib/ldaputil/certmap.c
+++ b/lib/ldaputil/certmap.c
@@ -1472,6 +1472,7 @@ void ldapu_propval_list_free (void *propval_list)
{
LDAPUPropValList_t *list = (LDAPUPropValList_t *)propval_list;
ldapu_list_free(list, ldapu_propval_free);
+ free(list);
}
int ldapu_certmap_init (const char *config_file,