Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • S Solid
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 1
    • Issues 1
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 13
    • Merge requests 13
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • FrameworksFrameworks
  • Solid
  • Merge requests
  • !77

Don’t create a full Solid::Device to check encryption

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Méven Car requested to merge meven/solid:bug-398908 into master Feb 05, 2022
  • Overview 21
  • Commits 1
  • Pipelines 6
  • Changes 1

CCBUG: 398908

The heaptrack traces reported in the trace points towards those lines causing leaking: http://45.86.162.4/files/2022/heaptrack.dolphin.1944565_19.zst

Screenshot_20220205_125156

I don't understand where the leak occurs but there seem to be one in the Solid::Backends::UDisks2::Device::Device() ctor.

This change should help even-though it does not prevent the leak, it will make it less occurring.

Edited Feb 05, 2022 by Méven Car
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: bug-398908