aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorPatrick L <Nawuko@users.noreply.github.com>2016-04-20 17:03:40 +0200
committerShauren <shauren.trinity@gmail.com>2016-04-20 17:03:40 +0200
commit514b908170927078c816bc3764a67b7403070f9e (patch)
treea56606a3c073d4339c4c739b4a9360eef1628c9a /src
parent636c487704f3a019f69c821e02668b05262d638d (diff)
Core/PacketIO: Missing changes from 548aa119ac2884bb1c34f80e2fb077a66bcdfd9f that could result in a crash due to double Locking (#17008)
Socket will be closed because of ReadDataHandlerResult::Error.
Diffstat (limited to 'src')
-rw-r--r--src/server/game/Server/WorldSocket.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/server/game/Server/WorldSocket.cpp b/src/server/game/Server/WorldSocket.cpp
index e3c6179a34a..1c77283d812 100644
--- a/src/server/game/Server/WorldSocket.cpp
+++ b/src/server/game/Server/WorldSocket.cpp
@@ -344,11 +344,12 @@ WorldSocket::ReadDataHandlerResult WorldSocket::ReadDataHandler()
default:
{
sessionGuard.lock();
+
LogOpcodeText(opcode, sessionGuard);
+
if (!_worldSession)
{
TC_LOG_ERROR("network.opcode", "ProcessIncoming: Client not authed opcode = %u", uint32(opcode));
- CloseSocket();
return ReadDataHandlerResult::Error;
}