From 9d295906efc7eec58dba3f7494f4e444d2c8d00f Mon Sep 17 00:00:00 2001 From: Loic Blot Date: Tue, 10 Oct 2017 00:47:37 +0200 Subject: [PATCH] NetworkPacket::putRawPacket: resize m_data to datasize + memcpy In some cases NetworkPacket was created using default constructor and m_data is not properly sized. This fixed out of bounds memory copy Also use memcpy instead of std::vector affectation to enhance packet creation --- src/network/networkpacket.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/network/networkpacket.cpp b/src/network/networkpacket.cpp index a35db9574..14b1ac440 100644 --- a/src/network/networkpacket.cpp +++ b/src/network/networkpacket.cpp @@ -59,9 +59,11 @@ void NetworkPacket::putRawPacket(u8 *data, u32 datasize, session_t peer_id) m_datasize = datasize - 2; m_peer_id = peer_id; + m_data.resize(m_datasize); + // split command and datas m_command = readU16(&data[0]); - m_data = std::vector(&data[2], &data[2 + m_datasize]); + memcpy(&m_data[0], &data[2], m_datasize); } const char* NetworkPacket::getString(u32 from_offset)