Fix potential SEGV in Mueller effects (dresden-elektronik#7873)

The index in `RStateEffectValues` wasn't protected.
Smanar · Aug 6, 2024 · 56ad566 · 56ad566
1 parent d648c8b
commit 56ad566
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/de_web_plugin.cpp b/de_web_plugin.cpp
@@ -3690,7 +3690,10 @@ LightNode *DeRestPluginPrivate::updateLightNode(const deCONZ::NodeEvent &event)
                     {
                         if (RStateEffectValuesMueller.indexOf(lightNode->toString(RStateEffect), 0) <= 1)
                         {
-                            lightNode->setValue(RStateEffect, RStateEffectValues[ia->numericValue().u8]);
+                            if ((int)ia->numericValue().u8 < RStateEffectValues.size())
+                            {
+                                lightNode->setValue(RStateEffect, RStateEffectValues[ia->numericValue().u8]);
+                            }
                         }
                     }
                     else if (ia->id() == 0x4004) // color loop time