commit 6d8613dfb673d3419d9ed0802db5e7c36069b935
author fesseha-eve <88329315+fessehaeve@users.noreply.github.com> Tue Feb 27 15:43:59 2024 +0100
committer GitHub <noreply@github.com> Tue Feb 27 14:43:59 2024 +0000
tree 65193feead3993296cb6447d7d612bb34cc9241b
parent 8428564bed7736f8572e4df3d592bba8f1687e9a

fix crash caused by lacking null pointer, which is caused by using wrong cluster id (#32339)

src/app/clusters/on-off-server/on-off-server.cpp

diff --git a/src/app/clusters/on-off-server/on-off-server.cpp b/src/app/clusters/on-off-server/on-off-server.cpp
index ac1c4a0..e4ba82b 100644
--- a/src/app/clusters/on-off-server/on-off-server.cpp
+++ b/src/app/clusters/on-off-server/on-off-server.cpp
@@ -214,6 +214,7 @@
               ScenesManagement::ScenesServer::Instance().IsHandlerRegistered(endpoint, LevelControlServer::GetSceneHandler())))
 #endif
         {
+            VerifyOrReturnError(mTransitionTimeInterface.sceneEventControl(endpoint) != nullptr, CHIP_ERROR_INVALID_ARGUMENT);
             OnOffServer::Instance().scheduleTimerCallbackMs(mTransitionTimeInterface.sceneEventControl(endpoint), timeMs);
         }
 
@@ -221,7 +222,7 @@
     }
 
 private:
-    OnOffTransitionTimeInterface mTransitionTimeInterface = OnOffTransitionTimeInterface(Attributes::OnOff::Id, sceneOnOffCallback);
+    OnOffTransitionTimeInterface mTransitionTimeInterface = OnOffTransitionTimeInterface(OnOff::Id, sceneOnOffCallback);
 };
 static DefaultOnOffSceneHandler sOnOffSceneHandler;