Everybody seems to commit only in whatever branch they are working now? AFAICT, this patch is necessary also in PHP_5_2, why didn't you commit there?
--Jani 23.12.2009 11:40, Antony Dovgal wrote:
tony2001 Wed, 23 Dec 2009 09:40:28 +0000 Revision: http://svn.php.net/viewvc?view=revision&revision=292519 Log: fix minor leak in CGI sapi (happened once per child) Changed paths: U php/php-src/branches/PHP_5_3/sapi/cgi/fastcgi.c U php/php-src/trunk/sapi/cgi/fastcgi.c Modified: php/php-src/branches/PHP_5_3/sapi/cgi/fastcgi.c =================================================================== --- php/php-src/branches/PHP_5_3/sapi/cgi/fastcgi.c 2009-12-23 09:37:28 UTC (rev 292518) +++ php/php-src/branches/PHP_5_3/sapi/cgi/fastcgi.c 2009-12-23 09:40:28 UTC (rev 292519) @@ -255,6 +255,9 @@ zend_hash_destroy(&fcgi_mgmt_vars); } is_fastcgi = 0; + if (allowed_clients) { + free(allowed_clients); + } } #ifdef _WIN32 Modified: php/php-src/trunk/sapi/cgi/fastcgi.c =================================================================== --- php/php-src/trunk/sapi/cgi/fastcgi.c 2009-12-23 09:37:28 UTC (rev 292518) +++ php/php-src/trunk/sapi/cgi/fastcgi.c 2009-12-23 09:40:28 UTC (rev 292519) @@ -186,6 +186,9 @@ zend_hash_destroy(&fcgi_mgmt_vars); } is_fastcgi = 0; + if (allowed_clients) { + free(allowed_clients); + } } int fcgi_init(void)
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php