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.

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

Bluetooth API called before finished initialization. #21105

Closed
joerchan opened this issue Dec 2, 2019 · 1 comment · Fixed by #21083
Closed

Bluetooth API called before finished initialization. #21105

joerchan opened this issue Dec 2, 2019 · 1 comment · Fixed by #21083
Assignees
Labels
area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@joerchan
Copy link
Contributor

joerchan commented Dec 2, 2019

Describe the bug
Some Bluetooth API function does not check for if the bluetooth device has finished initialized. This can lead to undefined behavior as the stack is operating on uninitialized kernel objects and memory.

To Reproduce
Steps to reproduce the behavior:
Call public API functions before calling bt_enable, or directly after bt_enable when supplied a ready callback.
Affected APIs:
OOB API
Gatt Notification and Indication
HCI commands

Expected behavior
APIs should return error code consistent with the rest of the API.

Impact
Undefined behavior. Minor annoyance, application should be able to not end up in this situation.

@joerchan joerchan added the bug The issue is a bug, or the PR is fixing a bug label Dec 2, 2019
@joerchan
Copy link
Contributor Author

joerchan commented Dec 2, 2019

Accidental enter hit. Will update description.

@joerchan joerchan self-assigned this Dec 2, 2019
@joerchan joerchan added area: Bluetooth priority: low Low impact/importance bug labels Dec 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant