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

Bug #9093 Patches for Out-of-memory and Segfaults, with testsuite [take2]
Submitted: 2006-10-19 08:38 UTC
From: robbat2 at gentoo dot org Assigned: tacker
Status: Closed Package: File_Bittorrent (version 1.0.2)
PHP Version: Irrelevant OS: Linux
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 7 + 20 = ?

 
 [2006-10-19 08:38 UTC] robbat2 at gentoo dot org (Robin Johnson)
Description: ------------ This patch fixes many input cases where File_Bittorrent did not do the correct thing. Correct in this case being the Python and Perl testsuite, which I ported to this implementation, and also added a few other test cases that I ran into while working on fixing the issues. Of major signifiance: No more DoS effects from a list with an end marker, or for an non-string key in a dictionary. Patch is located at: http://www.orbis-terrarum.net/~robbat2/File_Bittorrent-1.0.2-TestSuite-Fixes-take2.patch It's had one further revision since the previous version submitted by email, for a case where '0:' was not correctly decoded. Test script: --------------- Please see the TestSuite contained inside the patch: http://www.orbis-terrarum.net/~robbat2/File_Bittorrent-1.0.2-TestSuite-Fixes-take2.patch Expected result: ---------------- Decoding should return FALSE for corrupt torrent. Actual result: -------------- Various different input leads to an infinte CPU-loop, an instant segfault, and rapid memory consumption.

Comments

 [2006-10-23 11:04 UTC] tacker at php dot net (Markus Tacker)
Thank you for your bug report. This issue has been fixed in the latest released version of the package, which you can download at http://pear.php.net/get/File_Bittorrent Thanks very much!