fix: Catch potential OCIO exception that we were missing (#4379)

Signed-off-by: Larry Gritz <[email protected]>
AcademySoftwareFoundation · Aug 21, 2024 · 956ce13 · 956ce13
1 parent a47c073
commit 956ce13
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/libOpenImageIO/color_ocio.cpp b/src/libOpenImageIO/color_ocio.cpp
@@ -1025,10 +1025,15 @@ bool
 ColorConfig::Impl::isColorSpaceLinear(string_view name) const
 {
     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;
+        }
     }
     return Strutil::iequals(name, "linear")
            || Strutil::istarts_with(name, "linear ")