prasad-alatkar opened a new pull request #932: URL: https://github.com/apache/mynewt-nimble/pull/932
Issue: - When storage capacity of bonds is over, `ble_gap_unpair_oldest_peer` is called to make space for new bond. Then `ble_store_util_bonded_peers --> ble_store_iterate` are called. The callback `ble_store_util_iter_unique_peer` sets `set->status = BLE_HS_ENOMEM` when `max_peers` indicated by caller to `ble_store_util_bonded_peers` are iterated over in `ble_store_iterate`. - When `max_peer`(input parameter to API) is less than stored bonds, `ble_store_util_bonded_peers` returns `BLE_HS_ENOMEM` status code unnecessarily. It results in non deletion of oldest bond when `ble_gap_unpair_oldest_peer` is called. Solution: - `ble_store_util_bonded_peers` should not return error depending upon `set.status`. ---------------------------------------------------------------- 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]
