-
Notifications
You must be signed in to change notification settings - Fork 3k
mbed-coap buffer overflow fix #12146
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
mbed-coap buffer overflow fix #12146
Conversation
@ristohuhtala, thank you for your changes. |
Added CHANGELOG.md changes to this PR like @yogpan01 suggested. |
As #12147 contains as well the changelog, please remove it from this one. This will be merged first, and 12147 right after containing the changelog. |
I will remove it |
CI started |
Test run: FAILEDSummary: 1 of 11 test jobs failed Failed test jobs:
|
mbed-os-ci_cloud-client-pytest restarted |
I'm assuming @JanneKiiskila will want it in next patch? |
Summary of changes
Fixed mbed-coap buffer overflow when paring CoAP message. Changed buffer handling when CoAP message is parsed. Buffer size is checked to avoid overflow.
Impact of changes
Migration actions required
Documentation
Pull request type
Test results
Reviewers
@anttiylitokola @yogpan01