add an assert that enqueued packet is part of message

This commit is contained in:
Stephen Birarda 2015-08-27 17:24:57 -07:00
parent 33dc65a00a
commit 5fc09a68a8
2 changed files with 5 additions and 1 deletions

View file

@ -737,6 +737,10 @@ void Connection::updateCongestionControlAndSendQueue(std::function<void ()> cong
}
void PendingReceivedMessage::enqueuePacket(std::unique_ptr<Packet> packet) {
Q_ASSERT_X(packet->isPartOfMessage(),
"PendingReceivedMessage::enqueuePacket",
"called with a packet that is not part of a message");
if (_isComplete) {
qCDebug(networking) << "UNEXPECTED: Received packet for a message that is already complete";
return;

View file

@ -215,7 +215,7 @@ void Socket::readPendingDatagrams() {
} else {
// setup a Packet from the data we just read
auto packet = Packet::fromReceivedPacket(std::move(buffer), packetSizeWithHeader, senderSockAddr);
// call our verification operator to see if this packet is verified
if (!_packetFilterOperator || _packetFilterOperator(*packet)) {
if (packet->isReliable()) {