rpms/samba/devel samba3_idmap_ldap_memleak.patch, NONE, 1.1 samba.spec, 1.132, 1.133

[Simo Sorce (simo)]

Author: simo

Update of /cvs/pkgs/rpms/samba/devel
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv17709

Modified Files:
	samba.spec 
Added Files:
	samba3_idmap_ldap_memleak.patch 
Log Message:
make: `clog' is up to date.


samba3_idmap_ldap_memleak.patch:

--- NEW FILE samba3_idmap_ldap_memleak.patch ---
Author: vlendec
Revision: r24649
Modified: /branches/SAMBA_3_0_25/source/nsswitch/idmap_ldap.c source/nsswitch/idmap_ldap.c /branches/SAMBA_3_2_0/source/nsswitch/idmap_ldap.c
Added: 
Removed: 


Attempt to fix bug 4917. Simo, please check!

Thanks Patrick Rynhart for reporting this.


Index: source/nsswitch/idmap_ldap.c
===================================================================
--- source/nsswitch/idmap_ldap.c	(revision 24648)
+++ source/nsswitch/idmap_ldap.c	(revision 24649)
@@ -896,7 +896,7 @@
 	uidNumber = get_attr_key2string(idpool_attr_list, LDAP_ATTR_UIDNUMBER);
 	gidNumber = get_attr_key2string(idpool_attr_list, LDAP_ATTR_GIDNUMBER);
 
-	attr_list = get_attr_list(ctx, sidmap_attr_list);
+	attr_list = get_attr_list(memctx, sidmap_attr_list);
 
 	if ( ! ids[1]) {
 		/* if we are requested just one mapping use the simple filter */
@@ -1113,7 +1113,7 @@
 	uidNumber = get_attr_key2string(idpool_attr_list, LDAP_ATTR_UIDNUMBER);
 	gidNumber = get_attr_key2string(idpool_attr_list, LDAP_ATTR_GIDNUMBER);
 
-	attr_list = get_attr_list(ctx, sidmap_attr_list);
+	attr_list = get_attr_list(memctx, sidmap_attr_list);
 
 	if ( ! ids[1]) {
 		/* if we are requested just one mapping use the simple filter */



Index: samba.spec
===================================================================
RCS file: /cvs/pkgs/rpms/samba/devel/samba.spec,v
retrieving revision 1.132
retrieving revision 1.133
diff -u -r1.132 -r1.133
--- samba.spec	24 Aug 2007 14:08:12 -0000	1.132
+++ samba.spec	24 Aug 2007 18:02:18 -0000	1.133
@@ -2,7 +2,7 @@
 Name: samba
 Epoch: 0
 Version: 3.0.25c
-Release: 3%{?dist}
+Release: 4%{?dist}
 License: GPLv2+ and LGPLv2+
 Group: System Environment/Daemons
 URL: http://www.samba.org/
@@ -42,6 +42,7 @@
 Patch111: samba-3.0.13-smbclient.patch
 Patch200: samba-3.0.25rc1-inotifiy.patch
 Patch210: samba-3.0.25-imdap-basedn.patch
+Patch211: samba3_idmap_ldap_memleak.patch
 Patch300: samba3-glibc-open.patch
 
 
@@ -161,6 +162,7 @@
 %patch111 -p1 -b .smbclient
 %patch200 -p0 -b .inotify
 %patch210 -p1 -b .idmap_basedn
+%patch211 -p0 -b .idmap_ldap_memleak
 %patch300 -p0 -b .glibc_open
 
 # crap
@@ -644,6 +646,9 @@
 #%{_includedir}/libmsrpc.h
 
 %changelog
+* Tue Aug 24 2007 Simo Sorce <ssorce at redhat.com> 3.0.25c-4.fc8
+- add fix reported upstream for heavy idmap_ldap memleak
+
 * Tue Aug 21 2007 Simo Sorce <ssorce at redhat.com> 3.0.25c-3.fc8
 - fix a few places were "open" is used an interfere with the new glibc