Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.5.6

Bug #9100 Error with nested boundaries where inner boundary is a subset of the outer
Submitted: 2006-10-20 04:31 UTC
From: rocca at start dot ca Assigned: alan_k
Status: Closed Package: Mail_mimeDecode (version 1.3.1)
PHP Version: Irrelevant OS: All
Roadmaps: 1.6.0    
Subscription  
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes. If this is not your bug, you can add a comment by following this link. If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
2010-09-02 17:30 UTC
Package:
Bug Type:
Summary:
From: rocca at start dot ca
New email:
PHP Version: Package Version: OS:

 

 [2006-10-20 04:31 UTC] rocca at start dot ca (Peter Rocca)
Description: ------------ If the input data has an inner boundary that contains a subset of the outer boundary the _boundarySplit function will incorrectly parse the data. For example: Content-Type: multipart/mixed; boundary="abc" --abc Content-Type: multipart/mixed; boundary="abc_a" --abc_a inner --abc_a-- outer --abc-- ...will return 'abc_a' as being at the same level as 'abc' and additionally create some garbage nodes. Test script: --------------- function _boundarySplit($input, $boundary) { - $tmp = explode('--' . $boundary, $input); + $tmp = preg_split("/--$boundary(\b|--)/", $input);

Comments

 [2006-11-07 14:50 UTC] rocca at start dot ca
Changing PHP/OS version to Irrelevant. Also the 'test script' is actually the diff required to fix the problem. Ie, using preg_split instead of explode.
 [2010-09-02 17:30 UTC] alan_k (Alan Knowles)
-Status: Analyzed +Status: Closed -Assigned To: +Assigned To: alan_k
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.