Plug a memory leak in res_config_ldap.
authorSean Bright <sean@malleable.com>
Thu, 14 Jan 2010 23:13:02 +0000 (23:13 +0000)
committerSean Bright <sean@malleable.com>
Thu, 14 Jan 2010 23:13:02 +0000 (23:13 +0000)
(closes issue #16257)
Reported by: nito
Patches:
      issue16257_20100111.diff uploaded by seanbright (license 71)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@240271 65c4cc65-6c06-0410-ace0-fbb531ad65f3

res/res_config_ldap.c

index 3e0b8e1..8496d80 100644 (file)
@@ -378,8 +378,10 @@ static struct ast_variable **realtime_ldap_result_to_vars(struct ldap_table_conf
        ldap_entry = ldap_first_entry(ldapConn, ldap_result_msg);
 
        for (tot_count = 0; ldap_entry; tot_count++) { 
-               tot_count += semicolon_count_var(realtime_ldap_entry_to_var(table_config, ldap_entry));
+               struct ast_variable *tmp = realtime_ldap_entry_to_var(table_config, ldap_entry);
+               tot_count += semicolon_count_var(tmp);
                ldap_entry = ldap_next_entry(ldapConn, ldap_entry);
+               ast_variables_destroy(tmp);
        }
 
        if (entries_count_ptr) {