locks part 2

gistfile1.cpp

diff --git a/src/game/Object.h b/src/game/Object.h
index 7b06e3b..9bdcdfc 100644
--- a/src/game/Object.h
+++ b/src/game/Object.h
@@ -651,4 +651,20 @@ class MANGOS_DLL_SPEC WorldObject : public Object
         WorldUpdateCounter m_updateTracker;
 };
 
+#ifndef MAPLOCK_READ(OBJ,TYPE)
+#  define MAPLOCK_READ(OBJ,TYPE) Map::ReadGuard Guard(OBJ->GetMap()->GetLock(TYPE));
+#endif
+
+#ifndef MAPLOCK_READ1(OBJ,TYPE)
+#  define MAPLOCK_READ1(OBJ,TYPE) Map::ReadGuard Guard1(OBJ->GetMap()->GetLock(TYPE));
+#endif
+
+#ifndef MAPLOCK_READ2(OBJ,TYPE)
+#  define MAPLOCK_READ2(OBJ,TYPE) Map::ReadGuard Guard2(OBJ->GetMap()->GetLock(TYPE));
+#endif
+
+#ifndef MAPLOCK_WRITE(OBJ,TYPE)
+#  define MAPLOCK_WRITE(OBJ,TYPE) Map::WriteGuard Guard(OBJ->GetMap()->GetLock(TYPE));
+#endif
+
 #endif
diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index 272e832..d7250bb 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -5133,6 +5133,7 @@ SpellCastResult Spell::CheckCast(bool strict)
 
     if (Unit *target = m_targets.getUnitTarget())
     {
+        MAPLOCK_READ(target,MAP_LOCK_TYPE_AURAS);
         // target state requirements (not allowed state), apply to self also
         // This check not need - checked in CheckTarget()
         // if (m_spellInfo->TargetAuraStateNot && target->HasAuraState(AuraState(m_spellInfo->TargetAuraStateNot)))
diff --git a/src/game/Unit.cpp b/src/game/Unit.cpp
index ea37cbe..6508a10 100644
--- a/src/game/Unit.cpp
+++ b/src/game/Unit.cpp
@@ -340,7 +340,10 @@ void Unit::Update( uint32 update_diff, uint32 p_time )
     m_Events.Update( update_diff );
     _UpdateSpells( update_diff );
 
-    CleanupDeletedAuras();
+    {
+        MAPLOCK_WRITE(this,MAP_LOCK_TYPE_AURAS);
+        CleanupDeletedAuras();
+    }
 
     if (m_lastManaUseTimer)
     {
@@ -3695,7 +3698,10 @@ void Unit::_UpdateSpells( uint32 time )
         SpellAuraHolder* i_holder = m_spellAuraHoldersUpdateIterator->second;
         ++m_spellAuraHoldersUpdateIterator;                            // need shift to next for allow update if need into aura update
         if (i_holder && !i_holder->IsDeleted() && !i_holder->IsEmptyHolder() && !i_holder->IsInUse())
+        {
+            MAPLOCK_READ(this,MAP_LOCK_TYPE_AURAS);
             i_holder->UpdateHolder(time);
+        }
     }
 
     // remove expired auras
@@ -5157,6 +5163,7 @@ void Unit::RemoveSpellAuraHolder(SpellAuraHolder *holder, AuraRemoveMode mode)
     // store it in holder list with delayed deletion
     if (holder && !holder->IsDeleted())
     {
+        MAPLOCK_READ(this,MAP_LOCK_TYPE_AURAS);
         holder->SetDeleted();
         m_deletedHolders.push_back(holder);
     }
@@ -5185,6 +5192,7 @@ void Unit::RemoveAura(Aura *Aur, AuraRemoveMode mode)
     // remove from list before mods removing (prevent cyclic calls, mods added before including to aura list - use reverse order)
     if (Aur->GetModifier()->m_auraname < TOTAL_AURAS)
     {
+        MAPLOCK_READ(this,MAP_LOCK_TYPE_AURAS);
         m_modAuras[Aur->GetModifier()->m_auraname].remove(Aur);
     }
 
@@ -5221,6 +5229,7 @@ void Unit::RemoveAura(Aura *Aur, AuraRemoveMode mode)
 
     // If aura in use (removed from code that plan access to it data after return)
     // store it in aura list with delayed deletion
+    MAPLOCK_READ(this,MAP_LOCK_TYPE_AURAS);
     m_deletedAuras.push_back(Aur);
 }
 
@@ -6783,6 +6792,8 @@ uint32 Unit::SpellDamageBonusDone(Unit *pVictim, SpellEntry const *spellProto, u
     if (spellProto->IsFitToFamily<SPELLFAMILY_WARLOCK, CF_WARLOCK_CONFLAGRATE>())
         return pdamage;
 
+    MAPLOCK_READ(this,MAP_LOCK_TYPE_AURAS);
+
     // For totems get damage bonus from owner (statue isn't totem in fact)
     if ( GetTypeId()==TYPEID_UNIT && ((Creature*)this)->IsTotem() && ((Totem*)this)->GetTotemType()!=TOTEM_STATUE)
     {
@@ -6800,7 +6811,7 @@ uint32 Unit::SpellDamageBonusDone(Unit *pVictim, SpellEntry const *spellProto, u
     float nonStackingPos = 0.0f;
     float nonStackingNeg = 0.0f;
 
-    AuraList const mModDamagePercentDone = GetAurasByType(SPELL_AURA_MOD_DAMAGE_PERCENT_DONE);
+    AuraList const& mModDamagePercentDone = GetAurasByType(SPELL_AURA_MOD_DAMAGE_PERCENT_DONE);
     for(AuraList::const_iterator i = mModDamagePercentDone.begin(); i != mModDamagePercentDone.end(); ++i)
     {
         if (!*i)
@@ -6867,6 +6878,10 @@ uint32 Unit::SpellDamageBonusDone(Unit *pVictim, SpellEntry const *spellProto, u
     // done scripted mod (take it from owner)
     Unit *owner = GetOwner();
     if (!owner) owner = this;
+
+    MAPLOCK_READ1(owner,MAP_LOCK_TYPE_AURAS);
+    MAPLOCK_READ2(pVictim,MAP_LOCK_TYPE_AURAS);
+
     AuraList const& mOverrideClassScript= owner->GetAurasByType(SPELL_AURA_OVERRIDE_CLASS_SCRIPTS);
     for(AuraList::const_iterator i = mOverrideClassScript.begin(); i != mOverrideClassScript.end(); ++i)
     {
@@ -7186,6 +7201,8 @@ uint32 Unit::SpellDamageBonusTaken(Unit *pCaster, SpellEntry const *spellProto,
     float TakenTotalMod = 1.0f;
     int32 TakenTotal = 0;
 
+    MAPLOCK_READ(this,MAP_LOCK_TYPE_AURAS);
+
     // ..taken
     TakenTotalMod *= GetTotalAuraMultiplierByMiscMask(SPELL_AURA_MOD_DAMAGE_PERCENT_TAKEN, schoolMask);
 

line 10 (OBJ-> should be ((OBJ)->