Skip to content

Fix explicit Elliptic Curve parsing (3.6 backport)#10630

Draft
Marandil wants to merge 6 commits intoMbed-TLS:mbedtls-3.6from
Marandil:fix/mbedtls-4314-parse-explicit-curve
Draft

Fix explicit Elliptic Curve parsing (3.6 backport)#10630
Marandil wants to merge 6 commits intoMbed-TLS:mbedtls-3.6from
Marandil:fix/mbedtls-4314-parse-explicit-curve

Conversation

@Marandil
Copy link
Copy Markdown

@Marandil Marandil commented Mar 4, 2026

Description

Fixes #4314.

  • Fix pk_group_from_specified self-referencing group generator G while loading G.
  • Fix pk_group_from_specified not accounting for the a = p-3 internal representation.
  • Fix mbedtls_pk_parse_public_key failing to load keys with explicit curve parameters.

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

Notes for the submitter

Please refer to the contributing guidelines, especially the
checklist for PR contributors.

Help make review efficient:

  • Multiple simple commits
    • please structure your PR into a series of small commits, each of which does one thing
  • Avoid force-push
    • please do not force-push to update your PR - just add new commit(s)
  • See our Guidelines for Contributors for more details about the review process.

@gilles-peskine-arm gilles-peskine-arm added needs-ci Needs to pass CI tests priority-medium Medium priority - this can be reviewed as time permits size-s Estimated task size: small (~2d) labels Mar 5, 2026
Marandil added 5 commits March 5, 2026 14:47
Signed-off-by: Marcin Słowik <marcin@edgecorp.io>
Signed-off-by: Marcin Słowik <marcin@edgecorp.io>
Signed-off-by: Marcin Słowik <marcin@edgecorp.io>
Signed-off-by: Marcin Słowik <marcin@edgecorp.io>
…curve) pkparse tests.

Signed-off-by: Marcin Słowik <marcin@edgecorp.io>
@Marandil Marandil force-pushed the fix/mbedtls-4314-parse-explicit-curve branch from b7819c2 to 9c18d83 Compare March 5, 2026 14:04
Signed-off-by: Marcin Słowik <marcin@edgecorp.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci Needs to pass CI tests priority-medium Medium priority - this can be reviewed as time permits size-s Estimated task size: small (~2d)

Projects

Status: In Development

Development

Successfully merging this pull request may close these issues.

2 participants