[development] Migrate from mbedtls_pk_can_do_ext to mbedtls_pk_can_do_psa (2/2) #10333
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
valeriosetti
added
needs-review
github-project-automation
bot
moved this to In Development
in Roadmap pull requests (new board)
4 tasks
valeriosetti
changed the title
valeriosetti
force-pushed
the
issue10266
branch
6 times, most recently
from
bd6acde to
cccb5b6
Compare
valeriosetti
force-pushed
the
issue10266
branch
from
cccb5b6 to
b43518d
Compare
valeriosetti
force-pushed
the
issue10266
branch
from
b43518d to
49627a6
Compare
valeriosetti
removed
the
needs-reviewer
valeriosetti
force-pushed
the
issue10266
branch
3 times, most recently
from
500fbe3 to
b71ac3c
Compare
valeriosetti
added
needs-reviewer
gilles-peskine-arm
added
needs-work
and removed
needs-review
github-project-automation
bot
moved this from In Development
to Has Approval
in Roadmap pull requests (new board)
gilles-peskine-arm
added
needs-work
and removed
needs-review
|
Sorry, another submodule update conflict. |
Signed-off-by: Valerio Setti <[email protected]>
When the key is parsed from PK it is assigned the pseudo-alg MBEDTLS_PK_ALG_ECDSA. Trying to run "mbedtls_pk_can_do_psa" with an hardcoded deterministc/randomized ECDSA can make the function to fail if the proper variant is not the one also used by PK. This commit fixes this problem. Signed-off-by: Valerio Setti <[email protected]>
Signed-off-by: Valerio Setti <[email protected]>
Signed-off-by: Valerio Setti <[email protected]>
Revert changes previously done at following test cases: - Handshake, select ECDHE-ECDSA-WITH-AES-256-CCM, opaque, PSA_ALG_ANY_HASH - Handshake, select ECDHE-ECDSA-WITH-AES-256-CCM, opaque, PSA_ALG_SHA_256 Signed-off-by: Valerio Setti <[email protected]>
valeriosetti
dismissed stale reviews from gilles-peskine-arm and mpg
via
e2aed3a
valeriosetti
force-pushed
the
issue10266
branch
from
20afd7d to
e2aed3a
Compare
valeriosetti
added
needs-review
gilles-peskine-arm
added
approved
Merged
via the queue into
Mbed-TLS:development
with commit Sep 16, 2025
d66898e
6 of 8 checks passed
github-project-automation
bot
moved this from Has Approval
to Done
in Roadmap pull requests (new board)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Resolves #10266
Depends on:
This is the PR 2 out of 2.
PR checklist