From a076dbbf8eefee8c84b0c7af12f9d2819db11452 Mon Sep 17 00:00:00 2001 From: Endi Sukma Dewata Date: Fri, 17 Sep 2010 16:46:03 -0400 Subject: Bug 630092 - Coverity #11991: Resource leaks issues https://bugzilla.redhat.com/show_bug.cgi?id=630092 Description: The acllas__client_match_URL() has been modified to release the hostport before it returns. --- ldap/servers/plugins/acl/acllas.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ldap/servers/plugins/acl/acllas.c b/ldap/servers/plugins/acl/acllas.c index 6271fb1e..99b0281b 100644 --- a/ldap/servers/plugins/acl/acllas.c +++ b/ldap/servers/plugins/acl/acllas.c @@ -3606,9 +3606,6 @@ acllas__client_match_URL (struct acl_pblock *aclpb, char *n_clientdn, char *url /* dn was allocated in slapi_dn_normalize_ext */ slapi_ch_free_string(&dn); } - if (hostport && ('/' != *hostport)) { - slapi_ch_free_string(&hostport); - } rc = slapi_ldap_url_parse(normed, &ludp, 1, NULL); if (rc) { slapi_log_error( SLAPI_LOG_FATAL, plugin_name, @@ -3673,6 +3670,7 @@ acllas__client_match_URL (struct acl_pblock *aclpb, char *n_clientdn, char *url result = ACL_FALSE; done: + slapi_ch_free_string(&hostport); ldap_free_urldesc( ludp ); slapi_ch_free_string(&normed); slapi_filter_free ( f, 1 ) ; -- cgit