svn commit: samba r20880 - in branches: SAMBA_3_0/source/libads SAMBA_3_0_24/source/libads

jra at samba.org jra at samba.org
Thu Jan 18 19:18:44 GMT 2007


Author: jra
Date: 2007-01-18 19:18:43 +0000 (Thu, 18 Jan 2007)
New Revision: 20880

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=20880

Log:
Fix memory leak in new sitename code. You got *really*
close Guenther, then you forgot to use "key" :-) :-).
Jeremy.

Modified:
   branches/SAMBA_3_0/source/libads/dns.c
   branches/SAMBA_3_0_24/source/libads/dns.c


Changeset:
Modified: branches/SAMBA_3_0/source/libads/dns.c
===================================================================
--- branches/SAMBA_3_0/source/libads/dns.c	2007-01-18 18:21:18 UTC (rev 20879)
+++ branches/SAMBA_3_0/source/libads/dns.c	2007-01-18 19:18:43 UTC (rev 20880)
@@ -617,7 +617,7 @@
 
 	if (!sitename || (sitename && !*sitename)) {
 		DEBUG(5,("sitename_store: deleting empty sitename!\n"));
-		ret = gencache_del(sitename_key(realm));
+		ret = gencache_del(key);
 		SAFE_FREE(key);
 		return ret;
 	}

Modified: branches/SAMBA_3_0_24/source/libads/dns.c
===================================================================
--- branches/SAMBA_3_0_24/source/libads/dns.c	2007-01-18 18:21:18 UTC (rev 20879)
+++ branches/SAMBA_3_0_24/source/libads/dns.c	2007-01-18 19:18:43 UTC (rev 20880)
@@ -617,7 +617,7 @@
 
 	if (!sitename || (sitename && !*sitename)) {
 		DEBUG(5,("sitename_store: deleting empty sitename!\n"));
-		ret = gencache_del(sitename_key(realm));
+		ret = gencache_del(key);
 		SAFE_FREE(key);
 		return ret;
 	}



More information about the samba-cvs mailing list