[Bf-blender-cvs] [0697631929d] blender-v3.4-release: Fix T102144: missing validation of spline resolution

Jacques Lucke noreply at git.blender.org
Sun Nov 6 15:37:04 CET 2022


Commit: 0697631929d9ec4be92feae9b8347dd1e32c6ddd
Author: Jacques Lucke
Date:   Sun Nov 6 15:36:53 2022 +0100
Branches: blender-v3.4-release
https://developer.blender.org/rB0697631929d9ec4be92feae9b8347dd1e32c6ddd

Fix T102144: missing validation of spline resolution

===================================================================

M	source/blender/nodes/geometry/nodes/node_geo_set_spline_resolution.cc

===================================================================

diff --git a/source/blender/nodes/geometry/nodes/node_geo_set_spline_resolution.cc b/source/blender/nodes/geometry/nodes/node_geo_set_spline_resolution.cc
index d46ceac92ba..dcd910b8ad2 100644
--- a/source/blender/nodes/geometry/nodes/node_geo_set_spline_resolution.cc
+++ b/source/blender/nodes/geometry/nodes/node_geo_set_spline_resolution.cc
@@ -24,11 +24,13 @@ static void set_resolution(bke::CurvesGeometry &curves,
   MutableAttributeAccessor attributes = curves.attributes_for_write();
   AttributeWriter<int> resolutions = attributes.lookup_or_add_for_write<int>("resolution",
                                                                              ATTR_DOMAIN_CURVE);
+  bke::AttributeValidator validator = attributes.lookup_validator("resolution");
 
   bke::CurvesFieldContext field_context{curves, ATTR_DOMAIN_CURVE};
   fn::FieldEvaluator evaluator{field_context, curves.curves_num()};
   evaluator.set_selection(selection_field);
-  evaluator.add_with_destination(resolution_field, resolutions.varray);
+  evaluator.add_with_destination(validator.validate_field_if_necessary(resolution_field),
+                                 resolutions.varray);
   evaluator.evaluate();
 
   resolutions.finish();



More information about the Bf-blender-cvs mailing list