Skip to content

Update mbed-coap to version 4.7.1 #8404

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

Merged
merged 1 commit into from
Oct 19, 2018

Conversation

anttiylitokola
Copy link
Contributor

Description

Add re-scan routine goto if message is caused user callback
This will fix hard fault when blockwise message sending timeouts. This happens cause same list is manipulated through rx callback.

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Add re-scan routine goto if message is caused user callback
This will fix hard fault when blockwise message sending timeouts. This happens cause same list is manipulated through rx callback.
@anttiylitokola
Copy link
Contributor Author

@yogpan01, please review.

@yogpan01
Copy link
Contributor

Approved. Fixes hardfault on devices which have lossy network like Thread/Mesh.
Please take this in along with #8273 for next Mbed OS patch release.

@teetak01
Copy link
Contributor

@0xc0170 can we get this for next patch release?

@teetak01
Copy link
Contributor

@0xc0170 @adbridge needed for 5.10.2

@adbridge
Copy link
Contributor

adbridge commented Oct 19, 2018

@teetak01 Technically this has missed the deadline for 5.10.2 but, I've started the ci for it and if it passes I will try and squeeze it in. If not we will get it in for 5.10.3

@adbridge
Copy link
Contributor

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2018

Build : SUCCESS

Build number : 3403
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8404/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2018

@mbed-ci
Copy link

mbed-ci commented Oct 19, 2018

@adbridge adbridge merged commit 83dada3 into ARMmbed:master Oct 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants