[Bf-blender-cvs] [647578b1439] blender-v2.93-release: Fix T93322: Freestyle Sinus Displacement Division by Zero Crash

Philipp Oeser noreply at git.blender.org
Sun Nov 28 17:23:31 CET 2021


Commit: 647578b1439520544e26b119fe3d42e2dd0e0ff9
Author: Philipp Oeser
Date:   Tue Nov 23 11:48:44 2021 +0100
Branches: blender-v2.93-release
https://developer.blender.org/rB647578b1439520544e26b119fe3d42e2dd0e0ff9

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 ca97f2c9a55..da49ff9e846 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