Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
network: Stricter handling of split packets
  • Loading branch information
sfan5 committed Aug 15, 2019
1 parent c449116 commit 2db0e93
Showing 1 changed file with 11 additions and 4 deletions.
15 changes: 11 additions & 4 deletions src/network/connection.cpp
Expand Up @@ -417,6 +417,11 @@ SharedBuffer<u8> IncomingSplitBuffer::insert(const BufferedPacket &p, bool relia
<< std::endl;
return SharedBuffer<u8>();
}
if (chunk_num >= chunk_count) {
errorstream << "IncomingSplitBuffer::insert(): chunk_num=" << chunk_num
<< " >= chunk_count=" << chunk_count << std::endl;
return SharedBuffer<u8>();
}

// Add if doesn't exist
if (m_buf.find(seqnum) == m_buf.end()) {
Expand All @@ -425,10 +430,12 @@ SharedBuffer<u8> IncomingSplitBuffer::insert(const BufferedPacket &p, bool relia

IncomingSplitPacket *sp = m_buf[seqnum];

if (chunk_count != sp->chunk_count)
LOG(derr_con<<"Connection: WARNING: chunk_count="<<chunk_count
<<" != sp->chunk_count="<<sp->chunk_count
<<std::endl);
if (chunk_count != sp->chunk_count) {
errorstream << "IncomingSplitBuffer::insert(): chunk_count="
<< chunk_count << " != sp->chunk_count=" << sp->chunk_count
<< std::endl;
return SharedBuffer<u8>();
}
if (reliable != sp->reliable)
LOG(derr_con<<"Connection: WARNING: reliable="<<reliable
<<" != sp->reliable="<<sp->reliable
Expand Down

0 comments on commit 2db0e93

Please sign in to comment.