From 767fa3dc02f39b64015539d9948874947cf20d5d Mon Sep 17 00:00:00 2001 From: "Christoph M. Becker" Date: Fri, 15 Feb 2019 18:50:13 +0100 Subject: Fix #77626: Persistence confusion in php_com_import_typelib() We apply only the most minimal fix here, and will cater to the unnecessary re-allocation for PHP-7.4. We don't need to add a regression test, since bug39606.phpt and bug77621.phpt already show the misbehavior. --- NEWS | 1 + ext/com_dotnet/com_typeinfo.c | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 852eb976e3..34c4434c9f 100644 --- a/NEWS +++ b/NEWS @@ -17,6 +17,7 @@ PHP NEWS - COM: . Fixed bug #77621 (Already defined constants are not properly reported). (cmb) + . Fixed bug #77626 (Persistence confusion in php_com_import_typelib()). (cmb) - Mbstring: . Fixed bug #77514 (mb_ereg_replace() with trailing backslash adds null byte). diff --git a/ext/com_dotnet/com_typeinfo.c b/ext/com_dotnet/com_typeinfo.c index 6da55f7477..6925e20379 100644 --- a/ext/com_dotnet/com_typeinfo.c +++ b/ext/com_dotnet/com_typeinfo.c @@ -184,7 +184,7 @@ PHP_COM_DOTNET_API int php_com_import_typelib(ITypeLib *TL, int mode, int codepa } const_name = php_com_olestring_to_string(bstr_ids, &len, codepage); - c.name = zend_string_init(const_name, len, 1); + c.name = zend_string_init(const_name, len, mode & CONST_PERSISTENT); // TODO: avoid reallocation??? efree(const_name); if(c.name == NULL) { @@ -200,7 +200,7 @@ PHP_COM_DOTNET_API int php_com_import_typelib(ITypeLib *TL, int mode, int codepa if (COMG(autoreg_verbose) && !compare_function(&results, &value, exists)) { php_error_docref(NULL, E_WARNING, "Type library constant %s is already defined", ZSTR_VAL(c.name)); } - zend_string_release_ex(c.name, 1); + zend_string_release_ex(c.name, mode & CONST_PERSISTENT); ITypeInfo_ReleaseVarDesc(TypeInfo, pVarDesc); continue; } -- cgit v1.2.1