Bug #51250 [Opn->Csd]: iconv_mime_decode() does not ignore malformed Q-encoded words

From: Date: Wed, 01 Dec 2010 14:04:20 +0000
Subject: Bug #51250 [Opn->Csd]: iconv_mime_decode() does not ignore malformed Q-encoded words
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-156621@lists.php.net to get a copy of this message
Edit report at http://bugs.php.net/bug.php?id=51250&edit=1

 ID:                 51250
 Updated by:         iliaa@php.net
 Reported by:        tomlove at gmail dot com
 Summary:            iconv_mime_decode() does not ignore malformed
                     Q-encoded words
-Status:             Open
+Status:             Closed
 Type:               Bug
 Package:            ICONV related
 Operating System:   Linux / Windows
 PHP Version:        5.2.13
-Assigned To:        
+Assigned To:        iliaa
 Block user comment: N
 Private report:     N

 New Comment:

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.




Previous Comments:
------------------------------------------------------------------------
[2010-03-19 12:30:57] tomlove at gmail dot com

Correction - this sentence: "Accordingly, B-encoded encoded-words (as
per RFC 
2047) are silently ignored"  should read  "Accordingly, MALFORMED
B-encoded 
encoded-words are silently ignored"

------------------------------------------------------------------------
[2010-03-09 20:49:43] tomlove at gmail dot com

Description:
------------
When calling iconv_mime_decode() with $mode =
ICONV_MIME_DECODE_CONTINUE_ON_ERROR, 
the manual states that "iconv_mime_decode_headers() attempts to ignore
any 
grammatical errors and continue to process a given header."

Accordingly, B-encoded encoded-words (as per RFC 2047) are silently
ignored.

But when using Q (quoted-printable) encoding, encoded-words with illegal

characters (> 127) cause iconv_mime_decode() to return false as if 
ICONV_MIME_DECODE_CONTINUE_ON_ERROR was not specified. 

As such it is not resilient towards malformed headers. 

Test script:
---------------
$m = ICONV_MIME_DECODE_CONTINUE_ON_ERROR;

var_dump(iconv_mime_decode("Legal encoded-word: =?utf-8?B?Kg==?= .",
$m));
var_dump(iconv_mime_decode("Legal encoded-word: =?utf-8?Q?*?= .", $m));
var_dump(iconv_mime_decode("Illegal encoded-word:
=?utf-8?B?".chr(0xA1)."?= .", $m));
var_dump(iconv_mime_decode("Illegal encoded-word:
=?utf-8?Q?".chr(0xA1)."?= .", $m));

Expected result:
----------------
string(23) "Legal encoded-word: * ."
string(23) "Legal encoded-word: * ."
string(24) "Illegal encoded-word:  ."
string(24) "Illegal encoded-word:  ."

Actual result:
--------------
string(23) "Legal encoded-word: * ."
string(23) "Legal encoded-word: * ."
string(24) "Illegal encoded-word:  ."
bool(false)


------------------------------------------------------------------------



-- 
Edit this bug report at http://bugs.php.net/bug.php?id=51250&edit=1


Thread (3 messages)

« previous php.bugs (#156621) next »