Skip to content
Snippets Groups Projects

Fix several memory leaks

Merged 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

Pipeline #55991 passed

Pipeline passed for 844fcb46 on wip/smcv/memleaks

Merged by Ludovico de NittisLudovico de Nittis 1 year ago (Aug 2, 2023 9:44am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading