search

ARMmbed / ble-nrf51822

Nordic stack and drivers for the mbed BLE_API
Other
46 stars 51 forks source link

Fix incorrect handles of characteristics descriptors. #112

Closed pan- closed 8 years ago

pan- commented 8 years ago

The variable descriptorCount is used as an index, and this index is used to set the handle of the last registered descriptor.

Prior to this fix, descriptorCount was incremented before the handle the update of the handle of a characteristics.

Short story, all characteristics descriptors handle were equaols to 0 instead of the actual value.

ciarmcom commented 8 years ago

Automatic CI verification build not done, please verify manually.

pan- commented 8 years ago

@rgrover any comment on this ?

rgrover commented 8 years ago

thanks. looks better now.

pan- commented 8 years ago

retest this please.