fix: Catch potential OCIO exception that we were missing (AcademySoft…

…wareFoundation#4379) Signed-off-by: Larry Gritz <lg@larrygritz.com>
lgritz · Aug 22, 2024 · 89b49e2 · 89b49e2
1 parent 8d4e5b9
commit 89b49e2
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/libOpenImageIO/color_ocio.cpp b/src/libOpenImageIO/color_ocio.cpp
@@ -1074,10 +1074,15 @@ ColorConfig::Impl::isColorSpaceLinear(string_view name) const
 {
 #if OCIO_VERSION_HEX >= MAKE_OCIO_VERSION_HEX(2, 2, 0)
     if (config_ && !disable_builtin_configs && !disable_ocio) {
-        return config_->isColorSpaceLinear(c_str(name),
-                                           OCIO::REFERENCE_SPACE_SCENE)
-               || config_->isColorSpaceLinear(c_str(name),
-                                              OCIO::REFERENCE_SPACE_DISPLAY);
+        try {
+            return config_->isColorSpaceLinear(c_str(name),
+                                               OCIO::REFERENCE_SPACE_SCENE)
+                   || config_->isColorSpaceLinear(c_str(name),
+                                                  OCIO::REFERENCE_SPACE_DISPLAY);
+        } catch (const std::exception& e) {
+            error("ColorConfig error: {}", e.what());
+            return false;
+        }
     }
 #endif
     return Strutil::iequals(name, "linear")