Core/ArenaTeam: Correct player_divider use in ArenaTeam::GetAverageMMR and add a sanity check
--HG-- branch : trunk
This commit is contained in:
@@ -606,7 +606,7 @@ uint32 ArenaTeam::GetAverageMMR(Group *group) const
|
|||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
uint32 matchmakerrating = 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)
|
for (MemberList::const_iterator itr = m_members.begin(); itr != m_members.end(); ++itr)
|
||||||
{
|
{
|
||||||
// If player not online
|
// If player not online
|
||||||
@@ -621,6 +621,10 @@ uint32 ArenaTeam::GetAverageMMR(Group *group) const
|
|||||||
++player_divider;
|
++player_divider;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
//- x/0 = crash
|
||||||
|
if (player_divider == 0)
|
||||||
|
player_divider = 1;
|
||||||
|
|
||||||
matchmakerrating /= player_divider;
|
matchmakerrating /= player_divider;
|
||||||
|
|
||||||
return matchmakerrating;
|
return matchmakerrating;
|
||||||
|
|||||||
Reference in New Issue
Block a user