tony2001 Wed Jan 23 09:52:57 2008 UTC
Added files: (Branch: PHP_5_3)
/php-src/ext/simplexml/tests bug37076_1.phpt
Modified files:
/php-src/ext/simplexml simplexml.c
Log:
fix leak appearing when appending data to unnamed attribute
http://cvs.php.net/viewvc.cgi/php-src/ext/simplexml/simplexml.c?r1=1.151.2.22.2.35.2.7&r2=1.151.2.22.2.35.2.8&diff_format=u
Index: php-src/ext/simplexml/simplexml.c
diff -u php-src/ext/simplexml/simplexml.c:1.151.2.22.2.35.2.7
php-src/ext/simplexml/simplexml.c:1.151.2.22.2.35.2.8
--- php-src/ext/simplexml/simplexml.c:1.151.2.22.2.35.2.7 Tue Jan 22
20:42:21 2008
+++ php-src/ext/simplexml/simplexml.c Wed Jan 23 09:52:57 2008
@@ -18,7 +18,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: simplexml.c,v 1.151.2.22.2.35.2.7 2008/01/22 20:42:21 helly Exp $ */
+/* $Id: simplexml.c,v 1.151.2.22.2.35.2.8 2008/01/23 09:52:57 tony2001 Exp $ */
#ifdef HAVE_CONFIG_H
#include "config.h"
@@ -458,7 +458,7 @@
/* {{{ sxe_property_write()
*/
-static void sxe_prop_dim_write(zval *object, zval *member, zval *value,
zend_bool elements, zend_bool attribs, xmlNodePtr *pnewnode TSRMLS_DC)
+static int sxe_prop_dim_write(zval *object, zval *member, zval *value,
zend_bool elements, zend_bool attribs, xmlNodePtr *pnewnode TSRMLS_DC)
{
php_sxe_object *sxe;
xmlNodePtr node;
@@ -472,6 +472,7 @@
int test = 0;
int new_value = 0;
long cnt = 0;
+ int retval = SUCCESS;
zval tmp_zv, trim_zv, value_copy;
sxe = php_sxe_fetch_object(object TSRMLS_CC);
@@ -486,7 +487,7 @@
* and this is during runtime.
*/
php_error_docref(NULL TSRMLS_CC, E_ERROR, "Cannot
create unnamed attribute");
- return;
+ return FAILURE;
}
} else {
if (Z_TYPE_P(member) != IS_STRING) {
@@ -503,7 +504,7 @@
if (member == &tmp_zv) {
zval_dtor(&tmp_zv);
}
- return;
+ return FAILURE;
}
}
@@ -527,7 +528,7 @@
* and this is during runtime.
*/
php_error_docref(NULL TSRMLS_CC, E_ERROR, "Cannot
create unnamed attribute");
- return;
+ return FAILURE;
}
if (attribs && !node && sxe->iter.type == SXE_ITER_ELEMENT) {
node = xmlNewChild(mynode, mynode->ns, sxe->iter.name,
NULL);
@@ -565,7 +566,7 @@
zval_dtor(&tmp_zv);
}
zend_error(E_WARNING, "It is not yet possible
to assign complex types to %s", attribs ? "attributes" : "properties");
- return;
+ return FAILURE;
}
}
@@ -600,7 +601,7 @@
if (!member || Z_TYPE_P(member) == IS_LONG) {
if (node->type == XML_ATTRIBUTE_NODE) {
php_error_docref(NULL TSRMLS_CC,
E_ERROR, "Cannot create duplicate attribute");
- return;
+ return FAILURE;
}
if (sxe->iter.type == SXE_ITER_NONE) {
@@ -608,6 +609,7 @@
++counter;
if (member && Z_LVAL_P(member) > 0) {
php_error_docref(NULL
TSRMLS_CC, E_WARNING, "Cannot add element %s number %ld when only 0 such
elements exist", mynode->name, Z_LVAL_P(member));
+ retval = FAILURE;
}
} else if (member) {
newnode =
sxe_get_element_by_offset(sxe, Z_LVAL_P(member), node, &cnt);
@@ -644,6 +646,7 @@
}
} else if (counter > 1) {
php_error_docref(NULL TSRMLS_CC, E_WARNING, "Cannot
assign to an array of nodes (duplicate subnodes or attr detected)");
+ retval = FAILURE;
} else if (elements) {
if (!node) {
if (!member || Z_TYPE_P(member) == IS_LONG) {
@@ -654,12 +657,14 @@
} else if (!member || Z_TYPE_P(member) == IS_LONG) {
if (member && cnt < Z_LVAL_P(member)) {
php_error_docref(NULL TSRMLS_CC,
E_WARNING, "Cannot add element %s number %ld when only %ld such elements
exist", mynode->name, Z_LVAL_P(member), cnt);
+ retval = FAILURE;
}
newnode = xmlNewTextChild(mynode->parent,
mynode->ns, mynode->name, value ? (xmlChar *)Z_STRVAL_P(value) : NULL);
}
} else if (attribs) {
if (Z_TYPE_P(member) == IS_LONG) {
php_error_docref(NULL TSRMLS_CC, E_WARNING,
"Cannot change attribute number %ld when only %d attributes exist",
Z_LVAL_P(member), nodendx);
+ retval = FAILURE;
} else {
newnode = (xmlNodePtr)xmlNewProp(node, (xmlChar
*)Z_STRVAL_P(member), value ? (xmlChar *)Z_STRVAL_P(value) : NULL);
}
@@ -678,6 +683,7 @@
if (new_value) {
zval_ptr_dtor(&value);
}
+ return retval;
}
/* }}} */
@@ -714,7 +720,9 @@
if (node) {
return NULL;
}
- sxe_prop_dim_write(object, member, NULL, 1, 0, &node TSRMLS_CC);
+ if (sxe_prop_dim_write(object, member, NULL, 1, 0, &node TSRMLS_CC) !=
SUCCESS) {
+ return NULL;
+ }
type = SXE_ITER_NONE;
name = NULL;
@@ -2496,7 +2504,7 @@
{
php_info_print_table_start();
php_info_print_table_header(2, "Simplexml support", "enabled");
- php_info_print_table_row(2, "Revision", "$Revision: 1.151.2.22.2.35.2.7
$");
+ php_info_print_table_row(2, "Revision", "$Revision: 1.151.2.22.2.35.2.8
$");
php_info_print_table_row(2, "Schema support",
#ifdef LIBXML_SCHEMAS_ENABLED
"enabled");
http://cvs.php.net/viewvc.cgi/php-src/ext/simplexml/tests/bug37076_1.phpt?view=markup&rev=1.1
Index: php-src/ext/simplexml/tests/bug37076_1.phpt
+++ php-src/ext/simplexml/tests/bug37076_1.phpt
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
