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

Reply via email to