HomePhabricator
Diffusion MITK 9235478ae848

Fix weak pointer usage

Description

Fix weak pointer usage

Summary:
Fix expiry checks before actual use of objects referenced by mitk::WeakPointer or berry::WeakPointer.

A call to .IsExpired() or .Expired() does not guarantee that the reference is still valid after the check.
Actually .Lock() the pointer and check against null to ensure valid pointers.

Fixes T27136

Test Plan: code review

Reviewers: floca, O3 MITK Reviewer Group II, O1 MITK Reviewer Group I

Reviewed By: floca, O3 MITK Reviewer Group II, O1 MITK Reviewer Group I

Maniphest Tasks: T27136

Differential Revision: https://phabricator.mitk.org/D636

Details

Provenance
kislinskAuthored on May 4 2022, 7:58 PM
kislinskPushed on May 4 2022, 8:22 PM
Reviewer
O3: MITK Reviewer Group II
Differential Revision
Restricted Differential Revision
Parents
rMITKad68dae38623: Fix return types for locked weak pointer getters
rMITK0944e0d5e6a4: Rename sections label.
Branches
Unknown
Tags
Unknown
Tasks
T27136: Ensure correct pattern of mitk::WeakPointer::Lock()