Fix crash when deleting vendoritems with multiple extendedcosts - thanks to Vladimir

Fixes issue #1989

--HG--
branch : trunk

1 files changed, 4 insertions, 2 deletions

diff --git a/src/game/Creature.cpp b/src/game/Creature.cpp
index 3a124b2a999..34372ac614d 100644
--- a/src/game/Creature.cpp
+++ b/src/game/Creature.cpp
@@ -63,13 +63,15 @@ TrainerSpell const* TrainerSpellData::Find(uint32 spell_id) const
 bool VendorItemData::RemoveItem(uint32 item_id)
 {
     bool found = false;
-    for (VendorItemList::iterator i = m_items.begin(); i != m_items.end(); ++i)
+    for (VendorItemList::iterator i = m_items.begin(); i != m_items.end();)
     {
         if ((*i)->item == item_id)
         {
-            i = m_items.erase(i);
+            i = m_items.erase(i++);
             found = true;
         }
+        else
+            ++i;
     }
     return found;
 }