[GitHub] [mynewt-nimble] KKopyscinski opened a new pull request, #1520: apps/bttester: fix retrieving app included services

["KKopyscinski (via GitHub)" <gi...@apache.org>]

KKopyscinski opened a new pull request, #1520:
URL: https://github.com/apache/mynewt-nimble/pull/1520

   Local Included Service(s) values are retrieved using `ble_att_svr_read_handle`, which will lead to calling `entry->ha_cb` for Included Service. This will be `ble_gatts_inc_access`, which will not add UUID if it's not of 16 bit type. Use 16 bit UUID for included service.
   
   `out_att_err` is now set to 0 by default - it may be not overwritten if no error occured. This will lead to undefined behavior and incorrect data in BTP response.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [mynewt-nimble] sjanc merged pull request #1520: apps/bttester: fix retrieving app included services

["sjanc (via GitHub)" <gi...@apache.org>]

sjanc merged PR #1520:
URL: https://github.com/apache/mynewt-nimble/pull/1520


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org