2008-01-04 Tatsuhiro Tsujikawa <tujikawa at rednoah dot com>

	Fixed segmentation fault when bad torrent metainfo is parsed.
	Added dynamic_cast properly to detect the misconfiguration of 
metainfo
	and then throw exception or skip it.
	* src/DefaultBtContext.{h, cc}
	* test/DefaultBtContextTest.cc
	* src/AnnounceList.cc
	* src/CompactPeerListProcessor.cc
	* src/message.h
	* src/DefaultBtAnnounce.{h, cc}
	* test/DefaultBtAnnounceTest.cc
	* src/BencodeVisitor.cc

1 files changed, 4 insertions, 2 deletions

diff --git a/src/CompactPeerListProcessor.cc b/src/CompactPeerListProcessor.cc
index d51bc9aa..a3ddc70f 100644
--- a/src/CompactPeerListProcessor.cc
+++ b/src/CompactPeerListProcessor.cc
@@ -41,8 +41,10 @@ bool CompactPeerListProcessor::canHandle(const MetaEntry* peersEntry) const {
 
 Peers CompactPeerListProcessor::extractPeer(const MetaEntry* peersEntry) {
   Peers peers;
-
-  const Data* peersData = (const Data*)peersEntry;
+  const Data* peersData = dynamic_cast<const Data*>(peersEntry);
+  if(!peersData) {
+    return peers;
+  }
   if(peersData->getLen()%6 == 0) {
     for(int32_t i = 0; i < peersData->getLen(); i += 6) {
       struct in_addr in;