restart_advertising_id: Prevent use of illegal ID - Githubissues

sekigon-gonnoc / qmk_firmware

keyboard controller firmware for Atmel AVR and ARM USB families

http://qmk.fm

GNU General Public License v2.0

202 stars 82 forks source link

restart_advertising_id: Prevent use of illegal ID #25

Closed Lenbok closed 4 years ago

Lenbok commented 4 years ago

Description

Fix crash when given an ID one greater than permitted.
For split keyboards, ID 0 is reserved for slave, so make ADV_ID0 a no-op.

Types of Changes

[x] Core
[x] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[ ] Keyboard (addition or update)
[ ] Keymap/layout/userspace (addition or update)
[ ] Documentation

Issues Fixed or Closed by This PR

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have tested the changes and verified that they work and don't break anything (as well as I can manage).