Sky/EQ2Emu
1
0
Fork 0
Code Issues Activity

the destroy spells function was using the wrong mutex locks

Browse Source
This commit is contained in:
Emagi 2025-02-07 14:35:00 -05:00
parent 921f00410b
commit d639e54961
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
source/WorldServer/LuaInterface.cpp
View File

@ -143,7 +143,7 @@ void LuaInterface::Process() {
} }
void LuaInterface::DestroySpells() { void LuaInterface::DestroySpells() {
MSpells.lock(); MSpellScripts.writelock(__FUNCTION__, __LINE__);
map<string, map<lua_State*, LuaSpell*> >::iterator spell_script_itr; map<string, map<lua_State*, LuaSpell*> >::iterator spell_script_itr;
for(spell_script_itr = spell_scripts.begin(); spell_script_itr != spell_scripts.end(); spell_script_itr++) { for(spell_script_itr = spell_scripts.begin(); spell_script_itr != spell_scripts.end(); spell_script_itr++) {
map<lua_State*, LuaSpell*>::iterator inner_itr; map<lua_State*, LuaSpell*>::iterator inner_itr;
@ -166,7 +166,9 @@ void LuaInterface::DestroySpells() {
zone->GetSpellProcess()->CheckRemoveTargetFromSpell(cur_spell, false, true); zone->GetSpellProcess()->CheckRemoveTargetFromSpell(cur_spell, false, true);
} }
SetLuaUserDataStale(cur_spell); SetLuaUserDataStale(cur_spell);
MSpells.lock();
RemoveCurrentSpell(inner_itr->first, inner_itr->second, false, true, false); RemoveCurrentSpell(inner_itr->first, inner_itr->second, false, true, false);
MSpells.unlock();
lua_close(inner_itr->first); lua_close(inner_itr->first);
safe_delete(cur_spell); safe_delete(cur_spell);
MSpellDelete.unlock(); MSpellDelete.unlock();
@ -175,9 +177,13 @@ void LuaInterface::DestroySpells() {
Mutex* mutex = GetSpellScriptMutex(spell_script_itr->first.c_str()); Mutex* mutex = GetSpellScriptMutex(spell_script_itr->first.c_str());
safe_delete(mutex); safe_delete(mutex);
} }
current_spells.clear();
spell_scripts_mutex.clear(); spell_scripts_mutex.clear();
spell_scripts.clear(); spell_scripts.clear();
MSpellScripts.releasewritelock(__FUNCTION__, __LINE__);
MSpells.lock();
current_spells.clear();
MSpells.unlock(); MSpells.unlock();
} }