Fixed possible crash

It could crash if MinAmount - MaxAmount was 0 because it would execute (Number % 0)
cuberite · Oct 15, 2014 · 0bdd276 · 0bdd276
1 parent 06b4664
commit 0bdd276
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/ItemGrid.cpp b/src/ItemGrid.cpp
@@ -658,7 +658,14 @@ void cItemGrid::GenerateRandomLootWithBooks(const cLootProbab * a_LootProbabs, s
 			if (LootRnd < 0)
 			{
 				CurrentLoot = a_LootProbabs[i].m_Item;
-				CurrentLoot.m_ItemCount = a_LootProbabs[i].m_MinAmount + (Rnd % (a_LootProbabs[i].m_MaxAmount - a_LootProbabs[i].m_MinAmount));
+				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));
+				}
+				else
+				{
+					CurrentLoot.m_ItemCount = a_LootProbabs[i].m_MinAmount;
+				}
 				Rnd >>= 8;
 				break;
 			}