* Fix a crash in Player::BuyItemFromVendor caused by player spoofing packets (Backported from TC2)

* Another crash exploit prevention using the same method in Bag::StoreItem, by Visagalis

--HG--
branch : trunk

1 files changed, 5 insertions, 1 deletions

diff --git a/src/game/Bag.cpp b/src/game/Bag.cpp
index 331b12acde5..a4b7bcb5c2d 100644
--- a/src/game/Bag.cpp
+++ b/src/game/Bag.cpp
@@ -151,7 +151,11 @@ void Bag::RemoveItem( uint8 slot, bool /*update*/ )
 
 void Bag::StoreItem( uint8 slot, Item *pItem, bool /*update*/ )
 {
-    assert(slot < MAX_BAG_SIZE);
+    if(slot > MAX_BAG_SIZE)
+    {
+        sLog.outError("Player GUID" UI64FMTD "tried to manipulate packets and crash the server.", GetOwnerGUID());
+        return;
+    }
 
     if( pItem )
     {