diff options
| author | jackpoz <giacomopoz@gmail.com> | 2019-05-18 16:12:23 +0200 |
|---|---|---|
| committer | jackpoz <giacomopoz@gmail.com> | 2019-05-18 16:12:23 +0200 |
| commit | a0bd68cf465517786d16bcb145ed3fc8bc5b5b98 (patch) | |
| tree | 7e64037d0eaf370b8d2e8812713695de5ebce193 /src/server/game/Server | |
| parent | a0d41c314fcce2b7e941628b3b9b1710cc6acee8 (diff) | |
Core/Misc: Fix static analysis issues reported by Coverity
Diffstat (limited to 'src/server/game/Server')
| -rw-r--r-- | src/server/game/Server/WorldSocket.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/server/game/Server/WorldSocket.cpp b/src/server/game/Server/WorldSocket.cpp index 8b0c01db8ce..69d8be6c8b7 100644 --- a/src/server/game/Server/WorldSocket.cpp +++ b/src/server/game/Server/WorldSocket.cpp @@ -378,6 +378,7 @@ WorldSocket::ReadDataHandlerResult WorldSocket::ReadDataHandler() if (!_worldSession) { TC_LOG_ERROR("network.opcode", "ProcessIncoming: Client not authed opcode = %u", uint32(opcode)); + delete packetToQueue; return ReadDataHandlerResult::Error; } @@ -385,6 +386,7 @@ WorldSocket::ReadDataHandlerResult WorldSocket::ReadDataHandler() if (!handler) { TC_LOG_ERROR("network.opcode", "No defined handler for opcode %s sent by %s", GetOpcodeNameForLogging(static_cast<OpcodeClient>(packet.GetOpcode())).c_str(), _worldSession->GetPlayerInfo().c_str()); + delete packetToQueue; return ReadDataHandlerResult::Error; } |
