diff options
| author | jackpoz <giacomopoz@gmail.com> | 2015-11-15 16:17:12 +0100 |
|---|---|---|
| committer | jackpoz <giacomopoz@gmail.com> | 2015-11-15 16:17:12 +0100 |
| commit | fbaed38b5ea605e3dbd65a99835db1778ac24391 (patch) | |
| tree | a659fea206a4a9595e3b62edb4fa1a8dc9105c83 | |
| parent | 63f2675076e96abe9dad04baf1f7310b429d9eae (diff) | |
Core/Quest: Fix issues reported by static analysis
Coverity defect IDs: 1339309, 1339310
| -rw-r--r-- | src/server/game/Entities/Player/Player.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/server/game/Entities/Player/Player.cpp b/src/server/game/Entities/Player/Player.cpp index 2d70329fabf..1a9c0fbce4b 100644 --- a/src/server/game/Entities/Player/Player.cpp +++ b/src/server/game/Entities/Player/Player.cpp @@ -15562,8 +15562,9 @@ bool Player::GetQuestRewardStatus(uint32 quest_id) const if (qInfo->IsSeasonal() && !qInfo->IsRepeatable()) { uint16 eventId = sGameEventMgr->GetEventIdForQuest(qInfo); - if (m_seasonalquests.find(eventId) != m_seasonalquests.end()) - return m_seasonalquests.find(eventId)->second.find(quest_id) != m_seasonalquests.find(eventId)->second.end(); + auto seasonalQuestItr = m_seasonalquests.find(eventId); + if (seasonalQuestItr != m_seasonalquests.end()) + return seasonalQuestItr->second.find(quest_id) != seasonalQuestItr->second.end(); return false; } @@ -15591,7 +15592,8 @@ QuestStatus Player::GetQuestStatus(uint32 quest_id) const if (qInfo->IsSeasonal() && !qInfo->IsRepeatable()) { uint16 eventId = sGameEventMgr->GetEventIdForQuest(qInfo); - if (m_seasonalquests.find(eventId) == m_seasonalquests.end() || m_seasonalquests.find(eventId)->second.find(quest_id) == m_seasonalquests.find(eventId)->second.end()) + auto seasonalQuestItr = m_seasonalquests.find(eventId); + if (seasonalQuestItr == m_seasonalquests.end() || seasonalQuestItr->second.find(quest_id) == seasonalQuestItr->second.end()) return QUEST_STATUS_NONE; } |