Skip to content

Add SM cipher crypto cb paths - #10857

Open
padelsbach wants to merge 4 commits into
wolfSSL:masterfrom
padelsbach:sm-crypto-cb
Open

Add SM cipher crypto cb paths#10857
padelsbach wants to merge 4 commits into
wolfSSL:masterfrom
padelsbach:sm-crypto-cb

Conversation

@padelsbach

@padelsbach padelsbach commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Description

Add the cryptocb functionality and plumbing for the SM algs.

Note that the wolfsm changes are needed for CI testing in wolfssl. Those will be added as follow-up PRs.

Testing

Added new test cases using swdev, but they are not enabled until #10857 is merged.

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@padelsbach
padelsbach force-pushed the sm-crypto-cb branch 2 times, most recently from d047c96 to fc9c156 Compare August 12, 2026 04:51
@padelsbach
padelsbach marked this pull request as ready for review August 12, 2026 05:14
@github-actions

Copy link
Copy Markdown

retest this please

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #10857

Scan targets checked: wolfcrypt-bugs, wolfcrypt-port-bugs, wolfcrypt-rs-bugs, wolfcrypt-src, wolfssl-bugs, wolfssl-src

Findings: 9
9 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread wolfcrypt/test/test.c
Comment thread wolfssl/wolfcrypt/types.h Outdated
Comment thread wolfcrypt/test/test.c Outdated
Comment thread wolfcrypt/test/test.c
Comment thread wolfcrypt/test/test.c
Comment thread wolfcrypt/test/test.c
Comment thread wolfcrypt/test/test.c Outdated
Comment thread wolfcrypt/test/test.c
Comment thread wolfcrypt/test/test.c Outdated
@padelsbach

padelsbach commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

jenkins retest this please

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.

2 participants