topic_transport: receiver/depacketizer: prune before accessing the list

This could potentially trigger a segfault when all messages are older than 5s. Maybe related: AIS-Bonn#17
leggedrobotics · Aug 13, 2021 · 366a827 · 366a827
1 parent 2c0ecef
commit 366a827
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/nimbro_topic_transport/src/receiver/depacketizer.cpp b/nimbro_topic_transport/src/receiver/depacketizer.cpp
@@ -35,10 +35,10 @@ void Depacketizer::addPacket(const Packet::Ptr& packet)
 
 	if(it == m_messageBuffer.end())
 	{
+		pruneMessages();
+
 		m_messageBuffer.push_front(PartialMessage(msg_id));
 		it = m_messageBuffer.begin();
-
-		pruneMessages();
 	}
 
 	handleMessagePacket(it, packet);