#45166 [Ana->Csd]: substr() overflow changes

ID: 45166 Updated by: felipe@php.net Reported By: marrtins at dqdp dot net -Status: Analyzed +Status: Closed Bug Type: Strings related -PHP Version: 5.3 +PHP Version: 5.2 New Comment: This bug has been fixed in CVS. 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/. Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2008-11-05 17:14:06] philip@php.net It appears to be. The change came from the following NEWS entry: - Fixed bug #40754 (added substr() & substr_replace() overflow checks). (Ilia) Some concerns were expressed in #40754 about the change... I'm changing this bug to "String related" until we know for sure. ------------------------------------------------------------------------ [2008-11-05 17:06:22] bjori@php.net See also bug#40754 ------------------------------------------------------------------------ [2008-11-05 16:54:58] marrtins at dqdp dot net Seems to me it`s a bug, not documention problem. ------------------------------------------------------------------------ [2008-11-05 16:50:50] philip@php.net Question: Did 5.3.0 intentionally revert to earlier behaviour? And if not already, we need a test case in php-src for this. ------------------------------------------------------------------------ [2008-11-05 16:49:07] philip@php.net 4.3.1 - 5.2.1: cd 5.2.2 - 5.2.6: Boolean false 5.3.0a2: cd ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/45166 -- Edit this bug report at http://bugs.php.net/?id=45166&edit=1