[SVN] r637 -
/branches/cyrus23-farzy/cyrus-imapd-2.3-2.3.7/debian/patches/13-master_process_handling.dpatch
debian at incase.de
debian at incase.de
Fri Sep 29 15:02:01 UTC 2006
Author: farzy
Date: Fri Sep 29 17:01:59 2006
New Revision: 637
URL: https://mail.incase.de/viewcvs?rev=637&root=cyrus22&view=rev
Log:
Withdraw the patch which adds "free(Services[i].exec)", as it causes "cyrmaster" to segfault on reload in version 2.3.7 with a double free in glibc.
The other free() patch, "free(Services[i].name)", does not seem to cause any trouble.
Modified:
branches/cyrus23-farzy/cyrus-imapd-2.3-2.3.7/debian/patches/13-master_process_handling.dpatch
Modified: branches/cyrus23-farzy/cyrus-imapd-2.3-2.3.7/debian/patches/13-master_process_handling.dpatch
URL: https://mail.incase.de/viewcvs/branches/cyrus23-farzy/cyrus-imapd-2.3-2.3.7/debian/patches/13-master_process_handling.dpatch?rev=637&root=cyrus22&r1=636&r2=637&view=diff
==============================================================================
--- branches/cyrus23-farzy/cyrus-imapd-2.3-2.3.7/debian/patches/13-master_process_handling.dpatch (original)
+++ branches/cyrus23-farzy/cyrus-imapd-2.3-2.3.7/debian/patches/13-master_process_handling.dpatch Fri Sep 29 17:01:59 2006
@@ -146,18 +146,6 @@
#else
}
#endif /* HAVE_GETRLIMIT */
-@@ -1575,7 +1607,10 @@
-
- /* disable all services -
- they will be re-enabled if they appear in config file */
-- for (i = 0; i < nservices; i++) Services[i].exec = NULL;
-+ for (i = 0; i < nservices; i++) {
-+ if (Services[i].exec) free((void*)Services[i].exec);
-+ Services[i].exec = NULL;
-+ }
-
- /* read services */
- masterconf_getsection("SERVICES", &add_service, (void*) 1);
@@ -1590,13 +1625,18 @@
Services[i].stat[0], Services[i].stat[1]);
More information about the Pkg-Cyrus-imapd-Debian-devel
mailing list