only set conn handle if adv stop event is successful

paul-szczepanek-arm · paul-szczepanek-arm · commit 0072938d3930 · 2021-06-09T14:32:55.000+01:00
diff --git a/connectivity/FEATURE_BLE/source/cordio/source/PalGapImpl.cpp b/connectivity/FEATURE_BLE/source/cordio/source/PalGapImpl.cpp
@@ -871,7 +871,7 @@ void PalGap::gap_handler(const wsfMsgHdr_t *msg)
 
             connection_handle_t connection_handle = DM_CONN_ID_NONE;
             /* the way we distinguish between local close and connection is the invalid HCI conn handle */
-            if (evt->handle != DM_CONN_HCI_HANDLE_NONE) {
+            if (evt->status == HCI_SUCCESS && evt->handle != DM_CONN_HCI_HANDLE_NONE) {
                 /* use the translated conn handle */
                 connection_handle = evt->hdr.param;
             }

Original file line number	Diff line number	Diff line change
`@@ -871,7 +871,7 @@ void PalGap::gap_handler(const wsfMsgHdr_t *msg)`
`871`	`871`
`872`	`872`	`connection_handle_t connection_handle = DM_CONN_ID_NONE;`
`873`	`873`	`/* the way we distinguish between local close and connection is the invalid HCI conn handle */`
`874`		`- if (evt->handle != DM_CONN_HCI_HANDLE_NONE) {`
	`874`	`+ if (evt->status == HCI_SUCCESS && evt->handle != DM_CONN_HCI_HANDLE_NONE) {`
`875`	`875`	`/* use the translated conn handle */`
`876`	`876`	`connection_handle = evt->hdr.param;`
`877`	`877`	`}`