diff options
author | Christoph M. Becker <cmbecker69@gmx.de> | 2020-11-14 15:03:58 +0100 |
---|---|---|
committer | Christoph M. Becker <cmbecker69@gmx.de> | 2020-11-22 14:02:29 +0100 |
commit | 221345a013cad854718b438c9dc869332853df63 (patch) | |
tree | 84f1cddd1396a75d077457f0c2d3793214b06e99 | |
parent | ed949a197333ee0fef5a4f6b497e8e39158730c4 (diff) | |
download | php-git-221345a013cad854718b438c9dc869332853df63.tar.gz |
Fix #77594: ob_tidyhandler is never reset
We reset to original INI value on request shutdown.
Closes GH-6425.
-rw-r--r-- | NEWS | 3 | ||||
-rw-r--r-- | ext/tidy/tests/bug77594.phpt | 15 | ||||
-rw-r--r-- | ext/tidy/tidy.c | 10 |
3 files changed, 27 insertions, 1 deletions
@@ -9,6 +9,9 @@ PHP NEWS (cmb) . Fixed bug #72964 (White space not unfolded for CC/Bcc headers). (cmb) +- Tidy: + . Fixed bug #77594 (ob_tidyhandler is never reset). (cmb) + 26 Nov 2020, PHP 7.4.13 - Core: diff --git a/ext/tidy/tests/bug77594.phpt b/ext/tidy/tests/bug77594.phpt new file mode 100644 index 0000000000..0d84a035f7 --- /dev/null +++ b/ext/tidy/tests/bug77594.phpt @@ -0,0 +1,15 @@ +--TEST-- +Bug #77594 (ob_tidyhandler is never reset) +--DESCRIPTION-- +Test is useful only with --repeat 2 (or more) +--SKIPIF-- +<?php +if (!extension_loaded('tidy')) die('skip tidy extension not available'); +?> +--FILE-- +<?php +ob_start('ob_tidyhandler'); +var_dump(ob_end_clean()); +?> +--EXPECT-- +bool(true) diff --git a/ext/tidy/tidy.c b/ext/tidy/tidy.c index a1a5184dd5..266e6510ef 100644 --- a/ext/tidy/tidy.c +++ b/ext/tidy/tidy.c @@ -240,6 +240,7 @@ static int php_tidy_output_handler(void **nothing, php_output_context *output_co static PHP_MINIT_FUNCTION(tidy); static PHP_MSHUTDOWN_FUNCTION(tidy); static PHP_RINIT_FUNCTION(tidy); +static PHP_RSHUTDOWN_FUNCTION(tidy); static PHP_MINFO_FUNCTION(tidy); static PHP_FUNCTION(tidy_getopt); @@ -500,7 +501,7 @@ zend_module_entry tidy_module_entry = { PHP_MINIT(tidy), PHP_MSHUTDOWN(tidy), PHP_RINIT(tidy), - NULL, + PHP_RSHUTDOWN(tidy), PHP_MINFO(tidy), PHP_TIDY_VERSION, PHP_MODULE_GLOBALS(tidy), @@ -1106,6 +1107,13 @@ static PHP_RINIT_FUNCTION(tidy) return SUCCESS; } +static PHP_RSHUTDOWN_FUNCTION(tidy) +{ + TG(clean_output) = INI_ORIG_BOOL("tidy.clean_output"); + + return SUCCESS; +} + static PHP_MSHUTDOWN_FUNCTION(tidy) { UNREGISTER_INI_ENTRIES(); |