Skip to content

Commit

Permalink
BUG: Fix crash when deleting multiple markups in subject hierarchy (#…
Browse files Browse the repository at this point in the history
…7736)

Add nullptr check to vtkSlicerMarkupsInteractionWidgetRepresentation::UpdateHandleToWorldTransform.

See discussion: https://discourse.slicer.org/t/reproducible-slicer-crash-deleting-markupsroinodes-from-subject-hierarchy/36007
  • Loading branch information
Sunderlandkyl committed May 10, 2024
1 parent bd2c929 commit bd476ee
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -559,6 +559,10 @@ void vtkSlicerMarkupsInteractionWidgetRepresentation::UpdateROIScaleHandles2D()
void vtkSlicerMarkupsInteractionWidgetRepresentation::UpdateHandleToWorldTransform(vtkTransform* handleToWorldTransform)
{
handleToWorldTransform->Identity();
if (!this->GetMarkupsNode())
{
return;
}
handleToWorldTransform->Concatenate(this->GetMarkupsNode()->GetInteractionHandleToWorldMatrix());
}

Expand Down

0 comments on commit bd476ee

Please sign in to comment.