[Telink] Fixed hanging on BLE Shutdown (TC-SC-4.1) (#30152)

interfer · restyled-commits · pull[bot] · commit 141129449592 · 2023-12-08T22:27:14.000Z
* [Telink] Fixed hanging on BLE Shutdown (TC-SC-4.1)

Refactored BLE shutdown condition to pass test TC-SC-4.1.
Device would become stuck at scheduler routines after open-basic-commissioning-window time expires.

Signed-off-by: Dmytro Huz  &lt;dmytro.huz@telink-semi.com&gt;

* Restyled by clang-format

---------

Signed-off-by: Dmytro Huz  &lt;dmytro.huz@telink-semi.com&gt;
Co-authored-by: Restyled.io &lt;commits@restyled.io&gt;
diff --git a/src/platform/telink/BLEManagerImpl.cpp b/src/platform/telink/BLEManagerImpl.cpp
@@ -180,8 +180,11 @@ CHIP_ERROR BLEManagerImpl::_Init(void)
 
 void BLEManagerImpl::_Shutdown()
 {
-    bt_disable();
-    mBLERadioInitialized = false;
+    if (mBLERadioInitialized)
+    {
+        bt_disable();
+        mBLERadioInitialized = false;
+    }
 }
 
 void BLEManagerImpl::DriveBLEState(intptr_t arg)

Original file line number	Diff line number	Diff line change
`@@ -180,8 +180,11 @@ CHIP_ERROR BLEManagerImpl::_Init(void)`
`180`	`180`
`181`	`181`	`void BLEManagerImpl::_Shutdown()`
`182`	`182`	`{`
`183`		`- bt_disable();`
`184`		`- mBLERadioInitialized = false;`
	`183`	`+ if (mBLERadioInitialized)`
	`184`	`+ {`
	`185`	`+ bt_disable();`
	`186`	`+ mBLERadioInitialized = false;`
	`187`	`+ }`
`185`	`188`	`}`
`186`	`189`
`187`	`190`	`void BLEManagerImpl::DriveBLEState(intptr_t arg)`