Browse Source

Bug 697027 - 2 - minor memory leaks found by Valgrind + TET

https://bugzilla.redhat.com/show_bug.cgi?id=697027
[Case 2]
Description: Adding slapi_ch_free_string to free inst->inst_parent_dir_name.
Noriko Hosoi 14 years ago
parent
commit
065d051e81
1 changed files with 3 additions and 2 deletions
  1. 3 2
      ldap/servers/slapd/back-ldbm/instance.c

+ 3 - 2
ldap/servers/slapd/back-ldbm/instance.c

@@ -381,9 +381,10 @@ ldbm_instance_destructor(void **arg)
     LDAPDebug(LDAP_DEBUG_ANY, "Destructor for instance %s called\n", 
               inst->inst_name, 0, 0);
 
-    slapi_ch_free((void **)&inst->inst_name);
+    slapi_ch_free_string(&inst->inst_name);
     PR_DestroyLock(inst->inst_config_mutex);
-    slapi_ch_free((void **)&inst->inst_dir_name);
+    slapi_ch_free_string(&inst->inst_dir_name);
+    slapi_ch_free_string(&inst->inst_parent_dir_name);
     PR_DestroyLock(inst->inst_db_mutex);
     PR_DestroyLock(inst->inst_handle_list_mutex);
     PR_DestroyLock(inst->inst_nextid_mutex);