#23690 [Opn->Asn]: Config_Container::toArray() Bug + FIX :-)
| From: | arnaud@php.net | Date: | Mon, 19 May 2003 06:51:55 +0000 |
| Subject: | #23690 [Opn->Asn]: Config_Container::toArray() Bug + FIX :-) | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-16450@lists.php.net to get a copy of this message | ||
ID: 23690
Updated by: arnaud@php.net
Reported By: info at rhalff dot com
-Status: Open
+Status: Assigned
Bug Type: PEAR related
Operating System: All
PHP Version: 4.3.2RC3
-Assigned To:
+Assigned To: mansion
Previous Comments:
------------------------------------------------------------------------
[2003-05-18 21:46:43] info at rhalff dot com
Somewhat more readable code:
Solution:
Original Code:
650: foreach ($newArr as $key => $value) {
651: if (isset($array[$this->name][$key])) {
652: // duplicate name/type
653: if (!isset($array[$this->name][$key][0])) {
654: $old = $array[$this->name][$key];
655: unset($array[$this->name][$key]);
656: $array[$this->name][$key][0] = $old;
657: }
658: $array[$this->name][$key][] = $value;
659: } elseif (isset($array[$this->name][$key][0])) {
660: $array[$this->name][$key][] = $value;
661: } else {
662: $array[$this->name][$key] = $value;
663: }
664: }
Fixed Code:
650: foreach ($newArr as $key => $value) {
651: if (isset($array[$this->name][$key])) {
652: // duplicate name/type
if(!is_array($array[$this->name][$key])) {
$old = $array[$this->name][$key];
unset($array[$this->name][$key]);
$array[$this->name][$key][0] = $old;
} else {
$array[$this->name][$key][] = $value;
}
661: } else {
662: $array[$this->name][$key] = $value;
663: }
664: }
------------------------------------------------------------------------
[2003-05-18 21:38:08] info at rhalff dot com
Reproduction of this bug:
1. Create the two files below:
-------------------------
php_snippet.ini:
[PHP]
extension = bcmath.so
extension = bz2.so
extension = calendar.so
extension = cdpf.so
extension = crack.so
extension = curl.so
-------------------------
<?php
require_once('Config.php');
$datasrc = 'php_snippet.ini';
$phpIni = new Config();
$root =& $phpIni->parseConfig($datasrc, 'inicommented');
if (PEAR::isError($root)) {
die($root->getMessage());
}
print_r($root->toArray());
?>
------------------------
2. Run test.php
The result will be:
Fatal error: [] operator not supported for strings in
/usr/lib/php/Config/Container.php on line 658
Solution:
Original Code:
650: foreach ($newArr as $key => $value) {
651: if
(isset($array[$this->name][$key])) {
652: // duplicate name/type
653: if
(!isset($array[$this->name][$key][0])) {
654: $old =
$array[$this->name][$key];
655:
unset($array[$this->name][$key]);
656:
$array[$this->name][$key][0] = $old;
657: }
658: $array[$this->name][$key][] =
$value;
659: } elseif
(isset($array[$this->name][$key][0])) {
660: $array[$this->name][$key][] =
$value;
661: } else {
662: $array[$this->name][$key] =
$value;
663: }
664: }
Fixed Code:
650: foreach ($newArr as $key => $value) {
651: if
(isset($array[$this->name][$key])) {
652: // duplicate name/type
if(!is_array($array[$this->name][$key])) {
$old =
$array[$this->name][$key];
unset($array[$this->name][$key]);
$array[$this->name][$key][0] =
$old;
} else {
$array[$this->name][$key][] =
$value;
}
661: } else {
662: $array[$this->name][$key] =
$value;
663: }
664: }
Explaination:
The original code checks if $array[$this->name][$key][0] is set
and falsly assumes that if [0] exists it will be an array.
In the example first it will set $array['PHP']['extension'] =
bcmath.so
The second time around it will detect that $array['PHP']['extension']
allready exists.
But $array['PHP']['extension'][0] will be 'b' as
$array['PHP']['extension'][1] will be 'c' and
$array['PHP']['extension'][2] will be 'm' and
$array['PHP']['extension'][3] will be 'a' and
$array['PHP']['extension'][4] will be 't' and
$array['PHP']['extension'][5] will be 'h' etc..
So when the code reaches $array[$this->name][$key][] = $value;
$array[$this->name][$key] will still be a string and this explains the
error message.
That's why the check should be like:
!is_array($array[$this->name][$key])
The rest of the changes in the Fixed code should be obvious.
Greetings,
Rob Halff
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=23690&edit=1