diff --git a/ext/simplexml/simplexml.c b/ext/simplexml/simplexml.c index 483421e1931..bc9cec23b9d 100644 --- a/ext/simplexml/simplexml.c +++ b/ext/simplexml/simplexml.c @@ -1172,7 +1172,7 @@ static HashTable *sxe_get_prop_hash(zend_object *object, int is_debug) /* {{{ */ } } - if (node->type == XML_ELEMENT_NODE && (! match_ns(sxe, node, sxe->iter.nsprefix, sxe->iter.isprefix))) { + if (node->type == XML_ELEMENT_NODE && (! match_ns(sxe, node, sxe->iter.nsprefix, sxe->iter.isprefix)) || node->type == XML_ENTITY_DECL) { goto next_iter; } @@ -1871,7 +1871,7 @@ static int sxe_object_cast_ex(zend_object *readobj, zval *writeobj, int type) if (sxe->node && sxe->node->node) { if (sxe->node->node->children) { - contents = xmlNodeListGetString((xmlDocPtr) sxe->document->ptr, sxe->node->node->children, 1); + contents = xmlNodeListGetRawString((xmlDocPtr) sxe->document->ptr, sxe->node->node->children, 1); } } } diff --git a/ext/simplexml/tests/bug80852.phpt b/ext/simplexml/tests/bug80852.phpt new file mode 100644 index 00000000000..abda2deda1e --- /dev/null +++ b/ext/simplexml/tests/bug80852.phpt @@ -0,0 +1,21 @@ +--TEST-- +Bug #80852 (Stack-overflow when json_encode()'ing SimpleXMLElement) +--SKIPIF-- + +--FILE-- + ]>b&xee2;'; +$sxe = simplexml_load_string($xml); +var_dump(json_encode($sxe)); +var_dump($sxe); +?> +--EXPECT-- +string(11) "{"xee2":{}}" +object(SimpleXMLElement)#1 (1) { + ["xee2"]=> + object(SimpleXMLElement)#3 (0) { + } +}