Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • W weston
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 341
    • Issues 341
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 132
    • Merge requests 132
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • waylandwayland
  • weston
  • Issues
  • #352
Closed
Open
Issue created Jan 30, 2020 by Pekka Paalanen@pqMaintainer0 of 1 checklist item completed0/1 checklist item

Follow-up from "fix some problems related to weston_log_context"

The following discussion from !358 (merged) should be addressed:

  • @pq started a discussion: (+2 comments)

    I think this should be the last thing after everything else about subscribers and scopes are destroyed, because they are created from/for/into the log context.

    It looks like both weston_log_subscriber_destroy_log() and weston_log_subscriber_destroy_flight_rec() are broken: both leak struct weston_log_subscriber::subscription_list members. I think the current code relies on all scopes to have been destroyed first, which is implied by destroying the log context first. So maybe leave this for another time.

    Ok, this MR is good as is here. If you want to fix the destruction order and those functions in another MR, that would be welcome.

To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
Time tracking