From 1d16d1ce7d43034af25bdcf07f7a9631f14cbcbc Mon Sep 17 00:00:00 2001 From: Uli Schlachter Date: Sat, 4 Feb 2012 14:14:16 +0100 Subject: [PATCH] Fix a stupid bug in CUser::~CUser() A CIRCNetwork's destructor removes the network from its associated user's list of networks. Now that you know this, stare at the diff until you figure out the problem. Yeah, "ouch". Signed-off-by: Uli Schlachter --- src/User.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/User.cpp b/src/User.cpp index cdf1c8b0..45c75828 100644 --- a/src/User.cpp +++ b/src/User.cpp @@ -89,11 +89,9 @@ CUser::CUser(const CString& sUserName) CUser::~CUser() { // Delete networks - for (unsigned int c = 0; c < m_vIRCNetworks.size(); c++) { - CIRCNetwork* pNetwork = m_vIRCNetworks[c]; - delete pNetwork; + while (!m_vIRCNetworks.empty()) { + delete *m_vIRCNetworks.begin(); } - m_vIRCNetworks.clear(); // Delete clients for (unsigned int c = 0; c < m_vClients.size(); c++) {