Fix crash introduced in 4610bbf047.

- Update SqlAsyncTask´s destructor, properly NULLpoint SqlOperation* m_op so other threads can deal with it accordingly.
- Don´t keep NULL´d elements in transaction queues, remove them instead.

This now means you should be able to run a stable server with > 2 MapUpdate threads. Please leave feedback.

Courtesy of click and myself.

--HG--
branch : trunk

1 files changed, 2 insertions, 0 deletions

diff --git a/src/shared/Database/Database.cpp b/src/shared/Database/Database.cpp
index d03f8704896..40ab58fa621 100644
--- a/src/shared/Database/Database.cpp
+++ b/src/shared/Database/Database.cpp
@@ -591,6 +591,7 @@ bool Database::CommitTransaction()
     if (i != m_tranQueues.end() && i->second != NULL)
     {
         m_threadBody->Delay(i->second);
+        m_tranQueues.erase(i);
         return true;
     }
     else
@@ -620,6 +621,7 @@ bool Database::RollbackTransaction()
     {
         delete i->second;
         i->second = NULL;
+        m_tranQueues.erase(i);
     }
 
     return true;