Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bluetooth: Mesh: TTL max not checked on send #29855

Closed
trond-snekvik opened this issue Nov 7, 2020 · 0 comments · Fixed by #29856
Closed

Bluetooth: Mesh: TTL max not checked on send #29855

trond-snekvik opened this issue Nov 7, 2020 · 0 comments · Fixed by #29856
Assignees
@trond-snekvik
Labels
area: Bluetooth Mesh area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@trond-snekvik
Copy link
Contributor

The TTL max value is not checked in the message context when sending. This leads to unexpected behavior, as the TTL value wraps around at 128.
@trond-snekvik trond-snekvik added the bug The issue is a bug, or the PR is fixing a bug label Nov 7, 2020
@trond-snekvik trond-snekvik self-assigned this Nov 7, 2020
trond-snekvik added a commit to trond-snekvik/zephyr that referenced this issue Nov 7, 2020
@trond-snekvik
Bluetooth: Mesh: Check TTL max value on transport TX
1304fbc 
Adds check for TTL max in the transport send functions, and moves
setting of default TTL to transport.

Fixes zephyrproject-rtos#29855.

Signed-off-by: Trond Einar Snekvik <[email protected]>
@trond-snekvik trond-snekvik mentioned this issue Nov 7, 2020
Merged
@nashif nashif added the priority: low Low impact/importance bug label Nov 10, 2020
carlescufi pushed a commit that referenced this issue Nov 11, 2020
@trond-snekvik @carlescufi
Bluetooth: Mesh: Check TTL max value on transport TX
a2aad2b 
Adds check for TTL max in the transport send functions, and moves
setting of default TTL to transport.

Fixes #29855.

Signed-off-by: Trond Einar Snekvik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trond-snekvik trond-snekvik

Labels
area: Bluetooth Mesh area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bluetooth: Mesh: Check TTL max value on transport TX trond-snekvik/zephyr
2 participants
@nashif @trond-snekvik