A memory leak in test_midbus was recently fixed in a wrong place
1256df11 fixed the leak in midebugsession code by making a debugger plugin a debug session's parent. However, DebugController
is responsible for destroying debug sessions, not the debugger plugin. This works correctly in KDevelop. See my older fixes: 89ffddc7 and 37ff587e. Potentially destroying a debug session earlier, before the last safe moment in ~DebugController()
, may unnecessarily kill its debugger process. I think the leak should be fixed in the test instead. Does it even create a debug controller? If not, it should provide a minimal replacement for it.
@mwolff, since your commit is the trigger of this issue, could you please comment on it?