From 3097dcda2b1de0511be3c3f8f938fa307ce9d406 Mon Sep 17 00:00:00 2001 From: tomrus88 Date: Thu, 20 Aug 2009 16:18:34 +0400 Subject: [PATCH] Removed useless code. --- src/game/MiscHandler.cpp | 1 - src/game/MovementHandler.cpp | 28 ---------------------------- 2 files changed, 29 deletions(-) diff --git a/src/game/MiscHandler.cpp b/src/game/MiscHandler.cpp index 976541dac..33c7ac9a6 100644 --- a/src/game/MiscHandler.cpp +++ b/src/game/MiscHandler.cpp @@ -1537,7 +1537,6 @@ void WorldSession::HandleSetTaxiBenchmarkOpcode( WorldPacket & recv_data ) void WorldSession::HandleQueryInspectAchievements( WorldPacket & recv_data ) { - uint64 guid; if(!recv_data.readPackGUID(guid)) return; diff --git a/src/game/MovementHandler.cpp b/src/game/MovementHandler.cpp index fd15c7ca5..bd6e92448 100644 --- a/src/game/MovementHandler.cpp +++ b/src/game/MovementHandler.cpp @@ -231,13 +231,6 @@ void WorldSession::HandleMovementOpcodes( WorldPacket & recv_data ) ReadMovementInfo(recv_data, &movementInfo); /*----------------*/ - if(recv_data.size() != recv_data.rpos()) - { - sLog.outError("MovementHandler: player %s (guid %d, account %u) sent a packet (opcode %u) that is " SIZEFMTD " bytes larger than it should be. Kicked as cheater.", _player->GetName(), _player->GetGUIDLow(), _player->GetSession()->GetAccountId(), opcode, recv_data.size() - recv_data.rpos()); - KickPlayer(); - return; - } - if (!MaNGOS::IsValidMapCoord(movementInfo.x, movementInfo.y, movementInfo.z, movementInfo.o)) return; @@ -372,13 +365,6 @@ void WorldSession::HandleForceSpeedChangeAck(WorldPacket &recv_data) recv_data >> newspeed; /*----------------*/ - if(recv_data.size() != recv_data.rpos()) - { - sLog.outError("MovementHandler: player %s (guid %d, account %u) sent a packet (opcode %u) that is " SIZEFMTD " bytes larger than it should be. Kicked as cheater.", _player->GetName(), _player->GetGUIDLow(), _player->GetSession()->GetAccountId(), opcode, recv_data.size() - recv_data.rpos()); - KickPlayer(); - return; - } - // client ACK send one packet for mounted/run case and need skip all except last from its // in other cases anti-cheat check can be fail in false case UnitMoveType move_type; @@ -463,13 +449,6 @@ void WorldSession::HandleMoveNotActiveMover(WorldPacket &recv_data) mi.guid = old_mover_guid; ReadMovementInfo(recv_data, &mi); - if(recv_data.size() != recv_data.rpos()) - { - sLog.outError("MovementHandler: player %s (guid %d, account %u) sent a packet (opcode %u) that is " SIZEFMTD " bytes larger than it should be. Kicked as cheater.", _player->GetName(), _player->GetGUIDLow(), _player->GetSession()->GetAccountId(), recv_data.GetOpcode(), recv_data.size() - recv_data.rpos()); - KickPlayer(); - return; - } - _player->m_movementInfo = mi; } @@ -492,13 +471,6 @@ void WorldSession::HandleDismissControlledVehicle(WorldPacket &recv_data) mi.guid = guid; ReadMovementInfo(recv_data, &mi); - if(recv_data.size() != recv_data.rpos()) - { - sLog.outError("MovementHandler: player %s (guid %d, account %u) sent a packet (opcode %u) that is " SIZEFMTD " bytes larger than it should be. Kicked as cheater.", _player->GetName(), _player->GetGUIDLow(), _player->GetSession()->GetAccountId(), recv_data.GetOpcode(), recv_data.size() - recv_data.rpos()); - KickPlayer(); - return; - } - _player->m_movementInfo = mi; // using charm guid, because we don't have vehicle guid...