Bug #65499 [Com]: json_decode reports invalid literal as valid JSON
| From: | remi@php.net | Date: | Fri, 23 Aug 2013 05:39:05 +0000 |
| Subject: | Bug #65499 [Com]: json_decode reports invalid literal as valid JSON | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-181138@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=65499&edit=1
ID: 65499
Comment by: remi@php.net
Reported by: m dot kurzyna at crystalpoint dot pl
Summary: json_decode reports invalid literal as valid JSON
Status: Closed
Type: Bug
Package: JSON related
Operating System: Linux
PHP Version: 5.5.2
Block user comment: N
Private report: N
New Comment:
As Fedora, Ubuntu and some other Linux distributions have switch to JSONC extension, you can report
this strictness change in https://github.com/remicollet/pecl-json-c/issues?state=open
Previous Comments:
------------------------------------------------------------------------
[2013-08-22 15:16:32] m dot kurzyna at crystalpoint dot pl
I'm at Ubuntu.
I've built current master (from git) and also didn't reproduce this behaviour so just as
you suggest - it seems to be packagers problem.
I will report to Ubuntu maintainers.
------------------------------------------------------------------------
[2013-08-22 11:34:15] yohgaki@php.net
It seems my fedora 19 x86_64 does this, while my PHP-5.5 branch don't on the
same machine. It seems packager's issue.
$ php
<?php
var_dump(
json_decode("9af5"),
json_last_error()
);
int(9)
int(0)
[yohgaki@dev PHP-5.5]$ ./php-bin
<?php
var_dump(
json_decode("9af5"),
json_last_error()
);
NULL
int(4)
------------------------------------------------------------------------
[2013-08-22 11:24:39] remi@php.net
What is the return of php -r 'echo phpversion("json");'
------------------------------------------------------------------------
[2013-08-22 11:22:19] mike@php.net
Cannot reproduce. Do you use Debian/Ubuntu/Fedora? Report there.
------------------------------------------------------------------------
[2013-08-22 08:15:54] m dot kurzyna at crystalpoint dot pl
To be more precise - introduced regression casts the string to integer so technically it takes all
first digits just like (int)"123abc" would do.
------------------------------------------------------------------------
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
https://bugs.php.net/bug.php?id=65499
--
Edit this bug report at https://bugs.php.net/bug.php?id=65499&edit=1