Wrong verification in lib_aci_close_remote_pipe

NordicSemiconductor / ble-sdk-arduino

MIT License

249 stars 97 forks source link

Wrong verification in lib_aci_close_remote_pipe #3

Closed vladiulianbogdan closed 10 years ago

vladiulianbogdan commented 10 years ago

if((p_services_pipe_type_map[pipe-1].location == ACI_STORE_REMOTE)&& ((p_services_pipe_type_map[pipe-1].pipe_type == ACI_RX)|| (p_services_pipe_type_map[pipe-1].pipe_type == ACI_RX_ACK_AUTO)|| (p_services_pipe_type_map[pipe-1].pipe_type == ACI_RX_ACK))) { return false; }

It is wrong, i think. It will always return "false" if the pipe is ACI_RX and STORE REMOTE and it shouldn't.

Skjerve commented 10 years ago

Fixed in version 0.9.1.beta