Ensured status was always initialised before returning, added missing…

… break in switch case
project-chip · Apr 28, 2023 · 71ea59b · 71ea59b
1 parent 18d2634
commit 71ea59b
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/app/clusters/fan-control-server/fan-control-server.cpp b/src/app/clusters/fan-control-server/fan-control-server.cpp
@@ -136,6 +136,10 @@ MatterFanControlClusterServerPreAttributeChangedCallback(const ConcreteAttribute
             }
             res = Status::Success;
         }
+        else
+        {
+            res = Status::Success;
+        }
         break;
     }
     case SpeedSetting::Id: {
@@ -384,6 +388,7 @@ void MatterFanControlClusterServerAttributeChangedCallback(const app::ConcreteAt
                                ChipLogError(Zcl, "Failed to set PercentSetting with error: 0x%02x", status));
             }
         }
+        break;
     }
     default:
         break;