diff options
| author | Anubisss <none@none> | 2010-01-23 14:45:58 +0100 |
|---|---|---|
| committer | Anubisss <none@none> | 2010-01-23 14:45:58 +0100 |
| commit | d9cb0702158fd045285f2c0a904cb31a45a3864a (patch) | |
| tree | cdc6a7c6a83c6eea603ae296a653fb9f04945aad /src/game/Channel.cpp | |
| parent | c784110d87666579f18620a98e1e57118db4a9cf (diff) | |
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
Diffstat (limited to 'src/game/Channel.cpp')
| -rw-r--r-- | src/game/Channel.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/game/Channel.cpp b/src/game/Channel.cpp index da125b6392b..3b32f29371c 100644 --- a/src/game/Channel.cpp +++ b/src/game/Channel.cpp @@ -53,7 +53,7 @@ Channel::Channel(const std::string& name, uint32 channel_id, uint32 Team) //load not built in channel if saved std::string _name(name); CharacterDatabase.escape_string(_name); - QueryResult *result = CharacterDatabase.PQuery("SELECT m_name, m_team, m_announce, m_moderate, m_password, BannedList FROM channels WHERE m_name = '%s' AND m_team = '%u'", _name.c_str(), m_Team); + QueryResult_AutoPtr result = CharacterDatabase.PQuery("SELECT m_name, m_team, m_announce, m_moderate, m_password, BannedList FROM channels WHERE m_name = '%s' AND m_team = '%u'", _name.c_str(), m_Team); if (result)//load { Field *fields = result->Fetch(); @@ -64,8 +64,6 @@ Channel::Channel(const std::string& name, uint32 channel_id, uint32 Team) m_password = fields[4].GetString(); const char* db_BannedList = fields[5].GetString(); - delete result; - m_IsSaved = true; if (db_BannedList) |