summaryrefslogtreecommitdiff
blob: 7659615afa27f76bc6bfeed6a473b9bdbe189e18 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
Index: modules/vuln-mydoom/MydoomDialogue.cpp
===================================================================
--- modules/vuln-mydoom/MydoomDialogue.cpp	(Revision 515)
+++ modules/vuln-mydoom/MydoomDialogue.cpp	(Arbeitskopie)
@@ -120,7 +120,7 @@
 				m_Download = new Download(msg->getRemoteHost(),"mydoom://foo/bar",msg->getRemoteHost(),"some triggerline");
 				m_Download->getDownloadBuffer()->addData((char *)m_Buffer->getData(),m_Buffer->getSize());
 				m_Buffer->clear();
-				return CL_ASSIGN;
+				return CL_ASSIGN_AND_DONE;
 			}
 		}
 		if (m_Buffer->getSize() > 128 )
@@ -130,6 +130,7 @@
 	case MYDOOM_FILETRANSFERR:
 		{
 			m_Download->getDownloadBuffer()->addData((char *)msg->getMsg(),msg->getSize());
+			return CL_ASSIGN_AND_DONE;
 		}
 		break;
 
@@ -196,7 +197,6 @@
 	if (m_Download != NULL)
 	{
     	g_Nepenthes->getSubmitMgr()->addSubmission(m_Download);
-		return CL_ASSIGN_AND_DONE;
 	}
 	return CL_DROP;
 }
Index: modules/vuln-bagle/BagleDialogue.cpp
===================================================================
--- modules/vuln-bagle/BagleDialogue.cpp	(Revision 515)
+++ modules/vuln-bagle/BagleDialogue.cpp	(Arbeitskopie)
@@ -158,6 +158,7 @@
 				m_State = BAGLE_BINARY;
 				m_Download = new Download(m_Socket->getRemoteHost(),"bagle://",m_Socket->getRemoteHost(),"bagle://foo/bar");
 				m_Download->getDownloadBuffer()->addData(msg->getMsg()+4,msg->getSize()-4);
+				return CL_ASSIGN_AND_DONE;
 			}
 		}
 		break;
@@ -165,6 +166,7 @@
 	case BAGLE_BINARY:
 		// FIXME m_MaxFileSize
 		m_Download->getDownloadBuffer()->addData(msg->getMsg(),msg->getSize());
+		return CL_ASSIGN_AND_DONE;
 		break;
 
 	}
@@ -232,8 +234,6 @@
 		if ( m_Download->getDownloadBuffer()->getSize() == m_FileSize )
 		{
 			g_Nepenthes->getSubmitMgr()->addSubmission(m_Download);
-			// destructor will delete it
-			return CL_ASSIGN_AND_DONE;
 		}
 	}
 	return CL_DROP;