From 514b908170927078c816bc3764a67b7403070f9e Mon Sep 17 00:00:00 2001
From: Patrick L <Nawuko@users.noreply.github.com>
Date: Wed, 20 Apr 2016 17:03:40 +0200
Subject: 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.
---
 src/server/game/Server/WorldSocket.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'src/server')

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;
             }
 
-- 
cgit v1.2.3