diff options
| author | Shauren <shauren.trinity@gmail.com> | 2014-06-11 19:28:25 +0200 |
|---|---|---|
| committer | Shauren <shauren.trinity@gmail.com> | 2014-06-11 19:28:25 +0200 |
| commit | 06962b16f147275cf3ef23caf2f41ce0d25c25ef (patch) | |
| tree | d05a0def256ffe7aad0c36666e99f894484c9520 /src | |
| parent | 2ba05406ce0632d30498b05981a02f2b7d4f75da (diff) | |
Core/PacketIO: Fixed logic in DosProtection::EvaluateOpcode
Diffstat (limited to 'src')
| -rw-r--r-- | src/server/game/Server/WorldSession.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/server/game/Server/WorldSession.cpp b/src/server/game/Server/WorldSession.cpp index 4d51ccb91c9..9f8b3785e92 100644 --- a/src/server/game/Server/WorldSession.cpp +++ b/src/server/game/Server/WorldSession.cpp @@ -1260,12 +1260,12 @@ bool WorldSession::DosProtection::EvaluateOpcode(WorldPacket& p, time_t time) co } // Check if player is flooding some packets - if (++packetCounter.amountCounter > maxPacketCounterAllowed) - { - TC_LOG_WARN("network", "AntiDOS: Account %u, IP: %s, Ping: %u, Character: %s, flooding packet (opc: %s (0x%X), count: %u)", - Session->GetAccountId(), Session->GetRemoteAddress().c_str(), Session->GetLatency(), Session->GetPlayerName().c_str(), - opcodeTable[p.GetOpcode()].name, p.GetOpcode(), packetCounter.amountCounter); - } + if (++packetCounter.amountCounter <= maxPacketCounterAllowed) + return true; + + TC_LOG_WARN("network", "AntiDOS: Account %u, IP: %s, Ping: %u, Character: %s, flooding packet (opc: %s (0x%X), count: %u)", + Session->GetAccountId(), Session->GetRemoteAddress().c_str(), Session->GetLatency(), Session->GetPlayerName().c_str(), + opcodeTable[p.GetOpcode()].name, p.GetOpcode(), packetCounter.amountCounter); switch (_policy) { |
