Fix MeshShape::clear() for safer mesh removal (#3164) (#3167)

Co-authored-by: Matt Wang <[email protected]>
moveit · Dec 16, 2024 · ccc75f1 · ccc75f1
1 parent e94c891
commit ccc75f1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/moveit_ros/visualization/rviz_plugin_render_tools/src/mesh_shape.cpp b/moveit_ros/visualization/rviz_plugin_render_tools/src/mesh_shape.cpp
@@ -140,7 +140,11 @@ void MeshShape::clear()
   if (entity_)
   {
     entity_->detachFromParent();
-    Ogre::MeshManager::getSingleton().remove(entity_->getMesh()->getName());
+    const auto& mesh_name = entity_->getMesh()->getName();
+    if (Ogre::MeshPtr mesh = Ogre::MeshManager::getSingleton().getByName(mesh_name))
+    {
+      Ogre::MeshManager::getSingleton().remove(mesh);
+    }
     scene_manager_->destroyEntity(entity_);
     entity_ = nullptr;
   }