josh/TCPlayerbotCore
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Core/Dungeon Finder: add safe check in first call to FindNewGroup (Avoids Thousands of recursive calls

Browse Source 
)

--HG--
branch : trunk
This commit is contained in:
Spp
2010-09-16 18:53:25 +02:00
parent 4d88ce57d2
commit 5ec5d5b6be
1 changed files with 14 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/server/game/DungeonFinding/LFGMgr.cpp
+14 -1
View File
@@ -779,9 +779,22 @@ void LFGMgr::FindNewGroups(LfgGuidList &check, LfgGuidList all, LfgProposalList
if (!check.size() || check.size() > MAXGROUPSIZE) if (!check.size() || check.size() > MAXGROUPSIZE)
return; return;
if (check.size() == 1) // Consistency check
{
uint64 guid = *check.begin();
LfgQueueInfoMap::iterator itQueue = m_QueueInfoMap.find(guid);
if (itQueue == m_QueueInfoMap.end())
{
sLog.outError("LFGMgr::FindNewGroups: [" UI64FMTD "] is not queued but listed as queued!", guid);
RemoveFromQueue(guid);
return;
}
}
sLog.outDebug("LFGMgr::FindNewGroup: (%s) - all(%s)", ConcatenateGuids(check).c_str(), ConcatenateGuids(all).c_str()); sLog.outDebug("LFGMgr::FindNewGroup: (%s) - all(%s)", ConcatenateGuids(check).c_str(), ConcatenateGuids(all).c_str());
LfgGuidList compatibles;
// Check individual compatibilities // Check individual compatibilities
LfgGuidList compatibles;
for (LfgGuidList::iterator it = all.begin(); it != all.end(); ++it) for (LfgGuidList::iterator it = all.begin(); it != all.end(); ++it)
{ {
check.push_back(*it); check.push_back(*it);