prevent crash in the color brewer color ramp when requested color

alexbruy · nyalldawson · commit bdbc3585d6c1 · 2019-05-28T06:09:25.000+10:00
corresponding to the NaN value (fix #29509)
diff --git a/src/core/qgscolorramp.cpp b/src/core/qgscolorramp.cpp
@@ -552,7 +552,7 @@ double QgsColorBrewerColorRamp::value( int index ) const
 
 QColor QgsColorBrewerColorRamp::color( double value ) const
 {
-  if ( mPalette.isEmpty() || value < 0 || value > 1 )
+  if ( mPalette.isEmpty() || value < 0 || value > 1 || std::isnan( value ) )
     return QColor();
 
   int paletteEntry = static_cast< int >( value * mPalette.count() );

Original file line number	Diff line number	Diff line change
`@@ -552,7 +552,7 @@ double QgsColorBrewerColorRamp::value( int index ) const`
`552`	`552`
`553`	`553`	`QColor QgsColorBrewerColorRamp::color( double value ) const`
`554`	`554`	`{`
`555`		`- if ( mPalette.isEmpty() \|\| value < 0 \|\| value > 1 )`
	`555`	`+ if ( mPalette.isEmpty() \|\| value < 0 \|\| value > 1 \|\| std::isnan( value ) )`
`556`	`556`	`return QColor();`
`557`	`557`
`558`	`558`	`int paletteEntry = static_cast< int >( value * mPalette.count() );`