summaryrefslogtreecommitdiff
path: root/ext/com_dotnet/com_variant.c
diff options
context:
space:
mode:
authorChristoph M. Becker <cmbecker69@gmx.de>2020-06-29 19:06:56 +0200
committerChristoph M. Becker <cmbecker69@gmx.de>2020-06-29 19:06:56 +0200
commitbf6720d582de512fb6a8dd42c8515bc14e0953c5 (patch)
tree62ef1c490dbe33203d4350cc9978c7abfe50f5c1 /ext/com_dotnet/com_variant.c
parent43b5d5834197bef731a06f8ea025550df08c3fd1 (diff)
parent4af69b84779c1e13e0d98a8eac18a81c0d61f283 (diff)
downloadphp-git-bf6720d582de512fb6a8dd42c8515bc14e0953c5.tar.gz
Merge branch 'PHP-7.4'
* PHP-7.4: Fix #63208: BSTR to PHP string conversion not binary safe
Diffstat (limited to 'ext/com_dotnet/com_variant.c')
-rw-r--r--ext/com_dotnet/com_variant.c17
1 files changed, 3 insertions, 14 deletions
diff --git a/ext/com_dotnet/com_variant.c b/ext/com_dotnet/com_variant.c
index 72cf16b9e3..83940e93a1 100644
--- a/ext/com_dotnet/com_variant.c
+++ b/ext/com_dotnet/com_variant.c
@@ -94,7 +94,6 @@ bogus:
PHP_COM_DOTNET_API void php_com_variant_from_zval(VARIANT *v, zval *z, int codepage)
{
- OLECHAR *olestring;
php_com_dotnet_object *obj;
zend_uchar ztype = IS_NULL;
@@ -162,13 +161,7 @@ PHP_COM_DOTNET_API void php_com_variant_from_zval(VARIANT *v, zval *z, int codep
case IS_STRING:
V_VT(v) = VT_BSTR;
- olestring = php_com_string_to_olestring(Z_STRVAL_P(z), Z_STRLEN_P(z), codepage);
- if (CP_UTF8 == codepage) {
- V_BSTR(v) = SysAllocStringByteLen((char*)olestring, (UINT)(wcslen(olestring) * sizeof(OLECHAR)));
- } else {
- V_BSTR(v) = SysAllocStringByteLen((char*)olestring, (UINT)(Z_STRLEN_P(z) * sizeof(OLECHAR)));
- }
- efree(olestring);
+ V_BSTR(v) = php_com_string_to_bstr(Z_STR_P(z), codepage);
break;
case IS_RESOURCE:
@@ -234,12 +227,8 @@ PHP_COM_DOTNET_API int php_com_zval_from_variant(zval *z, VARIANT *v, int codepa
case VT_BSTR:
olestring = V_BSTR(v);
if (olestring) {
- size_t len;
- char *str = php_com_olestring_to_string(olestring,
- &len, codepage);
- ZVAL_STRINGL(z, str, len);
- // TODO: avoid reallocation???
- efree(str);
+ zend_string *str = php_com_bstr_to_string(olestring, codepage);
+ ZVAL_STR(z, str);
olestring = NULL;
}
break;