rymanluk commented on a change in pull request #836:
URL: https://github.com/apache/mynewt-nimble/pull/836#discussion_r447547201



##########
File path: nimble/controller/src/ble_ll_scan.c
##########
@@ -3021,7 +3021,8 @@ ble_ll_scan_rx_pkt_in_on_legacy(uint8_t pdu_type, struct 
os_mbuf *om,
 
     if (!BLE_MBUF_HDR_DEVMATCH(hdr) ||
         !BLE_MBUF_HDR_CRC_OK(hdr) ||
-        BLE_MBUF_HDR_IGNORED(hdr)) {
+        BLE_MBUF_HDR_IGNORED(hdr) ||
+        !scansm->scan_enabled) {

Review comment:
       isn't ignored in  ble_ll_scan_rx_isr_end()?
   
    ```
   case BLE_ADV_PDU_TYPE_ADV_EXT_IND:
           rc = ble_ll_scan_rx_isr_on_aux(pdu_type, rxbuf, hdr, &addrd);
           break;
   ```
   
   rc will be -1 on legacy scan 
   




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to