-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Update Mbed TLS to version 2.13.0 #7974
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I've verified the mbedtls code is identical to that in #7940, except that the version information has been updated.
/morph build |
Not to be merged until release approval has been obtained by the TLS team and communicated to us |
Build : FAILUREBuild number : 2999 |
I'm pleased to say this version has been approved for release, so the 'DO NOT MERGE' label can be removed. |
/morph build Yay, a new CI issue. The node failued to report the build success... |
Build : SUCCESSBuild number : 3000 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 2610 |
Test : SUCCESSBuild number : 2763 |
Does not link succesfully with ARMCC & IAR: x509.c@907,0: #223-D: function "gmtime_r" declared implicitly |
Tested with armcc 5.06 |
@juhoeskeli This issue is opened in Mbed-TLS/mbedtls#1907 |
Description
This PR updates Mbed TLS to the latest release, 2.13.0.
This PR supersedes PR #7940 which unfortunately I can't update, so have had to provide a new PR. #7940 can now be closed.
Note, this PR is title/tagged 'DO NOT MERGE' whilst formal release approval is pending.
PR should be reviewed by @RonEld and/or @Patater where available.
Pull request type