From d9cb0702158fd045285f2c0a904cb31a45a3864a Mon Sep 17 00:00:00 2001 From: Anubisss Date: Sat, 23 Jan 2010 14:45:58 +0100 Subject: Implement QueryResult_AutoPtr type which is ACE's reference counted auto_ptr(ACE_Refcounted_Auto_Ptr) for QueryResult pointers. Use this auto_ptr for every DB queries(except QueryNamedResult yet). This patch guarantees NO memory leaks from QueryResult pointers. Thanks to raczman for the idea and for the helping to make this patch. --HG-- branch : trunk --- src/game/AddonMgr.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'src/game/AddonMgr.cpp') diff --git a/src/game/AddonMgr.cpp b/src/game/AddonMgr.cpp index 15d5633f2cb..9116f504a1d 100644 --- a/src/game/AddonMgr.cpp +++ b/src/game/AddonMgr.cpp @@ -42,7 +42,7 @@ AddonMgr::~AddonMgr() void AddonMgr::LoadFromDB() { - QueryResult* result = CharacterDatabase.PQuery("SELECT name, crc FROM addons"); + QueryResult_AutoPtr result = CharacterDatabase.PQuery("SELECT name, crc FROM addons"); if(!result) { sLog.outErrorDb("The table `addons` is empty"); @@ -68,8 +68,6 @@ void AddonMgr::LoadFromDB() m_knownAddons.push_back(addon); } while(result->NextRow()); - delete result; - sLog.outString(); sLog.outString(">> Loaded %u known addons", count); } -- cgit v1.2.3