Skip to content

Rust wrapper: Minor fixes and edits to support FIPS v5.2.3 module - #11895

Merged
holtrop-wolfssl merged 2 commits into
wolfSSL:masterfrom
gasbytes:rust-wrapper-fips-fixes-v5
Oct 27, 2026
Merged

holtrop-wolfssl merged 2 commits into
wolfSSL:masterfrom
gasbytes:rust-wrapper-fips-fixes-v5

Conversation

@gasbytes

Copy link
Copy Markdown
Contributor

Description

Extended the logic to handle the renaming of some functions for v5, used when the module exports a different symbol for the standard name, made sure it runs once per build and is cached. Added a layer around wc_HKDF_Extract (since wc_HKDF_Extract_ex doesn't exist on v5 modules) and wc_HKDF_Expand. Did the same for wc_rng_new.
Also did some module-specific gating and fixes, and updated some tests too.

Testing

Tested against the crate's testsuite and the rustls provider. On v5.2.3, wc_RsaPSS_VerifyCheck has no crate test and is only covered by the provider's tests.

@gasbytes
gasbytes requested a balanced review from Copilot October 26, 2026 15:02
@gasbytes gasbytes self-assigned this Oct 26, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@gasbytes
gasbytes marked this pull request as ready for review October 26, 2026 15:45
@github-actions

Copy link
Copy Markdown

retest this please

Comment thread wrapper/rust/wolfssl-wolfcrypt/src/rsa.rs
@gasbytes
gasbytes force-pushed the rust-wrapper-fips-fixes-v5 branch from 42b4e35 to 0cb812f Compare October 27, 2026 09:17
@gasbytes
gasbytes force-pushed the rust-wrapper-fips-fixes-v5 branch from 0cb812f to 9dbe646 Compare October 27, 2026 09:56
@holtrop-wolfssl
holtrop-wolfssl merged commit 8e45756 into wolfSSL:master Oct 27, 2026
406 of 407 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.

4 participants