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/GlobalEvents.cpp b/src/game/GlobalEvents.cpp
index 458933244dd..defdd96f1f9 100644
--- a/src/game/GlobalEvents.cpp
+++ b/src/game/GlobalEvents.cpp
@@ -32,7 +32,7 @@
 #include "ObjectDefines.h"
 #include "Corpse.h"
 
-static void CorpsesEraseCallBack(QueryResult *result, bool bones)
+static void CorpsesEraseCallBack(QueryResult_AutoPtr result, bool bones)
 {
     if(!result)
         return;
@@ -68,8 +68,6 @@ static void CorpsesEraseCallBack(QueryResult *result, bool bones)
             CharacterDatabase.PExecute("DELETE FROM corpse WHERE guid = '%u'",guidlow);
         }
     } while (result->NextRow());
-
-    delete result;
 }
 
 /// Handle periodic erase of corpses and bones