Skip to content

Commit a37ad64

Browse files
ndosschendossche
committed
Merge branch 'PHP-8.2'
* PHP-8.2: Fix #80332: Completely broken array access functionality with DOMNamedNodeMap
2 parents 8904ac7 + 862487e commit a37ad64

File tree

7 files changed

+304
-107
lines changed

7 files changed

+304
-107
lines changed

‎ext/dom/namednodemap.c‎

Lines changed: 63 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
* Since:
3232
*/
3333

34-
static int get_namednodemap_length(dom_object *obj)
34+
int php_dom_get_namednodemap_length(dom_object *obj)
3535
{
3636
dom_nnodemap_object *objmap = (dom_nnodemap_object *) obj->ptr;
3737
if (!objmap) {
@@ -65,95 +65,74 @@ URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#core-
6565
*/
6666
int dom_namednodemap_length_read(dom_object *obj, zval *retval)
6767
{
68-
ZVAL_LONG(retval, get_namednodemap_length(obj));
68+
ZVAL_LONG(retval, php_dom_get_namednodemap_length(obj));
6969
return SUCCESS;
7070
}
7171

7272
/* }}} */
7373

74-
/* {{{ URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#core-ID-1074577549
75-
Since:
76-
*/
77-
PHP_METHOD(DOMNamedNodeMap, getNamedItem)
74+
xmlNodePtr php_dom_named_node_map_get_named_item(dom_nnodemap_object *objmap, const char *named, bool may_transform)
7875
{
79-
zval *id;
80-
int ret;
81-
size_t namedlen=0;
82-
dom_object *intern;
8376
xmlNodePtr itemnode = NULL;
84-
char *named;
85-
86-
dom_nnodemap_object *objmap;
87-
xmlNodePtr nodep;
88-
xmlNotation *notep = NULL;
89-
90-
id = ZEND_THIS;
91-
if (zend_parse_parameters(ZEND_NUM_ARGS(), "s", &named, &namedlen) == FAILURE) {
92-
RETURN_THROWS();
93-
}
94-
95-
intern = Z_DOMOBJ_P(id);
96-
97-
objmap = (dom_nnodemap_object *)intern->ptr;
98-
9977
if (objmap != NULL) {
10078
if ((objmap->nodetype == XML_NOTATION_NODE) ||
10179
objmap->nodetype == XML_ENTITY_NODE) {
10280
if (objmap->ht) {
10381
if (objmap->nodetype == XML_ENTITY_NODE) {
104-
itemnode = (xmlNodePtr)xmlHashLookup(objmap->ht, (xmlChar *) named);
82+
itemnode = (xmlNodePtr)xmlHashLookup(objmap->ht, (const xmlChar *) named);
10583
} else {
106-
notep = (xmlNotation *)xmlHashLookup(objmap->ht, (xmlChar *) named);
84+
xmlNotationPtr notep = xmlHashLookup(objmap->ht, (const xmlChar *) named);
10785
if (notep) {
108-
itemnode = create_notation(notep->name, notep->PublicID, notep->SystemID);
86+
if (may_transform) {
87+
itemnode = create_notation(notep->name, notep->PublicID, notep->SystemID);
88+
} else {
89+
itemnode = (xmlNodePtr) notep;
90+
}
10991
}
11092
}
11193
}
11294
} else {
113-
nodep = dom_object_get_node(objmap->baseobj);
95+
xmlNodePtr nodep = dom_object_get_node(objmap->baseobj);
11496
if (nodep) {
115-
itemnode = (xmlNodePtr)xmlHasProp(nodep, (xmlChar *) named);
97+
itemnode = (xmlNodePtr)xmlHasProp(nodep, (const xmlChar *) named);
11698
}
11799
}
118100
}
101+
return itemnode;
102+
}
119103

104+
void php_dom_named_node_map_get_named_item_into_zval(dom_nnodemap_object *objmap, const char *named, zval *return_value)
105+
{
106+
int ret;
107+
xmlNodePtr itemnode = php_dom_named_node_map_get_named_item(objmap, named, true);
120108
if (itemnode) {
121109
DOM_RET_OBJ(itemnode, &ret, objmap->baseobj);
122-
return;
123110
} else {
124-
RETVAL_NULL();
111+
RETURN_NULL();
125112
}
126113
}
127-
/* }}} end dom_namednodemap_get_named_item */
128114

129-
/* {{{ URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#core-ID-349467F9
115+
/* {{{ URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#core-ID-1074577549
130116
Since:
131117
*/
132-
PHP_METHOD(DOMNamedNodeMap, item)
118+
PHP_METHOD(DOMNamedNodeMap, getNamedItem)
133119
{
134-
zval *id;
135-
zend_long index;
136-
int ret;
137-
dom_object *intern;
138-
xmlNodePtr itemnode = NULL;
139-
140-
dom_nnodemap_object *objmap;
141-
xmlNodePtr nodep, curnode;
142-
int count;
120+
size_t namedlen;
121+
char *named;
143122

144-
id = ZEND_THIS;
145-
ZEND_PARSE_PARAMETERS_START(1, 1)
146-
Z_PARAM_LONG(index)
147-
ZEND_PARSE_PARAMETERS_END();
148-
if (index < 0 || ZEND_LONG_INT_OVFL(index)) {
149-
zend_argument_value_error(1, "must be between 0 and %d", INT_MAX);
123+
if (zend_parse_parameters(ZEND_NUM_ARGS(), "s", &named, &namedlen) == FAILURE) {
150124
RETURN_THROWS();
151125
}
152126

153-
intern = Z_DOMOBJ_P(id);
154-
155-
objmap = (dom_nnodemap_object *)intern->ptr;
127+
zval *id = ZEND_THIS;
128+
dom_nnodemap_object *objmap = Z_DOMOBJ_P(id)->ptr;
129+
php_dom_named_node_map_get_named_item_into_zval(objmap, named, return_value);
130+
}
131+
/* }}} end dom_namednodemap_get_named_item */
156132

133+
xmlNodePtr php_dom_named_node_map_get_item(dom_nnodemap_object *objmap, zend_long index)
134+
{
135+
xmlNodePtr itemnode = NULL;
157136
if (objmap != NULL) {
158137
if ((objmap->nodetype == XML_NOTATION_NODE) ||
159138
objmap->nodetype == XML_ENTITY_NODE) {
@@ -165,10 +144,10 @@ PHP_METHOD(DOMNamedNodeMap, item)
165144
}
166145
}
167146
} else {
168-
nodep = dom_object_get_node(objmap->baseobj);
147+
xmlNodePtr nodep = dom_object_get_node(objmap->baseobj);
169148
if (nodep) {
170-
curnode = (xmlNodePtr)nodep->properties;
171-
count = 0;
149+
xmlNodePtr curnode = (xmlNodePtr)nodep->properties;
150+
zend_long count = 0;
172151
while (count < index && curnode != NULL) {
173152
count++;
174153
curnode = (xmlNodePtr)curnode->next;
@@ -177,13 +156,38 @@ PHP_METHOD(DOMNamedNodeMap, item)
177156
}
178157
}
179158
}
159+
return itemnode;
160+
}
180161

162+
void php_dom_named_node_map_get_item_into_zval(dom_nnodemap_object *objmap, zend_long index, zval *return_value)
163+
{
164+
int ret;
165+
xmlNodePtr itemnode = php_dom_named_node_map_get_item(objmap, index);
181166
if (itemnode) {
182167
DOM_RET_OBJ(itemnode, &ret, objmap->baseobj);
183-
return;
168+
} else {
169+
RETURN_NULL();
170+
}
171+
}
172+
173+
/* {{{ URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#core-ID-349467F9
174+
Since:
175+
*/
176+
PHP_METHOD(DOMNamedNodeMap, item)
177+
{
178+
zend_long index;
179+
ZEND_PARSE_PARAMETERS_START(1, 1)
180+
Z_PARAM_LONG(index)
181+
ZEND_PARSE_PARAMETERS_END();
182+
if (index < 0 || ZEND_LONG_INT_OVFL(index)) {
183+
zend_argument_value_error(1, "must be between 0 and %d", INT_MAX);
184+
RETURN_THROWS();
184185
}
185186

186-
RETVAL_NULL();
187+
zval *id = ZEND_THIS;
188+
dom_object *intern = Z_DOMOBJ_P(id);
189+
dom_nnodemap_object *objmap = intern->ptr;
190+
php_dom_named_node_map_get_item_into_zval(objmap, index, return_value);
187191
}
188192
/* }}} end dom_namednodemap_item */
189193

@@ -254,7 +258,7 @@ PHP_METHOD(DOMNamedNodeMap, count)
254258
}
255259

256260
intern = Z_DOMOBJ_P(id);
257-
RETURN_LONG(get_namednodemap_length(intern));
261+
RETURN_LONG(php_dom_get_namednodemap_length(intern));
258262
}
259263
/* }}} end dom_namednodemap_count */
260264

