From: Loic Blot <loic.blot@unix-experience.fr>
Date: Mon, 9 Oct 2017 22:47:37 +0000 (+0200)
Subject: NetworkPacket::putRawPacket: resize m_data to datasize + memcpy
X-Git-Tag: 5.0.0~780
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=9d295906efc7eec58dba3f7494f4e444d2c8d00f;p=oweals%2Fminetest.git

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

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<u8>(&data[2], &data[2 + m_datasize]);
+	memcpy(&m_data[0], &data[2], m_datasize);
 }
 
 const char* NetworkPacket::getString(u32 from_offset)