flewkey/cuberite-2a
1
0
Fork 0
Code Releases Activity

Fixed possible crash

Browse Source 
It could crash if MinAmount - MaxAmount was 0 because it would execute (Number % 0)
This commit is contained in:
STRWarrior 2014-10-15 19:46:43 +02:00
parent 06b466471f
commit 0bdd2768ff
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/ItemGrid.cpp
View File

@ -658,7 +658,14 @@ void cItemGrid::GenerateRandomLootWithBooks(const cLootProbab * a_LootProbabs, s
if (LootRnd < 0) if (LootRnd < 0)
{ {
CurrentLoot = a_LootProbabs[i].m_Item; CurrentLoot = a_LootProbabs[i].m_Item;
if ((a_LootProbabs[i].m_MaxAmount - a_LootProbabs[i].m_MinAmount) > 0)
{
CurrentLoot.m_ItemCount = a_LootProbabs[i].m_MinAmount + (Rnd % (a_LootProbabs[i].m_MaxAmount - a_LootProbabs[i].m_MinAmount)); CurrentLoot.m_ItemCount = a_LootProbabs[i].m_MinAmount + (Rnd % (a_LootProbabs[i].m_MaxAmount - a_LootProbabs[i].m_MinAmount));
}
else
{
CurrentLoot.m_ItemCount = a_LootProbabs[i].m_MinAmount;
}
Rnd >>= 8; Rnd >>= 8;
break; break;
} }