fix crash when using a group without a surface array

KhronosGroup · Jan 25, 2023 · 6f97895 · 6f97895
1 parent 72b7d59
commit 6f97895
Showing 1 changed file with 19 additions and 16 deletions.
diff --git a/libs/helide/scene/Group.cpp b/libs/helide/scene/Group.cpp
@@ -75,22 +75,25 @@ void Group::embreeSceneConstruct()
   rtcReleaseScene(m_embreeScene);
   m_embreeScene = rtcNewScene(deviceState()->embreeDevice);
 
-  uint32_t id = 0;
-  std::for_each(m_surfaceData->handlesBegin(),
-      m_surfaceData->handlesEnd(),
-      [&](Object *o) {
-        auto *s = (Surface *)o;
-        if (s && s->isValid()) {
-          m_surfaces.push_back(s);
-          rtcAttachGeometryByID(
-              m_embreeScene, s->geometry()->embreeGeometry(), id);
-        } else {
-          reportMessage(ANARI_SEVERITY_DEBUG,
-              "helide::Group rejecting invalid surface(%p) in building BLS",
-              s);
-        }
-        id++;
-      });
+  if (m_surfaceData) {
+    uint32_t id = 0;
+    std::for_each(m_surfaceData->handlesBegin(),
+        m_surfaceData->handlesEnd(),
+        [&](Object *o) {
+          auto *s = (Surface *)o;
+          if (s && s->isValid()) {
+            m_surfaces.push_back(s);
+            rtcAttachGeometryByID(
+                m_embreeScene, s->geometry()->embreeGeometry(), id);
+          } else {
+            reportMessage(ANARI_SEVERITY_DEBUG,
+                "helide::Group rejecting invalid surface(%p) in building BLS",
+                s);
+          }
+          id++;
+        });
+  }
+
   m_objectUpdates.lastSceneConstruction = helium::newTimeStamp();
   m_objectUpdates.lastSceneCommit = 0;
   embreeSceneCommit();