Skip to content

AES callbacks for CFB and OFB#10748

Open
night1rider wants to merge 2 commits into
wolfSSL:masterfrom
night1rider:AES-Callbacks
Open

AES callbacks for CFB and OFB#10748
night1rider wants to merge 2 commits into
wolfSSL:masterfrom
night1rider:AES-Callbacks

Conversation

@night1rider

Copy link
Copy Markdown
Contributor

Adding wiring for AES CFB/OFB callbacks to pass off to hardware.

@night1rider night1rider self-assigned this Jun 22, 2026
@night1rider night1rider marked this pull request as ready for review June 22, 2026 19:25
@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 #10748

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

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

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

@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 #10748

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

No new issues found in the changed files. ✅

@rizlik rizlik 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.

Hi @night1rider ,
I would love to new cryptocb to have ONLY_AES support for new callbacks from day 1.

@night1rider

Copy link
Copy Markdown
Contributor Author

Hi @night1rider , I would love to new cryptocb to have ONLY_AES support for new callbacks from day 1.

Hi @rizlik, I added swdev_aes_cfb/swdev_aes_ofb handlers so the new CFB/OFB callbacks are exercised all the way through under WOLF_CRYPTO_CB_ONLY_AES. I didn't end up needing any aes.c changes. The shim from c5ef06013 that encrypts one block at a time already composes CFB/OFB from the ECB callback under ONLY_AES, so the modes work as soon as the dedicated callbacks exist. swdev just didn't have a CFB/OFB backend yet (it only implemented CBC/CTR/ECB/GCM/CCM), so I added one so it can service those callbacks. Does that match what you had in mind, or were you expecting something on the library side too?

@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 #10748

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

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

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

@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 #10748

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

No new issues found in the changed files. ✅

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