Merge pull request #29 from birarda/atp

add an assert that enqueue packet is truly from message

libraries/networking/src/udt/Connection.cpp

@@ -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;

libraries/networking/src/udt/Socket.cpp

@@ -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()) {