BUG: Fix Model DM crash when adding display node

* Fix vtkMRMLModelDisplayableManager crash if added model display node doesn't derive from vtkMRMLModelDisplayNode
lassoan · Jan 9, 2025 · 2e8989a · 2e8989a
1 parent ab9ff4a
commit 2e8989a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/Libs/MRML/DisplayableManager/vtkMRMLModelDisplayableManager.cxx b/Libs/MRML/DisplayableManager/vtkMRMLModelDisplayableManager.cxx
@@ -1018,7 +1018,7 @@ void vtkMRMLModelDisplayableManager::UpdateModelMesh(vtkMRMLDisplayableNode* dis
       }
     }
 
-    if (hasMesh && ait == this->Internal->DisplayedActors.end())
+    if (hasMesh && modelDisplayNode && ait == this->Internal->DisplayedActors.end())
     {
       this->GetRenderer()->AddViewProp(prop);
       this->Internal->DisplayedActors[modelDisplayNode->GetID()] = prop;
@@ -1033,7 +1033,7 @@ void vtkMRMLModelDisplayableManager::UpdateModelMesh(vtkMRMLDisplayableNode* dis
         this->Internal->DisplayedClipState[modelDisplayNode->GetID()] = 0;
       }
     }
-    else
+    else if (modelDisplayNode)
     {
       if (clipper)
       {
@@ -1045,7 +1045,7 @@ void vtkMRMLModelDisplayableManager::UpdateModelMesh(vtkMRMLDisplayableNode* dis
       }
     }
 
-    if (hasMesh && cait == this->Internal->DisplayedCapActors.end())
+    if (hasMesh && modelDisplayNode && cait == this->Internal->DisplayedCapActors.end())
     {
       this->GetRenderer()->AddViewProp(capProp);
       this->Internal->DisplayedCapActors[modelDisplayNode->GetID()] = capProp;