lbarnaud Mon Aug 18 04:08:17 2008 UTC Added files: (Branch: PHP_5_3) /php-src/tests/lang bug45392.phpt
Modified files: /php-src/main output.c Log: MFH: Fixed bug #45392 (ob_start()/ob_end_clean() and memory_limit). http://cvs.php.net/viewvc.cgi/php-src/main/output.c?r1=1.167.2.3.2.4.2.7&r2=1.167.2.3.2.4.2.8&diff_format=u Index: php-src/main/output.c diff -u php-src/main/output.c:1.167.2.3.2.4.2.7 php-src/main/output.c:1.167.2.3.2.4.2.8 --- php-src/main/output.c:1.167.2.3.2.4.2.7 Sat Aug 2 04:46:07 2008 +++ php-src/main/output.c Mon Aug 18 04:08:17 2008 @@ -18,7 +18,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: output.c,v 1.167.2.3.2.4.2.7 2008/08/02 04:46:07 felipe Exp $ */ +/* $Id: output.c,v 1.167.2.3.2.4.2.8 2008/08/18 04:08:17 lbarnaud Exp $ */ #include "php.h" #include "ext/standard/head.h" @@ -336,7 +336,13 @@ PHPAPI void php_end_ob_buffers(zend_bool send_buffer TSRMLS_DC) { while (OG(ob_nesting_level)!=0) { - php_end_ob_buffer(send_buffer, 0 TSRMLS_CC); + /* in case of unclean_shutdown, do not output the buffer if it is not + * meant to be until end of script or ob_end_*() call */ + if (CG(unclean_shutdown) && !OG(active_ob_buffer).chunk_size) { + php_end_ob_buffer(0, 0 TSRMLS_CC); + } else { + php_end_ob_buffer(send_buffer, 0 TSRMLS_CC); + } } } /* }}} */ http://cvs.php.net/viewvc.cgi/php-src/tests/lang/bug45392.phpt?view=markup&rev=1.1 Index: php-src/tests/lang/bug45392.phpt +++ php-src/tests/lang/bug45392.phpt --TEST-- Bug #45392 (ob_start()/ob_end_clean() and memory_limit) --INI-- display_errors=stderr --FILE-- <?php echo __LINE__ . "\n"; ini_set('memory_limit', 100); ob_start(NULL, 10); echo __LINE__ ."\n"; ob_start(); $i = 0; while($i++ < 5000) { echo str_repeat("may not be displayed ", 42); } ob_end_flush(); ob_end_clean(); ?> --EXPECTF-- 2 Fatal error: Allowed memory size of %d bytes exhausted%s 5 -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php