-
Notifications
You must be signed in to change notification settings - Fork 8k
Bluetooth: Classic: Add length check of user input #74283
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: Classic: Add length check of user input #74283
Conversation
Hello @ekleezg, and thank you very much for your first pull request to the Zephyr project! |
@ekleezg thanks, but no need to split these into multiple commits since they're all quite related to each other. Just squash the changes into a single commit and force push. Also add a commit message body (i.e. some additional text than just the commit subject), since the Compliance CI check will otherwise fail. |
1a778e1
to
4fffdee
Compare
4fffdee
to
bbfd9c0
Compare
@ekleezg you need to amend your git commit author name/email so that it matches the sign-off entry. Thanks! |
Added length checks for user input in `sdp_client_receive` and `l2cap_br_info_rsp`. Signed-off-by: Eunkyu Lee <[email protected]>
bbfd9c0
to
4133127
Compare
Thank you for let me know. I changed it. |
Hi @ekleezg! To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge. Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁 |
Added a check for the length of user input as it was missing.