php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #73322 mb_convert_variables detects recursive reference incorrectly
Submitted: 2016-10-14 14:48 UTC Modified: 2020-12-18 13:58 UTC
Votes:17
Avg. Score:4.5 ± 0.8
Reproduced:15 of 16 (93.8%)
Same Version:6 (40.0%)
Same OS:5 (33.3%)
From: townwear at gmail dot com Assigned: yohgaki (profile)
Status: Closed Package: mbstring related
PHP Version: Irrelevant OS: Irrelevant
Private report: No CVE-ID: None
View Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: townwear at gmail dot com
New email:
PHP Version: OS:

 

 [2016-10-14 14:48 UTC] townwear at gmail dot com 
Description:
------------
PHP-7.0.11 is no problem.
$ php -r '$x = ["a", "b"]; mb_convert_variables("UTF-8", "SJIS,EUC-JP", $x);'
// no output


PHP-7.0.12 is problem.
$ php -r '$x = ["a", "b"]; mb_convert_variables("UTF-8", "SJIS,EUC-JP", $x);'
PHP Warning:  mb_convert_variables(): Cannot handle recursive references in Command line code on line 1

Warning: mb_convert_variables(): Cannot handle recursive references in Command line code on line 1

Test script:
---------------
$ php -r '$x = ["a", "b"]; mb_convert_variables("UTF-8", "SJIS,EUC-JP", $x);'

Expected result:
----------------
// no output

Actual result:
--------------
PHP Warning:  mb_convert_variables(): Cannot handle recursive references in Command line code on line 1

Warning: mb_convert_variables(): Cannot handle recursive references in Command line code on line 1

Patches

Pull Requests

Pull requests:

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-10-14 15:57 UTC] cmb@php.net
-Status: Open +Status: Verified -Assigned To: +Assigned To: yohgaki
 [2016-10-14 15:57 UTC] cmb@php.net 
Confirmed: <https://3v4l.org/IA2Cp#v7012>. The regression has
been introduced by fixing bug #66964.

Yasuo, can you please have a look at this issue.
 [2016-10-14 23:36 UTC] yohgaki@php.net
-Status: Verified +Status: Assigned -Operating System: CentOS7 +Operating System: Irrelevant -PHP Version: 7.0.12 +PHP Version: Irrelevant
 [2016-10-14 23:36 UTC] yohgaki@php.net 
The code has obvious issue...
I'll fix it.
 [2016-10-15 01:07 UTC] yohgaki@php.net
-Summary: mb_convert_variables is problem +Summary: mb_convert_variables detects recursive reference incorrectly
 [2016-10-16 03:07 UTC] yohgaki@php.net 
PR is made. PHP 5.6 has some issues also, but I would like to leave 5.6 as it is now.
 [2016-12-19 14:46 UTC] msangapu at microsoft dot com 
Hello, we have customers on MS Azure using 5.6.28 and facing these warnings. I see from the comments that this will be fixed in PHP 7.x. Can you please pull this fix into 5.6.x branch as well?
 [2020-12-18 13:58 UTC] cmb@php.net
-Status: Assigned +Status: Closed
 [2020-12-18 13:58 UTC] cmb@php.net 
According to <https://3v4l.org/IA2Cp#v7012> this is fixed as of
PHP 7.3.0.  Older versions are out of active support for quite
some time now, so the fix will not be backported.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Nov 21 17:01:32 2024 UTC