From 71499fbe98b35b7253dbe158ba9d417adecefdc9 Mon Sep 17 00:00:00 2001
From: Andrew Tridgell <tridge@samba.org>
Date: Sun, 23 Apr 2000 07:40:13 +0000
Subject: avoided a memory leak in the ubi code by deleting a mangled cache
 entry before adding to ensure that we don't ever add a duplicate entry

this code can be removed when ubi gets fixed
(This used to be commit 062d79bf8ba5308803393c7b9140d1f77db3690f)
---
 source3/smbd/mangle.c | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

(limited to 'source3')

diff --git a/source3/smbd/mangle.c b/source3/smbd/mangle.c
index e47bcd896f..8ac91d49c8 100644
--- a/source3/smbd/mangle.c
+++ b/source3/smbd/mangle.c
@@ -465,7 +465,8 @@ static signed int cache_compare( ubi_btItemPtr ItemPtr, ubi_btNodePtr NodePtr )
  */
 static void cache_free_entry( ubi_trNodePtr WarrenZevon )
   {
-  free( WarrenZevon );
+	  ZERO_STRUCTP(WarrenZevon);
+	  free( WarrenZevon );
   } /* cache_free_entry */
 
 /* ************************************************************************** **
@@ -575,6 +576,15 @@ static void cache_mangled_name( char *mangled_name, char *raw_name )
   s2 = (char *)&(s1[mangled_len + 1]);
   (void)StrnCpy( s1, mangled_name, mangled_len );
   (void)StrnCpy( s2, raw_name,     raw_len );
+
+  /* possibly delete an old entry - this avoids a memory leak in the
+     ubi code to do with overwriting existing entries.
+
+     remove this test when ubi gets fixed */
+  if (ubi_cacheGet(mangled_cache, s1)) {
+	  ubi_cacheDelete(mangled_cache, s1);
+  }
+
   ubi_cachePut( mangled_cache, i, new_entry, s1 );
   } /* cache_mangled_name */
 
-- 
cgit