Skip to content

client: Free command parser after event loop

Ray Strode requested to merge fix-exit-crash into main

commit ed7b1690 added an explicit command parser free call on exit to make memory analysis tools give better results.

That commit freed the command parser a little too early, though, before the event loop gets freed and the event loop exit handlers are run.

The command parser uses an exit handler to clear a weak reference on the event loop, so freeing the command parser before the event loop can lead to crash.

This commit moves the free call a little lower.

Related: #175 (closed)

Edited by Ray Strode

Merge request reports