Prevent crash when spell training sends a spell ID of 0. This happens…

… on the 'setskillpoints' command in etheos.
ethanmoffat · Jun 10, 2022 · 5653707 · 5653707
1 parent 15ba78c
commit 5653707
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/EOLib/PacketHandlers/SpellTrainingHandler.cs b/EOLib/PacketHandlers/SpellTrainingHandler.cs
@@ -31,8 +31,11 @@ public override bool HandlePacket(IPacket packet)
             var spellId = packet.ReadShort();
             var spellLevel = packet.ReadShort();
 
-            _characterInventoryRepository.SpellInventory.RemoveWhere(x => x.ID == spellId);
-            _characterInventoryRepository.SpellInventory.Add(new InventorySpell(spellId, spellLevel));
+            if (spellId > 0)
+            {
+                _characterInventoryRepository.SpellInventory.RemoveWhere(x => x.ID == spellId);
+                _characterInventoryRepository.SpellInventory.Add(new InventorySpell(spellId, spellLevel));
+            }
 
             var stats = _characterRepository.MainCharacter.Stats.WithNewStat(CharacterStat.SkillPoints, skillPoints);
             _characterRepository.MainCharacter = _characterRepository.MainCharacter.WithStats(stats);