Skip to content

Add common EC flags to mechanism info#419

Merged
simo5 merged 1 commit intolatchset:mainfrom
simo5:ec_flags
Feb 13, 2026
Merged

Add common EC flags to mechanism info#419
simo5 merged 1 commit intolatchset:mainfrom
simo5:ec_flags

Conversation

@simo5
Copy link
Member

@simo5 simo5 commented Feb 12, 2026

Description

Define COMMON_CKF_EC_FLAGS to include support for OIDs, named curves, and point compression. Apply these flags to ECDH, ECDSA, EdDSA, and Montgomery mechanism definitions to correctly advertise capabilities. Additionally, consolidate EC key size constants into the parent module.

Fixes: #416

Checklist

  • Test suite updated
  • Rustdoc string were added or updated
  • CHANGELOG and/or other documentation added or updated
  • This is not a code change

Reviewer's checklist:

  • Any issues marked for closing are fully addressed
  • There is a test suite reasonably covering new functionality or modifications
  • This feature/change has adequate documentation added
  • A changelog entry is added if the change is significant
  • Code conform to coding style that today cannot yet be enforced via the check style test
  • Commits have short titles and sensible text
  • Doc string are properly updated

@simo5 simo5 requested a review from Jakuje February 12, 2026 21:45
Define COMMON_CKF_EC_FLAGS to include support for OIDs, named curves,
and point compression. Apply these flags to ECDH, ECDSA, EdDSA, and
Montgomery mechanism definitions to correctly advertise capabilities.
Additionally, consolidate EC key size constants into the parent module.

Signed-off-by: Simo Sorce <simo@redhat.com>
Copy link
Contributor

@Jakuje Jakuje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. Thanks!

@simo5 simo5 merged commit 2225c3c into latchset:main Feb 13, 2026
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add CKF_EC_* flags to CKM_EC_* mechanisms

2 participants