archive/php-src
1
0
Fork 0
mirror of https://github.com/php/php-src.git synced 2025-08-18 06:58:55 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fixed malloc() / emalloc() bug

Browse source
This commit is contained in:
Harald Radi 2001-06-15 21:48:05 +00:00
parent 499334e498
commit 85bb9e77e4
2 changed files with 42 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

26
ext/com/COM.c
View file

@ -1239,7 +1239,7 @@ PHPAPI pval php_COM_get_property_handler(zend_property_reference *property_refer
{
efree(obj_prop);
obj_prop = NULL;
php_variant_to_pval(var_result, &return_value, TRUE, codepage);
php_variant_to_pval(var_result, &return_value, FALSE, codepage);
}
pval_destructor(&overloaded_property->element);
@ -1633,6 +1633,7 @@ static int php_COM_load_typelib(ITypeLib *TypeLib, int mode)
BSTR bstr_ids;
char *ids;
zend_constant c;
zval exists, results;
TypeInfo->lpVtbl->GetNames(TypeInfo, pVarDesc->memid, &bstr_ids, 1, &NameCount);
if(NameCount!=1)
@ -1640,15 +1641,30 @@ static int php_COM_load_typelib(ITypeLib *TypeLib, int mode)
j++;
continue;
}
LocalFree(bstr_ids);
ids = php_OLECHAR_to_char(bstr_ids, NULL, 1, codepage);
SysFreeString(bstr_ids);
c.name_len = strlen(ids)+1;
c.name = ids;
php_variant_to_pval(pVarDesc->lpvarValue, &c.value, 1, codepage);
if (zend_get_constant(c.name, c.name_len-1, &exists))
{
/* Oops, it already exists. No problem if it is defined as the same value */
/* Check to see if they are the same */
if (!compare_function(&results, &c.value, &exists))
{
php_error(E_WARNING,"Type library value %s is already defined and has a different value", c.name);
}
free(ids);
j++;
continue;
}
php_variant_to_pval(pVarDesc->lpvarValue, &c.value, FALSE, codepage);
c.flags = mode;
zend_register_constant(&c ELS_CC);
/*printf("%s -> %ld\n", ids, pVarDesc->lpvarValue->lVal);*/
/* Before registering the contsnt, let's see if we can find it */
{
zend_register_constant(&c ELS_CC);
}
j++;
}
TypeInfo->lpVtbl->Release(TypeInfo);

26
ext/rpc/com/com_wrapper.c
View file

@ -1239,7 +1239,7 @@ PHPAPI pval php_COM_get_property_handler(zend_property_reference *property_refer
{
efree(obj_prop);
obj_prop = NULL;
php_variant_to_pval(var_result, &return_value, TRUE, codepage);
php_variant_to_pval(var_result, &return_value, FALSE, codepage);
}
pval_destructor(&overloaded_property->element);
@ -1633,6 +1633,7 @@ static int php_COM_load_typelib(ITypeLib *TypeLib, int mode)
BSTR bstr_ids;
char *ids;
zend_constant c;
zval exists, results;
TypeInfo->lpVtbl->GetNames(TypeInfo, pVarDesc->memid, &bstr_ids, 1, &NameCount);
if(NameCount!=1)
@ -1640,15 +1641,30 @@ static int php_COM_load_typelib(ITypeLib *TypeLib, int mode)
j++;
continue;
}
LocalFree(bstr_ids);
ids = php_OLECHAR_to_char(bstr_ids, NULL, 1, codepage);
SysFreeString(bstr_ids);
c.name_len = strlen(ids)+1;
c.name = ids;
php_variant_to_pval(pVarDesc->lpvarValue, &c.value, 1, codepage);
if (zend_get_constant(c.name, c.name_len-1, &exists))
{
/* Oops, it already exists. No problem if it is defined as the same value */
/* Check to see if they are the same */
if (!compare_function(&results, &c.value, &exists))
{
php_error(E_WARNING,"Type library value %s is already defined and has a different value", c.name);
}
free(ids);
j++;
continue;
}
php_variant_to_pval(pVarDesc->lpvarValue, &c.value, FALSE, codepage);
c.flags = mode;
zend_register_constant(&c ELS_CC);
/*printf("%s -> %ld\n", ids, pVarDesc->lpvarValue->lVal);*/
/* Before registering the contsnt, let's see if we can find it */
{
zend_register_constant(&c ELS_CC);
}
j++;
}
TypeInfo->lpVtbl->Release(TypeInfo);