diff options
author | Machiavelli <none@none> | 2010-09-04 18:11:36 +0200 |
---|---|---|
committer | Machiavelli <none@none> | 2010-09-04 18:11:36 +0200 |
commit | 320cd16f209c95480c3dacf8d2a17466d551019f (patch) | |
tree | 09bd1a29cf7dcc10767eb6d31d3ea37464a0246f /src | |
parent | e3957bd6dead1564d7c0cccbd45716795efa2008 (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.cpp | 6 |
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; |