Page MenuHomeFreeBSD

aesni: Avoid modifying session keys in hmac_update()
ClosedPublic

Authored by markj on Apr 27 2021, 12:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 12, 9:38 PM
Unknown Object (File)
Sun, Feb 8, 1:46 PM
Unknown Object (File)
Sat, Feb 7, 5:10 PM
Unknown Object (File)
Fri, Jan 30, 8:33 PM
Unknown Object (File)
Fri, Jan 16, 5:25 PM
Unknown Object (File)
Dec 30 2025, 3:20 AM
Unknown Object (File)
Dec 20 2025, 9:29 PM
Unknown Object (File)
Dec 20 2025, 7:18 AM
Subscribers
None

Details

Summary

Otherwise aesni_process() is not thread-safe for AES+SHA-HMAC
transforms, since hmac_update() updates the caller-supplied key directly
to create the derived key. Use a buffer on the stack to store a copy of
the key used for computing inner and outer digests.

Note that this applies to stable/12 only.

Diff Detail

Repository
rG FreeBSD src repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 38878
Build 35767: arc lint + arc unit

Event Timeline

markj created this revision.
This revision is now accepted and ready to land.Apr 27 2021, 2:09 PM