From 4c0476bfd7d104386806e35edfa975634495dd2c Mon Sep 17 00:00:00 2001 From: jackpoz Date: Sat, 31 Jan 2015 22:59:21 +0100 Subject: Core/Misc: Properly sanitize ReputationSpillover data Fix an issue added in 024b57bb74a664cd515cf0822d4b8e939a91fe03 that didn't properly skip ReputationSpillover rows with invalid data. Fix also a static analysis issue reported by Coverity. (cherry picked from commit 95b1204798d3ab2a141f81b294b064255c335b8a) Conflicts: src/server/game/Globals/ObjectMgr.cpp --- src/server/shared/DataStores/DBCStore.h | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'src/server/shared') diff --git a/src/server/shared/DataStores/DBCStore.h b/src/server/shared/DataStores/DBCStore.h index 10d4ff1bec9..1cb67a4235e 100644 --- a/src/server/shared/DataStores/DBCStore.h +++ b/src/server/shared/DataStores/DBCStore.h @@ -87,6 +87,13 @@ class DBCStorage return (id >= nCount) ? NULL : indexTable.asT[id]; } + T const* AssertEntry(uint32 id) const + { + T const* entry = LookupEntry(id); + ASSERT(entry); + return entry; + } + uint32 GetNumRows() const { return nCount; } char const* GetFormat() const { return fmt; } uint32 GetFieldCount() const { return fieldCount; } -- cgit v1.2.3