summaryrefslogtreecommitdiff
path: root/main
diff options
context:
space:
mode:
authorNikita Popov <nikita.ppv@gmail.com>2020-10-16 17:29:22 +0200
committerNikita Popov <nikita.ppv@gmail.com>2020-10-16 17:29:22 +0200
commit808029c0498010b4b56cceb06276a9b0b6050587 (patch)
treef0ac012ec8a17b21f4465ec1659846682eab14fd /main
parent60ece88c28494945c9067448e8f9c4eeb18f8aff (diff)
parent97202d9d8401ce88b65d604df1f45af43c9a04a1 (diff)
downloadphp-git-808029c0498010b4b56cceb06276a9b0b6050587.tar.gz
Merge branch 'PHP-8.0'
* PHP-8.0: Make sure output start filename is not freed early
Diffstat (limited to 'main')
-rw-r--r--main/output.c19
-rw-r--r--main/php_output.h3
2 files changed, 16 insertions, 6 deletions
diff --git a/main/output.c b/main/output.c
index c895d3856e..cffadede80 100644
--- a/main/output.c
+++ b/main/output.c
@@ -103,16 +103,20 @@ static size_t (*php_output_direct)(const char *str, size_t str_len) = php_output
static void php_output_header(void)
{
if (!SG(headers_sent)) {
- if (!OG(output_start_filename)) {
+ if (!OG(output_start_filename_str)) {
if (zend_is_compiling()) {
- OG(output_start_filename) = ZSTR_VAL(zend_get_compiled_filename());
+ OG(output_start_filename_str) = zend_get_compiled_filename();
OG(output_start_lineno) = zend_get_compiled_lineno();
} else if (zend_is_executing()) {
- OG(output_start_filename) = zend_get_executed_filename();
+ OG(output_start_filename_str) = zend_get_executed_filename_ex();
OG(output_start_lineno) = zend_get_executed_lineno();
}
+ if (OG(output_start_filename_str)) {
+ zend_string_addref(OG(output_start_filename_str));
+ }
#if PHP_OUTPUT_DEBUG
- fprintf(stderr, "!!! output started at: %s (%d)\n", OG(output_start_filename), OG(output_start_lineno));
+ fprintf(stderr, "!!! output started at: %s (%d)\n",
+ ZSTR_VAL(OG(output_start_filename_str)), OG(output_start_lineno));
#endif
}
if (!php_header()) {
@@ -190,6 +194,11 @@ PHPAPI void php_output_deactivate(void)
}
zend_stack_destroy(&OG(handlers));
}
+
+ if (OG(output_start_filename_str)) {
+ zend_string_release(OG(output_start_filename_str));
+ OG(output_start_filename_str) = NULL;
+ }
}
/* }}} */
@@ -749,7 +758,7 @@ PHPAPI void php_output_set_implicit_flush(int flush)
* Get the file name where output has started */
PHPAPI const char *php_output_get_start_filename(void)
{
- return OG(output_start_filename);
+ return ZSTR_VAL(OG(output_start_filename_str));
}
/* }}} */
diff --git a/main/php_output.h b/main/php_output.h
index 60f389b2b5..eba63e6039 100644
--- a/main/php_output.h
+++ b/main/php_output.h
@@ -137,9 +137,10 @@ ZEND_BEGIN_MODULE_GLOBALS(output)
zend_stack handlers;
php_output_handler *active;
php_output_handler *running;
- const char *output_start_filename;
+ const char *output_start_filename; /* TODO: Unused, remove */
int output_start_lineno;
int flags;
+ zend_string *output_start_filename_str;
ZEND_END_MODULE_GLOBALS(output)
PHPAPI ZEND_EXTERN_MODULE_GLOBALS(output)