rpms/sylpheed/devel sylpheed-2.4.7-ldap.patch, 1.1, 1.2 sylpheed.spec, 1.63, 1.64

Michael Schwendt (mschwendt) fedora-extras-commits at redhat.com
Mon Dec 17 15:02:08 UTC 2007


Author: mschwendt

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

Modified Files:
	sylpheed-2.4.7-ldap.patch sylpheed.spec 
Log Message:
* Mon Dec 17 2007 Michael Schwendt <mschwendt[AT]users.sf.net> - 2.4.7-4
- Upstreamed patch lead to discovery of similar memory leaks in syldap.c


sylpheed-2.4.7-ldap.patch:

Index: sylpheed-2.4.7-ldap.patch
===================================================================
RCS file: /cvs/pkgs/rpms/sylpheed/devel/sylpheed-2.4.7-ldap.patch,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- sylpheed-2.4.7-ldap.patch	1 Nov 2007 23:24:01 -0000	1.1
+++ sylpheed-2.4.7-ldap.patch	17 Dec 2007 15:02:00 -0000	1.2
@@ -1,21 +1,21 @@
 diff -Nur sylpheed-2.4.7-orig/src/syldap.c sylpheed-2.4.7/src/syldap.c
 --- sylpheed-2.4.7-orig/src/syldap.c	2006-08-31 10:21:21.000000000 +0200
-+++ sylpheed-2.4.7/src/syldap.c	2007-11-02 00:08:27.000000000 +0100
-@@ -602,11 +602,10 @@
++++ sylpheed-2.4.7/src/syldap.c	2007-12-17 15:50:27.000000000 +0100
+@@ -602,10 +602,10 @@
  			if( g_ascii_strcasecmp( attribute, SYLDAP_ATTR_DN ) == 0 ) {
  				listDN = syldap_add_single_value( ld, e, attribute );
  			}
-+			/* Free memory used to store attribute */
-+			ldap_memfree( attribute );
- 		}
+-		}
  
 -		/* Free memory used to store attribute */
 -		ldap_memfree( attribute );
--
++			/* Free memory used to store attribute */
++			ldap_memfree( attribute );
++		}
+ 
  		/* Format and add items to cache */
  		syldap_build_items_fl( ldapServer, listAddress, listFirst, listLast );
- 
-@@ -637,16 +636,15 @@
+@@ -637,16 +637,15 @@
  
  /* syldap_display_search_results() - updates the ui. this function is called from the
   * main thread (the thread running the GTK event loop). */
@@ -34,3 +34,43 @@
  }
  
  /* ============================================================================================ */
+@@ -821,8 +820,8 @@
+ 					}
+ 					ldap_value_free( vals );
+ 				}
++				ldap_memfree( attribute );
+ 			}
+-			ldap_memfree( attribute );
+ 			if( ber != NULL ) {
+ 				ber_free( ber, 0 );
+ 			}
+@@ -863,8 +862,8 @@
+ 						}
+ 						ldap_value_free( vals );
+ 					}
++					ldap_memfree( attribute );
+ 				}
+-				ldap_memfree( attribute );
+ 				if( ber != NULL ) {
+ 					ber_free( ber, 0 );
+ 				}
+@@ -950,8 +949,8 @@
+ 					}
+ 					ldap_value_free( vals );
+ 				}
++				ldap_memfree( attribute );
+ 			}
+-			ldap_memfree( attribute );
+ 			if( ber != NULL ) {
+ 				ber_free( ber, 0 );
+ 			}
+@@ -994,8 +993,8 @@
+ 						}
+ 						ldap_value_free( vals );
+ 					}
++					ldap_memfree( attribute );
+ 				}
+-				ldap_memfree( attribute );
+ 				if( ber != NULL ) {
+ 					ber_free( ber, 0 );
+ 				}


Index: sylpheed.spec
===================================================================
RCS file: /cvs/pkgs/rpms/sylpheed/devel/sylpheed.spec,v
retrieving revision 1.63
retrieving revision 1.64
diff -u -r1.63 -r1.64
--- sylpheed.spec	5 Dec 2007 20:38:04 -0000	1.63
+++ sylpheed.spec	17 Dec 2007 15:02:00 -0000	1.64
@@ -5,7 +5,7 @@
 Summary: GTK+ based, lightweight, and fast email client
 Name: sylpheed
 Version: 2.4.7
-Release: 3
+Release: 4
 License: GPLv2+
 URL: http://sylpheed.sraoss.jp/
 Group: Applications/Internet
@@ -90,6 +90,9 @@
 %{_mandir}/man1/*
 
 %changelog
+* Mon Dec 17 2007 Michael Schwendt <mschwendt[AT]users.sf.net> - 2.4.7-4
+- Upstreamed patch lead to discovery of similar memory leaks in syldap.c
+
 * Wed Dec 05 2007 Michael Schwendt <mschwendt[AT]users.sf.net> - 2.4.7-3
 - rebuilt for new openldap/openssl per request on fedora-devel-list
 - in desktop file use absolute path to icon file




More information about the fedora-extras-commits mailing list