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

1 files changed, 1 insertions, 3 deletions

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);
 }