[Bf-blender-cvs] [50733e14799] blender-v2.83-release: Fix T93322: Freestyle Sinus Displacement Division by Zero Crash

Philipp Oeser noreply at git.blender.org
Mon Jan 17 17:16:48 CET 2022


Commit: 50733e1479943d2652cbb3a60d92877674250b60
Author: Philipp Oeser
Date:   Tue Nov 23 11:48:44 2021 +0100
Branches: blender-v2.83-release
https://developer.blender.org/rB50733e1479943d2652cbb3a60d92877674250b60

Fix T93322: Freestyle Sinus Displacement Division by Zero Crash

This happens if the Wavelength is set to 0.0f.

Not sure if we really need a do_version patch for old files, as an
alternative we could also force a slight offset in the
SinusDisplacementShader. This patch does not do either, just force a
positive range from now on.

Maniphest Tasks: T93322

Differential Revision: https://developer.blender.org/D13329

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

M	source/blender/makesrna/intern/rna_linestyle.c

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

diff --git a/source/blender/makesrna/intern/rna_linestyle.c b/source/blender/makesrna/intern/rna_linestyle.c
index 788246f5edd..945f13fa90b 100644
--- a/source/blender/makesrna/intern/rna_linestyle.c
+++ b/source/blender/makesrna/intern/rna_linestyle.c
@@ -1306,8 +1306,9 @@ static void rna_def_linestyle_modifiers(BlenderRNA *brna)
       srna, "Sinus Displacement", "Add sinus displacement to stroke backbone geometry");
   rna_def_geometry_modifier(srna);
 
-  prop = RNA_def_property(srna, "wavelength", PROP_FLOAT, PROP_NONE);
+  prop = RNA_def_property(srna, "wavelength", PROP_FLOAT, PROP_UNSIGNED);
   RNA_def_property_float_sdna(prop, NULL, "wavelength");
+  RNA_def_property_range(prop, 0.0001f, FLT_MAX);
   RNA_def_property_ui_text(prop, "Wavelength", "Wavelength of the sinus displacement");
   RNA_def_property_update(prop, NC_LINESTYLE, "rna_LineStyle_update");



More information about the Bf-blender-cvs mailing list