Merge pull request #395 from fpistm/advertising_enabled

pennam · web-flow · commit 5855ec1a46a5 · 2025-04-14T10:10:02.000+02:00
fix(HCI): SetAdvertise only if enabled
diff --git a/src/utility/HCI.cpp b/src/utility/HCI.cpp
@@ -832,7 +832,10 @@ void HCIClass::handleEventPkt(uint8_t /*plen*/, uint8_t pdata[])
     ATT.removeConnection(disconnComplete->handle, disconnComplete->reason);
     L2CAPSignaling.removeConnection(disconnComplete->handle, disconnComplete->reason);
 
-    HCI.leSetAdvertiseEnable(0x01);
+    if (GAP.advertising())
+    {
+      HCI.leSetAdvertiseEnable(0x01);
+    }
   }
   else if (eventHdr->evt == EVT_ENCRYPTION_CHANGE)
   {

Original file line number	Diff line number	Diff line change
`@@ -832,7 +832,10 @@ void HCIClass::handleEventPkt(uint8_t /plen/, uint8_t pdata[])`
`832`	`832`	`ATT.removeConnection(disconnComplete->handle, disconnComplete->reason);`
`833`	`833`	`L2CAPSignaling.removeConnection(disconnComplete->handle, disconnComplete->reason);`
`834`	`834`
`835`		`- HCI.leSetAdvertiseEnable(0x01);`
	`835`	`+ if (GAP.advertising())`
	`836`	`+ {`
	`837`	`+ HCI.leSetAdvertiseEnable(0x01);`
	`838`	`+ }`
`836`	`839`	`}`
`837`	`840`	`else if (eventHdr->evt == EVT_ENCRYPTION_CHANGE)`
`838`	`841`	`{`