]> granicus.if.org Git - php/commitdiff
Fix iteration limits in SXE test
authorNikita Popov <nikita.ppv@gmail.com>
Wed, 10 Jul 2019 08:57:02 +0000 (10:57 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Wed, 10 Jul 2019 09:09:47 +0000 (11:09 +0200)
Test doesn't look like it's specifically testing out of bounds
access, just two different iteration patterns, so make sure the
limits are correct.

ext/simplexml/tests/017.phpt

index ba42ac46a93ff637341a2c7f33baeb1938f0e9e8..42d6e3eccc1549b4f79bdb7b7a19c3731a08a81c 100644 (file)
@@ -34,12 +34,11 @@ function print_xml($xml) {
 }
 
 function print_xml2($xml) {
-  $persons = 2;
-  for ($i=0;$i<$persons;$i++) {
-    echo "person: ".$xml->person[$i]['name']."\n";
-       $children = 2;
-    for ($j=0;$j<$children;$j++) {
-      echo "  child: ".$xml->person[$i]->child[$j]['name']."\n";
+  for ($i=0;$i<count($xml->person);$i++) {
+    $person = $xml->person[$i];
+    echo "person: ".$person['name']."\n";
+    for ($j=0;$j<count($person->child);$j++) {
+      echo "  child: ".$person->child[$j]['name']."\n";
     }
   }
 }
@@ -54,7 +53,7 @@ echo "---22---\n";
 print_xml2(simplexml_load_string($xml1));
 ?>
 ===DONE===
---EXPECTF--
+--EXPECT--
 ---11---
 person: Joe
   child: Ann
@@ -75,12 +74,4 @@ person: Boe
 ---22---
 person: Joe
   child: Ann
-  child: 
-person: 
-
-Notice: Trying to get property 'child' of non-object in %s017.php on line %d
-  child: 
-
-Notice: Trying to get property 'child' of non-object in %s017.php on line %d
-  child: 
 ===DONE===