‎ext/dom/nodelist.c‎

Lines changed: 23 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ static zend_always_inline void reset_objmap_cache(dom_nnodemap_object *objmap)
4949
objmap->cached_length = -1;
5050
}
5151

52-
static int get_nodelist_length(dom_object *obj)
52+
int php_dom_get_nodelist_length(dom_object *obj)
5353
{
5454
dom_nnodemap_object *objmap = (dom_nnodemap_object *) obj->ptr;
5555
if (!objmap) {
@@ -114,7 +114,7 @@ URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#ID-20
114114
*/
115115
int dom_nodelist_length_read(dom_object *obj, zval *retval)
116116
{
117-
ZVAL_LONG(retval, get_nodelist_length(obj));
117+
ZVAL_LONG(retval, php_dom_get_nodelist_length(obj));
118118
return SUCCESS;
119119
}
120120

@@ -131,37 +131,16 @@ PHP_METHOD(DOMNodeList, count)
131131
}
132132

133133
intern = Z_DOMOBJ_P(id);
134-
RETURN_LONG(get_nodelist_length(intern));
134+
RETURN_LONG(php_dom_get_nodelist_length(intern));
135135
}
136136
/* }}} end dom_nodelist_count */
137137

138-
/* }}} */
139-
140-
/* {{{ URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#ID-844377136
141-
Since:
142-
*/
143-
PHP_METHOD(DOMNodeList, item)
138+
void php_dom_nodelist_get_item_into_zval(dom_nnodemap_object *objmap, zend_long index, zval *return_value)
144139
{
145-
zval *id;
146-
zend_long index;
147140
int ret;
148-
bool cache_itemnode = false;
149-
dom_object *intern;
150141
xmlNodePtr itemnode = NULL;
151-
152-
dom_nnodemap_object *objmap;
153-
xmlNodePtr basep;
154-
int count = 0;
155-
156-
id = ZEND_THIS;
157-
ZEND_PARSE_PARAMETERS_START(1, 1)
158-
Z_PARAM_LONG(index)
159-
ZEND_PARSE_PARAMETERS_END();
160-
142+
bool cache_itemnode = false;
161143
if (index >= 0) {
162-
intern = Z_DOMOBJ_P(id);
163-
164-
objmap = (dom_nnodemap_object *)intern->ptr;
165144
if (objmap != NULL) {
166145
if (objmap->ht) {
167146
if (objmap->nodetype == XML_ENTITY_NODE) {
@@ -178,7 +157,7 @@ PHP_METHOD(DOMNodeList, item)
178157
return;
179158
}
180159
} else if (objmap->baseobj) {
181-
basep = dom_object_get_node(objmap->baseobj);
160+
xmlNodePtr basep = dom_object_get_node(objmap->baseobj);
182161
if (basep) {
183162
xmlNodePtr nodep = basep;
184163
/* For now we're only able to use cache for forward search.
@@ -203,6 +182,7 @@ PHP_METHOD(DOMNodeList, item)
203182
nodep = cached_obj_xml_node;
204183
}
205184
}
185+
int count = 0;
206186
if (objmap->nodetype == XML_ATTRIBUTE_NODE || objmap->nodetype == XML_ELEMENT_NODE) {
207187
if (restart) {
208188
nodep = nodep->children;
@@ -255,6 +235,22 @@ PHP_METHOD(DOMNodeList, item)
255235

256236
RETVAL_NULL();
257237
}
238+
239+
/* {{{ URL: http://www.w3.org/TR/2003/WD-DOM-Level-3-Core-20030226/DOM3-Core.html#ID-844377136
240+
Since:
241+
*/
242+
PHP_METHOD(DOMNodeList, item)
243+
{
244+
zend_long index;
245+
ZEND_PARSE_PARAMETERS_START(1, 1)
246+
Z_PARAM_LONG(index)
247+
ZEND_PARSE_PARAMETERS_END();
248+
249+
zval *id = ZEND_THIS;
250+
dom_object *intern = Z_DOMOBJ_P(id);
251+
dom_nnodemap_object *objmap = intern->ptr;
252+
php_dom_nodelist_get_item_into_zval(objmap, index, return_value);
253+
}
258254
/* }}} end dom_nodelist_item */
259255

260256
ZEND_METHOD(DOMNodeList, getIterator)

0 commit comments

Comments
 (0)