Conduct platform shutdown after RunEventLoop returns (project-chip#8255)

Jagadish-NXP · Jul 9, 2021 · dce4293 · dce4293
1 parent 6c172a2
commit dce4293
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/src/messaging/tests/echo/common.cpp b/src/messaging/tests/echo/common.cpp
@@ -58,7 +58,6 @@ void InitializeChip(void)
 
 void ShutdownChip(void)
 {
-    chip::DeviceLayer::PlatformMgr().StopEventLoopTask();
     chip::DeviceLayer::PlatformMgr().Shutdown();
     gMessageCounterManager.Shutdown();
     gExchangeManager.Shutdown();

diff --git a/src/messaging/tests/echo/echo_requester.cpp b/src/messaging/tests/echo/echo_requester.cpp
@@ -98,12 +98,12 @@ void EchoTimerHandler(chip::System::Layer * systemLayer, void * appState, CHIP_E
         if (err != CHIP_NO_ERROR)
         {
             printf("Send request failed: %s\n", chip::ErrorStr(err));
-            Shutdown();
+            chip::DeviceLayer::PlatformMgr().StopEventLoopTask();
         }
     }
     else
     {
-        Shutdown();
+        chip::DeviceLayer::PlatformMgr().StopEventLoopTask();
     }
 }
 
@@ -276,12 +276,9 @@ int main(int argc, char * argv[])
 
     chip::DeviceLayer::PlatformMgr().RunEventLoop();
 
-exit:
-    if (err != CHIP_NO_ERROR)
-    {
-        Shutdown();
-    }
+    Shutdown();
 
+exit:
     if ((err != CHIP_NO_ERROR) || (gEchoRespCount != kMaxEchoCount))
     {
         printf("ChipEchoClient failed: %s\n", chip::ErrorStr(err));