Skip to content

Fix several memory leaks

Simon McVittie requested to merge wip/smcv/memleaks into main

An updated clang version in Debian testing/unstable is better at detecting memory leaks than previous versions, causing test failures. Unlike !577 (merged), I wasn't able to locate these until I recompiled GLib with ASan instrumentation.

/cc @denittis

  • lib: Fix several leaks when loading a JSON report

    In each case we weren't freeing the result of json_object_get_members(), which is (transfer container).

  • lib: Don't leak the GAppInfo that is the default for steam: URIs

  • tests: Don't leak the list of desktop entries

  • Revert "CI: Switch development container to Debian 12"

    This reverts commit 1982a2d4.

Edited by Simon McVittie

Merge request reports