chg: misc fixes

project-chip · Feb 11, 2023 · 2dd2edb · 2dd2edb
1 parent 2d53fb8
commit 2dd2edb
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 8 deletions.
diff --git a/src/app/clusters/color-control-server/color-control-server.cpp b/src/app/clusters/color-control-server/color-control-server.cpp
@@ -66,7 +66,9 @@ void ColorControlServer::cancelEndpointTimerCallback(EmberEventControl * control
 void ColorControlServer::cancelEndpointTimerCallback(EndpointId endpoint)
 {
     auto control = ColorControlServer::getEventControl(endpoint);
-    cancelEndpointTimerCallback(control);
+    if (control) {
+        cancelEndpointTimerCallback(control);
+    }
 }
 
 /**********************************************************

diff --git a/src/app/clusters/color-control-server/color-control-server.h b/src/app/clusters/color-control-server/color-control-server.h
@@ -247,7 +247,7 @@ class ColorControlServer
     Color16uTransitionState colorTempTransitionStates[EMBER_AF_COLOR_CONTROL_CLUSTER_SERVER_ENDPOINT_COUNT];
 #endif // EMBER_AF_PLUGIN_COLOR_CONTROL_SERVER_TEMP
 
-    EmberEventControl eventControls[EMBER_AF_COLOR_CONTROL_CLUSTER_SERVER_ENDPOINT_COUNT];
+    EmberEventControl eventControls[EMBER_AF_COLOR_CONTROL_CLUSTER_SERVER_ENDPOINT_COUNT + CHIP_DEVICE_CONFIG_DYNAMIC_ENDPOINT_COUNT];
 };
 
 /**********************************************************

diff --git a/src/app/clusters/level-control/level-control.cpp b/src/app/clusters/level-control/level-control.cpp
@@ -1294,6 +1294,7 @@ void emberAfLevelControlClusterServerInitCallback(EndpointId endpoint)
 
 void MatterLevelControlClusterServerShutdownCallback(EndpointId endpoint)
 {
+    emberAfOnOffClusterPrintln("Shuting down level control server cluster on endpoint %d", endpoint);
     DeviceLayer::SystemLayer().CancelTimer(timerCallback, reinterpret_cast<void *>(static_cast<uintptr_t>(endpoint)));
 }
 

diff --git a/src/app/clusters/on-off-server/on-off-server.cpp b/src/app/clusters/on-off-server/on-off-server.cpp
@@ -88,11 +88,14 @@ void OnOffServer::cancelEndpointTimerCallback(EmberEventControl * control)
 void OnOffServer::cancelEndpointTimerCallback(EndpointId endpoint)
 {
     auto control = OnOffServer::getEventControl(endpoint);
-    cancelEndpointTimerCallback(control);
+    if (control) {
+        cancelEndpointTimerCallback(control);
+    }
 }
 
 void MatterOnOffClusterServerShutdownCallback(EndpointId endpoint)
 {
+    emberAfOnOffClusterPrintln("Shuting down on/off server cluster on endpoint %d", endpoint);
     OnOffServer::Instance().cancelEndpointTimerCallback(endpoint);
 }
 

diff --git a/src/app/clusters/on-off-server/on-off-server.h b/src/app/clusters/on-off-server/on-off-server.h
@@ -93,7 +93,7 @@ class OnOffServer
      *********************************************************/
 
     static OnOffServer instance;
-    EmberEventControl eventControls[EMBER_AF_ON_OFF_CLUSTER_SERVER_ENDPOINT_COUNT];
+    EmberEventControl eventControls[EMBER_AF_ON_OFF_CLUSTER_SERVER_ENDPOINT_COUNT + CHIP_DEVICE_CONFIG_DYNAMIC_ENDPOINT_COUNT];
     chip::System::Clock::Timestamp nextDesiredOnWithTimedOffTimestamp;
 };
 

diff --git a/src/app/util/attribute-storage.cpp b/src/app/util/attribute-storage.cpp
@@ -904,10 +904,6 @@ bool emberAfEndpointEnableDisable(EndpointId endpoint, bool enable)
     {
         emAfEndpoints[index].bitmask |= EMBER_AF_ENDPOINT_ENABLED;
     }
-    else
-    {
-        emAfEndpoints[index].bitmask &= EMBER_AF_ENDPOINT_DISABLED;
-    }
 
 #if defined(EZSP_HOST)
     ezspSetEndpointFlags(endpoint, (enable ? EZSP_ENDPOINT_ENABLED : EZSP_ENDPOINT_DISABLED));
@@ -943,6 +939,11 @@ bool emberAfEndpointEnableDisable(EndpointId endpoint, bool enable)
                                                app::Clusters::Descriptor::Attributes::PartsList::Id);
     }
 
+    if (!enable)
+    {
+        emAfEndpoints[index].bitmask &= EMBER_AF_ENDPOINT_DISABLED;
+    }
+
     return true;
 }
-Original file line number
+Diff line change
@@ Expand Up @@
     void MatterLevelControlClusterServerShutdownCallback(EndpointId endpoint)
     {
+        emberAfOnOffClusterPrintln("Shuting down level control server cluster on endpoint %d", endpoint);
         DeviceLayer::SystemLayer().CancelTimer(timerCallback, reinterpret_cast<void *>(static_cast<uintptr_t>(endpoint)));
     }
@@ Expand Down @@