diff options
| author | Perttu Ahola <celeron55@gmail.com> | 2012-03-20 17:55:34 +0200 |
|---|---|---|
| committer | Perttu Ahola <celeron55@gmail.com> | 2012-03-20 17:55:34 +0200 |
| commit | c0530921ffac89c2249500ba0a1bddd9d288cfbe (patch) | |
| tree | c662c1b8872067160b4dac3fd84b553cd7093737 | |
| parent | 94f1ab4da3296be707f9164d7a2f7cd61e357e12 (diff) | |
| download | minetest-c0530921ffac89c2249500ba0a1bddd9d288cfbe.tar.gz minetest-c0530921ffac89c2249500ba0a1bddd9d288cfbe.tar.bz2 minetest-c0530921ffac89c2249500ba0a1bddd9d288cfbe.zip | |
Attempt to fix some minor memory leaks
| -rw-r--r-- | src/connection.cpp | 8 | ||||
| -rw-r--r-- | src/itemdef.cpp | 7 |
2 files changed, 13 insertions, 2 deletions
diff --git a/src/connection.cpp b/src/connection.cpp index 31aa93307..127cabdc5 100644 --- a/src/connection.cpp +++ b/src/connection.cpp @@ -537,6 +537,14 @@ Connection::Connection(u32 protocol_id, u32 max_packet_size, float timeout, Connection::~Connection() { stop(); + // Delete peers + for(core::map<u16, Peer*>::Iterator + j = m_peers.getIterator(); + j.atEnd() == false; j++) + { + Peer *peer = j.getNode()->getValue(); + delete peer; + } } /* Internal stuff */ diff --git a/src/itemdef.cpp b/src/itemdef.cpp index a646134af..934150aa1 100644 --- a/src/itemdef.cpp +++ b/src/itemdef.cpp @@ -290,8 +290,11 @@ public: // Ensure that the "" item (the hand) always has ToolCapabilities if(def.name == "") assert(def.tool_capabilities != NULL); - - m_item_definitions[def.name] = new ItemDefinition(def); + + if(m_item_definitions.count(def.name) == 0) + m_item_definitions[def.name] = new ItemDefinition(def); + else + *(m_item_definitions[def.name]) = def; // Remove conflicting alias if it exists bool alias_removed = (m_aliases.erase(def.name) != 0); |