Merge pull request #2856 from mathieui/netplay-free
[netplay] Fix a crash
This commit is contained in:
commit
b8a35f6996
|
@ -192,6 +192,8 @@ void NetPlayServer::ThreadFunc()
|
||||||
case ENET_EVENT_TYPE_DISCONNECT:
|
case ENET_EVENT_TYPE_DISCONNECT:
|
||||||
{
|
{
|
||||||
std::lock_guard<std::recursive_mutex> lkg(m_crit.game);
|
std::lock_guard<std::recursive_mutex> lkg(m_crit.game);
|
||||||
|
if (!netEvent.peer->data)
|
||||||
|
break;
|
||||||
auto it = m_players.find(*(PlayerId *)netEvent.peer->data);
|
auto it = m_players.find(*(PlayerId *)netEvent.peer->data);
|
||||||
if (it != m_players.end())
|
if (it != m_players.end())
|
||||||
{
|
{
|
||||||
|
@ -232,6 +234,18 @@ unsigned int NetPlayServer::OnConnect(ENetPeer* socket)
|
||||||
} while (epack == nullptr);
|
} while (epack == nullptr);
|
||||||
rpac.append(epack->data, epack->dataLength);
|
rpac.append(epack->data, epack->dataLength);
|
||||||
|
|
||||||
|
// give new client first available id
|
||||||
|
PlayerId pid = 1;
|
||||||
|
for (auto i = m_players.begin(); i != m_players.end(); ++i)
|
||||||
|
{
|
||||||
|
if (i->second.pid == pid)
|
||||||
|
{
|
||||||
|
pid++;
|
||||||
|
i = m_players.begin();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
socket->data = new PlayerId(pid);
|
||||||
|
|
||||||
std::string npver;
|
std::string npver;
|
||||||
rpac >> npver;
|
rpac >> npver;
|
||||||
// Dolphin netplay version
|
// Dolphin netplay version
|
||||||
|
@ -250,25 +264,12 @@ unsigned int NetPlayServer::OnConnect(ENetPeer* socket)
|
||||||
m_update_pings = true;
|
m_update_pings = true;
|
||||||
|
|
||||||
Client player;
|
Client player;
|
||||||
|
player.pid = pid;
|
||||||
player.socket = socket;
|
player.socket = socket;
|
||||||
rpac >> player.revision;
|
rpac >> player.revision;
|
||||||
rpac >> player.name;
|
rpac >> player.name;
|
||||||
|
|
||||||
enet_packet_destroy(epack);
|
enet_packet_destroy(epack);
|
||||||
|
|
||||||
// give new client first available id
|
|
||||||
PlayerId pid = 1;
|
|
||||||
for (auto i = m_players.begin(); i != m_players.end(); ++i)
|
|
||||||
{
|
|
||||||
if (i->second.pid == pid)
|
|
||||||
{
|
|
||||||
pid++;
|
|
||||||
i = m_players.begin();
|
|
||||||
}
|
|
||||||
}
|
|
||||||
player.pid = pid;
|
|
||||||
socket->data = new PlayerId(pid);
|
|
||||||
|
|
||||||
// try to automatically assign new user a pad
|
// try to automatically assign new user a pad
|
||||||
for (PadMapping& mapping : m_pad_map)
|
for (PadMapping& mapping : m_pad_map)
|
||||||
{
|
{
|
||||||
|
|
Loading…
Reference in New Issue