BRNSystems/minetest
1
0
Fork 0
forked from Mirrorlandia_minetest/minetest
Code Pull Requests 1 Activity

Network: fix a concurrency problem, by re-adding a copy in ConnectionCommand

Browse Source
This commit is contained in:
Loïc Blot 2017-09-04 16:46:03 +02:00
parent a383eb0eb9
commit 5b04f5e7d2
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/network/connection.h
View File

@ -330,6 +330,18 @@ struct ConnectionCommand
bool raw = false; bool raw = false;
ConnectionCommand() = default; ConnectionCommand() = default;
ConnectionCommand &operator=(const ConnectionCommand &other)
{
type = other.type;
address = other.address;
peer_id = other.peer_id;
channelnum = other.channelnum;
// We must copy the buffer here to prevent race condition
data = SharedBuffer<u8>(*other.data, other.data.getSize());
reliable = other.reliable;
raw = other.reliable;
return *this;
}
void serve(Address address_) void serve(Address address_)
{ {