possible crash candidate

[BlueHazzard]

Hi
i have found a possible crash candidate in macrosmanager.cpp:

start from Line 442:

Code Select Expand

    if (!target)
    {
        if (project)
        {
            // use the currently compiling target
            target = project->GetCurrentlyCompilingTarget();
            // if none,
            if (!target)
                // use the last known active target
                target = project->GetBuildTarget(project->GetActiveBuildTarget());
        }
    }

target can be a nullpointer but is never checked against it, and later:
line 470:

Code Select Expand

    while (m_RE_Script.Matches(buffer))
    {
        search = m_RE_Script.GetMatch(buffer, 1);
        wxString sc_name(_T("Replace Macro Target: "));
        sc_name.Append(target->GetTitle());
        replace = Manager::Get()->GetScriptingManager()->LoadBufferRedirectOutput(m_RE_Script.GetMatch(buffer, 2),sc_name);
        buffer.Replace(search, replace, false);
    }

it is used.

I work on my sqrat port and get a crash here. I'm still investigating why, but there should be a check for nullptr anyway

greetings

[oBFusCATed]

Post a patch please or some detailed steps to reproduce, even if it is against your branch.

[BlueHazzard]

mea culpa, i added this code by myself and fixed it now.

greetings