Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

memory leak in m_debuggerMenuHandler in SRC target

(1/1)

ollydbg:

--- Code: ---void MainFrame::SetupDebuggerUI()
{
    m_debuggerMenuHandler = new DebuggerMenuHandler;
    m_debuggerToolbarHandler = new DebuggerToolbarHandler(m_debuggerMenuHandler);
    m_debuggerMenuHandler->SetEvtHandlerEnabled(false);
    m_debuggerToolbarHandler->SetEvtHandlerEnabled(false);
    wxWindow* window = Manager::Get()->GetAppWindow();
    if (window)
    {
        window->PushEventHandler(m_debuggerMenuHandler);
        window->PushEventHandler(m_debuggerToolbarHandler);
    }
    m_debuggerMenuHandler->SetEvtHandlerEnabled(true);
    m_debuggerToolbarHandler->SetEvtHandlerEnabled(true);
....

--- End code ---

Question: I can't find there is a place to "delete m_debuggerMenuHandler", does it mean the wxwidgets framework can delete it? Or there is memory leak?

oBFusCATed:

--- Quote from: ollydbg on April 17, 2013, 02:36:58 am ---Or there is memory leak?

--- End quote ---
Just a memory leak. wxWidgets deletes only window type of object, I suppose.
I'll take care of it.

ollydbg:
Fev 8995 fix one memory leak(m_debuggerMenuHandler), what about m_debuggerToolbarHandler?

oBFusCATed:

--- Quote from: ollydbg on April 18, 2013, 04:04:03 pm ---Fev 8995 fix one memory leak(m_debuggerMenuHandler), what about m_debuggerToolbarHandler?

--- End quote ---
Fix it the same way.

ollydbg:

--- Quote from: oBFusCATed on April 18, 2013, 05:05:25 pm ---
--- Quote from: ollydbg on April 18, 2013, 04:04:03 pm ---Fev 8995 fix one memory leak(m_debuggerMenuHandler), what about m_debuggerToolbarHandler?

--- End quote ---
Fix it the same way.

--- End quote ---
I fixed it in rev 8998.

Navigation

[0] Message Index