X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/1544ba0edc8d1791d1d3a79bb0451831d0408051..f2c8079119e5887329d1cba5677498d9f6443f62:/src/common/cmdproc.cpp diff --git a/src/common/cmdproc.cpp b/src/common/cmdproc.cpp index 3d3d024abf..cb0d393049 100644 --- a/src/common/cmdproc.cpp +++ b/src/common/cmdproc.cpp @@ -122,13 +122,14 @@ void wxCommandProcessor::Store(wxCommand *command) while (node) { wxList::compatibility_iterator next = node->GetNext(); - delete (wxCommand *)node->GetData(); - m_commands.Erase(node); // Make sure m_lastSavedCommand won't point to freed memory - if ( m_lastSavedCommand == node ) + if ( m_lastSavedCommand && m_lastSavedCommand == node ) m_lastSavedCommand = wxList::compatibility_iterator(); + delete (wxCommand *)node->GetData(); + m_commands.Erase(node); + node = next; } } @@ -136,13 +137,14 @@ void wxCommandProcessor::Store(wxCommand *command) if ( (int)m_commands.GetCount() == m_maxNoCommands ) { wxList::compatibility_iterator firstNode = m_commands.GetFirst(); - wxCommand *firstCommand = (wxCommand *)firstNode->GetData(); - delete firstCommand; - m_commands.Erase(firstNode); // Make sure m_lastSavedCommand won't point to freed memory - if ( m_lastSavedCommand == firstNode ) + if ( m_lastSavedCommand && m_lastSavedCommand == firstNode ) m_lastSavedCommand = wxList::compatibility_iterator(); + + wxCommand *firstCommand = (wxCommand *)firstNode->GetData(); + delete firstCommand; + m_commands.Erase(firstNode); } m_commands.Append(command);