| author | Michael Krelin <hacker@klever.net> | 2011-04-27 10:10:00 (UTC) |
|---|---|---|
| committer | Michael Krelin <hacker@klever.net> | 2011-04-27 10:10:00 (UTC) |
| commit | d097b824b7fcad001c9581fb2e322bf3e3e5961d (patch) (side-by-side diff) | |
| tree | 5a1c55e82c4b0ff4acca276b8e1122547530cc0b /PumpKINDlg.cpp | |
| parent | 65981784977659461f08b48f537be9f9f77a2112 (diff) | |
| download | pumpkin-d097b824b7fcad001c9581fb2e322bf3e3e5961d.zip pumpkin-d097b824b7fcad001c9581fb2e322bf3e3e5961d.tar.gz pumpkin-d097b824b7fcad001c9581fb2e322bf3e3e5961d.tar.bz2 | |
fix for misleading error message about unexpected source
along with a double-freeing offense. When I fail to receive packet not only
I diagnose this unfortunate development, but also start talking about
packet from unexpected source which is kinda weird considering there's
no packet at all.
Signed-off-by: Michael Krelin <hacker@klever.net>
| -rwxr-xr-x[-rw-r--r--] | PumpKINDlg.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/PumpKINDlg.cpp b/PumpKINDlg.cpp index f41b69f..3ff1500 100644..100755 --- a/PumpKINDlg.cpp +++ b/PumpKINDlg.cpp @@ -580,48 +580,49 @@ int rv = socket->SendTo(udpBase(),length,(SOCKADDR*)saddr,sizeof(SOCKADDR_IN)); }
void CXferSocket::DoSelect(BOOL do_select)
{
if(m_Peer.sin_addr.s_addr!=INADDR_NONE)
AsyncSelect(FD_CLOSE|FD_READ|((m_Queue.IsEmpty()&&!do_select)?0:FD_WRITE));
}
void CXferSocket::OnReceive(int nErrorCode)
{
if(nErrorCode){
ASSERT(m_Daddy);
m_Daddy->LogLine(IDS_LOG_XFERRECEIVE);
return;
}
ASSERT(m_Daddy);
DWORD fionread = 0;
VERIFY(IOCtl(FIONREAD,&fionread));
tftp *p = tftp::Allocate(fionread);
ASSERT(p);
SOCKADDR_IN sin;
if(!p->Receive(this,fionread,&sin)){
m_Daddy->LogLine(IDS_LOG_XFERUDPRECEIVE);
delete p;
+ return;
}else
if(m_Peer.sin_addr.s_addr==INADDR_NONE){
m_Peer.sin_addr=sin.sin_addr;
m_Peer.sin_port=sin.sin_port;
}
BOOL alive = TRUE;
if(state==stateInit){
state=stateXfer;
m_Peer.sin_port=sin.sin_port;
UpdateList();
}
if(sin.sin_addr.s_addr!=m_Peer.sin_addr.s_addr || sin.sin_port!=m_Peer.sin_port){
m_Daddy->LogLine(IDS_LOG_XFERSOURCETID);
// *** Bounce it!
}else{
alive = OnTFTP(p);
}
delete p;
if(alive){
DoSelect();
ResetTimeout();
}
}
|