From 3b33f12165d37bae1e19aae62955e88e717c259a Mon Sep 17 00:00:00 2001 From: Noriko Hosoi Date: Apr 15 2011 21:09:09 +0000 Subject: Bug 697027 - 11 - minor memory leaks found by Valgrind + TET https://bugzilla.redhat.com/show_bug.cgi?id=697027 [Case 11] Description: Fixing a copy & paste mistake: pathname1 -> pathname2. Adding PR_CloseDir to release dirhandle. --- diff --git a/ldap/servers/slapd/back-ldbm/dblayer.c b/ldap/servers/slapd/back-ldbm/dblayer.c index 363c62d..ca2a2df 100644 --- a/ldap/servers/slapd/back-ldbm/dblayer.c +++ b/ldap/servers/slapd/back-ldbm/dblayer.c @@ -5403,7 +5403,7 @@ dblayer_backup(struct ldbminfo *li, char *dest_dir, Slapi_Task *task) return_value = dblayer_copyfile(pathname1, pathname2, 0, priv->dblayer_file_mode); slapi_ch_free_string(&pathname1); - slapi_ch_free_string(&pathname1); + slapi_ch_free_string(&pathname2); slapi_ch_free_string(&changelog_destdir); } if (priv->dblayer_enable_transactions) { @@ -5700,8 +5700,8 @@ static int dblayer_copy_dirand_contents(char* src_dir, char* dst_dir, int mode, } if (0 > return_value) break; - } + PR_CloseDir(dirhandle); } return return_value; }