Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • chatty chatty
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 317
    • Issues 317
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 19
    • Merge requests 19
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

  • Librem5
  • chattychatty
  • Merge requests
  • !1058

mmsd: Fix various memory leaks

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Mohammed Sadiq requested to merge mohammed.sadiq/chatty:wip/sadiq/fix-mmsd-leaks into master Feb 04, 2022
  • Overview 3
  • Commits 1
  • Pipelines 1
  • Changes 1

This commit also sets initial value of 'savepath' to NULL, so that it won't point to any invalid memory.

Likely fixes #674 (closed)

/cc @kop316 @scottt @lafleur (let me know if this MR fixes the crash). You can get the installable .deb package from the CI (from the green check button once build is completed)

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: wip/sadiq/fix-mmsd-leaks