#21182 [Ctl->Csd]: range() change value of string argument

ID: 21182 Updated by: iliaa@php.net Reported By: lofa@forma.kharkov.ua -Status: Critical +Status: Closed Bug Type: Arrays related Operating System: FreeBSD PHP Version: 4.3.0-dev Assigned To: moriyoshi New Comment: This bug has been fixed in CVS. In case this was a PHP problem, snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. In case this was a documentation problem, the fix will show up soon at http://www.php.net/manual/. In case this was a PHP.net website problem, the change will show up on the PHP.net site and on the mirror sites in short time. Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2002-12-25 13:44:13] philip@php.net Confirmed in 4.3.0. If they are strings, $a turns into 40! If $a is integer, it stays 20 as expected. Btw if we have '$a = "$a = "24"; $b = "140";' then $a turns into 04. There seems to be sort of a pattern but it's kinda weird :) The second argument ($b) does not get modified and the following behavoir exists: range(20, "30"); // okay in 4.3.0-dev range("20", "30"); // not okay in 4.3.0-dev And as a holiday bonus, in HEAD either use results in a segfault with the following backtrace for CLI: rock:/tmp# php range.php a1: 20 FATAL: erealloc(): Unable to allocate 1515870815 bytes Segmentation fault (core dumped) #0 0x400b9c51 in kill () from /lib/libc.so.6 #1 0x0815f28c in _erealloc (ptr=0x8360cdc, size=1515870815, allow_failure=0, __zend_filename=0x81b8100 "/cvs/php4/Zend/zend_operators.c", __zend_lineno=1013, __zend_orig_filename=0x0, __zend_orig_lineno=0) at /cvs/php4/Zend/zend_alloc.c:298 #2 0x0816fb47 in add_string_to_string (result=0xbfffd5f4, op1=0xbfffd5f4, op2=0xbfffd748) at /cvs/php4/Zend/zend_operators.c:1013 #3 0x0818766b in execute (op_array=0x83677c4) at /cvs/php4/Zend/zend_execute.c:1463 #4 0x08174924 in zend_execute_scripts (type=8, retval=0x0, file_count=3) at /cvs/php4/Zend/zend.c:931 #5 0x08139553 in php_execute_script (primary_file=0xbffffa2c) at /cvs/php4/main/main.c:1693 #6 0x0818e817 in main (argc=2, argv=0xbffffaa4) at /cvs/php4/sapi/cli/php_cli.c:744 range.php looks like this: <?php $a = "20"; $b = "30"; echo "a1: $a\n"; $result = range($a, $b); echo "a2: $a : type : " . gettype($a) . "\n"; ?> ------------------------------------------------------------------------ [2002-12-25 10:43:54] derdik@ukr.net I searched range's source (php4/ext/standard/array.c) It seems that problem is in string for (; *low <= *high; (*low) += (unsigned int)lstep) { add_next_index_stringl(return_value, low, 1, 1); } += increments not pointer, but value, to which low points. Therefore for(;;) iterates values between first characters of high and low strings. ------------------------------------------------------------------------ [2002-12-25 10:02:09] lofa@forma.kharkov.ua I have the following code: <?PHP $a = "20"; $b = "30"; $result = range($a, $b); echo $a; ?> For some reason it display "40". I know that I have to write $result = range((int)$a, (int)$b); But why range() change value of argument? ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=21182&edit=1