-
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
crypto: Update to Mbed Crypto 1.0.0d5 and then to 1.0.0d6 #9795
Conversation
Can you add Release notes section above ? |
There is no functionality change between released versions of Mbed OS, only between Mbed Crypto as present in the yet unmerged #9708 and this PR. Do we still want release notes? |
If 9708 captures it, then all fine. |
After updating this PR to include Mbed Crypto 1.0.0d6, there will indeed be a functional change. |
@Patater, thank you for your changes. |
af71d3f
to
7da5771
Compare
Rebased to update to 1.0.0d6, which does actually have some functional changes. Will add release notes. |
@@ -29,7 +29,7 @@ | |||
|
|||
# Set the Mbed Crypto release to import (this can/should be edited before | |||
# import) | |||
CRYPTO_RELEASE ?= mbedcrypto-1.0.0d2 | |||
CRYPTO_RELEASE ?= mbedcrypto-1.0.0d6 |
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.
did you mean mbedcrypto-1.0.0d5 or mbedcrypto-1.0.0d6
as in the headline?
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.
Updated headline to explain what we are doing now. Originally, this was just d5, now it is both d5 and then d6.
7da5771
to
e5c1e0b
Compare
Rebased atop latest master. |
@Patater As it's Friday, reviews are expected on Sunday due weekend? I'll schedule CI test run now |
Test run: FAILEDSummary: 1 of 8 test jobs failed Failed test jobs:
|
Restarting CI (failure unrelated, and seems in the CI rather) |
Test run: SUCCESSSummary: 12 of 12 test jobs passed |
@dgreen-arm has approved on behalf of @ARMmbed/mbed-os-crypto Awaiting review from @avolinski team |
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.
A bit concerned about the PSA defines used since a lot of PSA activity is moving about, but I think it should be fine.
|
Description
Update Mbed Crypto to development snapshot 1.0.0d5. This brings backwards compatbility with previous PSA Crypto error codes for use by the PSA Compliance tests.
Depends on PR #9708 and will need a rebase atop it afterwards.
Pull request type
Reviewers
@ARMmbed/mbed-os-crypto, @avolinski
Release Notes
Previously, calls to
psa_crypto_*_setup()
would succeed even if a user did not initialize their operation context. Help users of the PSA Crypto API catch their own bugs by returningPSA_ERROR_BAD_STATE
when a setup function is called with an uninitialized context.