diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Prevent parsing invalid octal digits as octal numbers

Browse Source
This commit is contained in:
Julien Falque 2020-09-07 20:13:13 +02:00
parent 8e34978e4e
commit c7dcd82f03
No known key found for this signature in database
GPG Key ID: 6B13BB4B40DBD0E9
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/Symfony/Component/Yaml/Inline.php
View File

@ -759,16 +759,16 @@ class Inline
switch (true) {
case ctype_digit($scalar):
if ('0' === $scalar[0]) {
return octdec(preg_replace('/[^0-7]/', '', $scalar));
if (preg_match('/^0[0-7]+$/', $scalar)) {
return octdec($scalar);
}
$cast = (int) $scalar;
return ($scalar === (string) $cast) ? $cast : $scalar;
case '-' === $scalar[0] && ctype_digit(substr($scalar, 1)):
if ('0' === $scalar[1]) {
return -octdec(preg_replace('/[^0-7]/', '', substr($scalar, 1)));
if (preg_match('/^-0[0-7]+$/', $scalar)) {
return -octdec(substr($scalar, 1));
}
$cast = (int) $scalar;

4
src/Symfony/Component/Yaml/Tests/InlineTest.php
View File

@ -853,11 +853,11 @@ class InlineTest extends TestCase
public function testParsePositiveOctalNumberContainingInvalidDigits()
{
self::assertSame(342391, Inline::parse('0123456789'));
self::assertSame('0123456789', Inline::parse('0123456789'));
}
public function testParseNegativeOctalNumberContainingInvalidDigits()
{
self::assertSame(-342391, Inline::parse('-0123456789'));
self::assertSame('-0123456789', Inline::parse('-0123456789'));
}
}