php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #52211 iconv() returns part of string on error
Submitted: 2010-06-30 15:27 UTC Modified: 2015-05-11 21:29 UTC
From: raphgivernaud at yahoo dot fr Assigned: felipe (profile)
Status: Closed Package: ICONV related
PHP Version: 5.3.2 OS: Windows Vista
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: raphgivernaud at yahoo dot fr
New email:
PHP Version: OS:

 

 [2010-06-30 15:27 UTC] raphgivernaud at yahoo dot fr
Description:
------------
When the function fails because of illegal characters, the function never returns false

Test script:
---------------
<?php

    $str = "PATHOLOGIES MÉDICO-CHIRUR. ADUL. PL";
    $str_iconv = iconv('CP850', 'ISO-8859-1', $str );
    var_dump($str_iconv);

?>

Expected result:
----------------
bool(false)

Actual result:
--------------
string(13) "PATHOLOGIES M"

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2010-07-01 03:08 UTC] felipe@php.net
-Summary: iconv return +Summary: iconv() returns part of string on error
 [2010-07-01 03:11 UTC] felipe@php.net
Automatic comment from SVN on behalf of felipe
Revision: http://svn.php.net/viewvc/?view=revision&amp;revision=300894
Log: - Fixed bug #52211 (iconv() returns part of string on error)
 [2010-07-01 03:15 UTC] felipe@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: felipe
 [2010-07-01 03:15 UTC] felipe@php.net
This bug has been fixed in SVN.

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.

This bug has been only fixed on trunk, because from what I can see, it might cause BC issue on 5.3 series.
 [2010-12-10 10:55 UTC] raphgivernaud at yahoo dot fr
Hello,

In version 5.3.3 and 5.3.4, the problem is still not
resolved
 [2010-12-10 12:20 UTC] felipe@php.net
Yes, as I mentioned, the fix was committed on trunk, not in 5.3 branch.
 [2010-12-10 12:39 UTC] raphgivernaud at yahoo dot fr
Ok, sorry
 [2011-12-18 18:36 UTC] ezyang@php.net
-Status: Closed +Status: Re-Opened
 [2011-12-18 18:36 UTC] ezyang@php.net
I don't think this bug was fixed correctly. In particular, iconv doesn't handle '//IGNORE' correctly now.

ezyang@javelin:~$ cat test.php
<?php var_dump(iconv("UTF-8", "ISO-8859-1//IGNORE", "\xE4\xB8\xAD"));
ezyang@javelin:~$ php --version
PHP 5.3.6-13ubuntu3.3 with Suhosin-Patch (cli) (built: Dec 13 2011 18:37:10) 
Copyright (c) 1997-2011 The PHP Group
Zend Engine v2.3.0, Copyright (c) 1998-2011 Zend Technologies
    with Xdebug v2.1.0, Copyright (c) 2002-2010, by Derick Rethans
ezyang@javelin:~$ php test.php
PHP Notice:  iconv(): Detected an illegal character in input string in /home/ezyang/test.php on line 1
PHP Stack trace:
PHP   1. {main}() /home/ezyang/test.php:0
PHP   2. iconv() /home/ezyang/test.php:1
string(0) ""
ezyang@javelin:~$ phpv 5.4.0RC3 --version
PHP 5.4.0RC3 (cli) (built: Dec 16 2011 15:14:12) 
Copyright (c) 1997-2011 The PHP Group
Zend Engine v2.4.0, Copyright (c) 1998-2011 Zend Technologies
ezyang@javelin:~$ phpv 5.4.0RC3 test.php
bool(false)
 [2012-01-08 12:34 UTC] pajoye@php.net
-Status: Re-Opened +Status: Closed
 [2012-01-08 12:34 UTC] pajoye@php.net
This is a totally unrelated issue to what you try to explain. And you already have 
a report for this, #48147
 [2015-05-08 19:11 UTC] stas@php.net
I note that the docs still reflect the old behavior...
 [2015-05-11 21:29 UTC] cmb@php.net
> I note that the docs still reflect the old behavior...

Which docs? <http://php.net/manual/en/function.iconv.php> seems to be fine.
 [2017-10-29 19:58 UTC] hho2002 at gmail dot com
Description:
------------
Fixed bug #52211 (iconv() returns part of string on error)
http://svn.php.net/viewvc/?view=revision&amp;revision=300894
http://git.php.net/?p=php-src.git;a=commit;h=e3fdf31b0bf3b7b799ea25ba5ec14be194813e0b

This fix make "iconv() with //IGNORE" return false

Test script:
---------------
<?php
    $str = "\xC3\xE6\xBC\xEC\xCC\xD6\xC9\xEE\xBF";
    $str_iconv = iconv('GBK', 'UTF-8//IGNORE', $str);
    var_dump($str_iconv);
?>

Expected result:
----------------
Notice: iconv(): Detected an incomplete multibyte character in input string in %s on line %d
string(12) "\xE9\x9D\xA2\xE6\xA3\x80\xE8\xAE\xA8\xE6\xB7\xB1"

Actual result:
--------------
Notice: iconv(): Detected an incomplete multibyte character in input string in %s on line %d
bool(false)
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Tue Mar 19 05:01:29 2024 UTC