aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorMachiavelli <none@none>2010-09-04 18:11:36 +0200
committerMachiavelli <none@none>2010-09-04 18:11:36 +0200
commit320cd16f209c95480c3dacf8d2a17466d551019f (patch)
tree09bd1a29cf7dcc10767eb6d31d3ea37464a0246f /src
parente3957bd6dead1564d7c0cccbd45716795efa2008 (diff)
Core/ArenaTeam: Correct player_divider use in ArenaTeam::GetAverageMMR and add a sanity check
--HG-- branch : trunk
Diffstat (limited to 'src')
-rw-r--r--src/server/game/Battlegrounds/ArenaTeam.cpp6
1 files changed, 5 insertions, 1 deletions
diff --git a/src/server/game/Battlegrounds/ArenaTeam.cpp b/src/server/game/Battlegrounds/ArenaTeam.cpp
index 02c849c9e61..605966e8e81 100644
--- a/src/server/game/Battlegrounds/ArenaTeam.cpp
+++ b/src/server/game/Battlegrounds/ArenaTeam.cpp
@@ -606,7 +606,7 @@ uint32 ArenaTeam::GetAverageMMR(Group *group) const
return 0;
uint32 matchmakerrating = 0;
- uint32 player_divider = 1;
+ uint32 player_divider = 0;
for (MemberList::const_iterator itr = m_members.begin(); itr != m_members.end(); ++itr)
{
// If player not online
@@ -620,6 +620,10 @@ uint32 ArenaTeam::GetAverageMMR(Group *group) const
matchmakerrating += itr->matchmaker_rating;
++player_divider;
}
+
+ //- x/0 = crash
+ if (player_divider == 0)
+ player_divider = 1;
matchmakerrating /= player_divider;