nimble/host: fix ble_svc_gatt_cl_sup_feat_access

piotrnarajowski · piotrnarajowski · commit c00ffff3c774 · 2025-09-03T08:54:17.000+02:00
Fix ble_svc_gatt_cl_sup_feat_access. If an error occured while
client writes to client supported features only BLE_ATT_ERR_UNLIKELY
was returned.
diff --git a/nimble/host/services/gatt/src/ble_svc_gatt.c b/nimble/host/services/gatt/src/ble_svc_gatt.c
@@ -113,8 +113,9 @@ ble_svc_gatt_cl_sup_feat_access(uint16_t conn_handle, uint16_t attr_handle,
         return 0;
     }
     if (ctxt->op == BLE_GATT_ACCESS_OP_WRITE_CHR) {
-        if (ble_gatts_peer_cl_sup_feat_update(conn_handle, ctxt->om)) {
-            return BLE_ATT_ERR_UNLIKELY;
+        rc = ble_gatts_peer_cl_sup_feat_update(conn_handle, ctxt->om);
+        if (rc != 0) {
+            return rc;
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -113,8 +113,9 @@ ble_svc_gatt_cl_sup_feat_access(uint16_t conn_handle, uint16_t attr_handle,`
`113`	`113`	`return 0;`
`114`	`114`	`}`
`115`	`115`	`if (ctxt->op == BLE_GATT_ACCESS_OP_WRITE_CHR) {`
`116`		`- if (ble_gatts_peer_cl_sup_feat_update(conn_handle, ctxt->om)) {`
`117`		`- return BLE_ATT_ERR_UNLIKELY;`
	`116`	`+ rc = ble_gatts_peer_cl_sup_feat_update(conn_handle, ctxt->om);`
	`117`	`+ if (rc != 0) {`
	`118`	`+ return rc;`
`118`	`119`	`}`
`119`	`120`	`}`
`120`	`121`