Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • W wireplumber
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 154
    • Issues 154
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 11
    • Merge requests 11
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and 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.

  • PipeWirePipeWire
  • wireplumber
  • Merge requests
  • !459

main.c: return transition error only once

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Ashok Sidipotu requested to merge main-crash-for-failed-plugin into master Nov 17, 2022
  • Overview 4
  • Commits 1
  • Pipelines 4
  • Changes 1

When a plugin activation fails, it is supposed to fail the wireplumber init. To be precise it fails the wp init transition and the init transition objected is freed.

However the subsequent plugins added(as reported by the plugin object manager) trigger double free, prevent this by taking a weak reference.

Fixes #371 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: main-crash-for-failed-plugin