Skip to content
Snippets Groups Projects

Fix mask deletion

Merged Mikhail Svechnikov requested to merge m.0 into main

Now two things are done before the mask item is deleted with MaskSet::delete_current():

  1. safely delete observable Overlay
  2. remove the mask item from m_mask2overlay map.

The related segfaults and memory leaks are gone.

Merge request reports

Pipeline #174062 passed

Pipeline passed for ce148556 on m.0

Approved by

Merged by Mikhail SvechnikovMikhail Svechnikov 5 months ago (Nov 21, 2024 3:43pm UTC)

Merge details

  • Changes merged into main with eae34820.
  • Did not delete the source branch.

Pipeline #174140 passed

Pipeline passed for eae34820 on main

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