This is an automated email from the ASF dual-hosted git repository.

sjanc pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git

commit ca6fd5d8916377ec7ca56cbf9d2540f13299d2e2
Author: Szymon Janc <[email protected]>
AuthorDate: Wed Oct 7 18:07:47 2026 +0200

    nimble/ll: Fix storing max subrate factor in LE Subrate Request
    
    Central stored requested Subrate_Max in acc_subrate_min, overwriting
    minimum value, while acc_subrate_max was left unchanged. This resulted
    in incorrect range being used when handling subrate requests from peer.
    
    Assisted-by: Claude:claude-opus-5-5
---
 nimble/controller/src/ble_ll_conn_hci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/nimble/controller/src/ble_ll_conn_hci.c 
b/nimble/controller/src/ble_ll_conn_hci.c
index 68362d037..ed35250df 100644
--- a/nimble/controller/src/ble_ll_conn_hci.c
+++ b/nimble/controller/src/ble_ll_conn_hci.c
@@ -1856,7 +1856,7 @@ ble_ll_conn_hci_subrate_req(const uint8_t *cmdbuf, 
uint8_t len,
 #if MYNEWT_VAL(BLE_LL_ROLE_CENTRAL)
     if (connsm->conn_role == BLE_LL_CONN_ROLE_CENTRAL) {
         connsm->acc_subrate_min = srp.subrate_min;
-        connsm->acc_subrate_min = srp.subrate_max;
+        connsm->acc_subrate_max = srp.subrate_max;
         connsm->acc_max_latency = srp.max_latency;
         connsm->acc_cont_num = srp.cont_num;
         connsm->acc_supervision_tmo = srp.supervision_tmo;

Reply via email to