[GitHub] [mynewt-nimble] rymanluk commented on a change in pull request #1025: Add pairing complete ev

[GitBox <gi...@apache.org>]

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



##########
File path: nimble/host/src/ble_sm.c
##########
@@ -933,6 +933,12 @@ ble_sm_process_result(uint16_t conn_handle, struct ble_sm_result *res)
 
         ble_hs_unlock();
 
+        if (res->enc_cb &&
+            res->app_status != BLE_HS_ENOTCONN) {
+            /* Do not send this event on broken connection */
+            ble_gap_pairing_complete_event(conn_handle, res->sm_err);

Review comment:
       shouldn't we send here:
   
   res->app_status = BLE_HS_SM_PEER_ERR(cmd->reason); ?




-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org