Page MenuHomeFreeBSD

rijndael (AES): Avoid leaking sensitive data on kernel stack
ClosedPublic

Authored by cem on Aug 29 2018, 4:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 6:40 AM
Unknown Object (File)
Mon, Apr 22, 1:41 PM
Unknown Object (File)
Feb 1 2024, 12:38 PM
Unknown Object (File)
Dec 20 2023, 4:28 AM
Unknown Object (File)
Oct 30 2023, 1:26 PM
Unknown Object (File)
Sep 5 2023, 9:19 PM
Unknown Object (File)
Sep 3 2023, 4:23 AM
Unknown Object (File)
Sep 3 2023, 4:22 AM
Subscribers

Details

Summary

Noticed this investigating Fortuna. Remove useless duplicate stack copies
of sensitive contents when possible, or if not possible, be sure to zero
them out when we're finished.

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 19256
Build 18869: arc lint + arc unit

Event Timeline

Mostly LGTM.

sys/crypto/rijndael/rijndael-api-fst.c
448

Could you please change this to use some better name, e.g. something like 'cleanup' instead?

sys/crypto/rijndael/rijndael-api-fst.c
448

I think 'out' is well understood:

$ cd sys
$ ag 'goto out;' | wc -l
    5333
$ ag 'goto cleanup;' | wc -l
     222

I can change it if you still prefer it, but I don't think "cleanup" is any better (or worse).

Ping -- do you still want a different label or is out ok? Thanks!

gordon added a subscriber: gordon.

Approved based on timeout from delphij.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 26 2018, 8:53 PM
This revision was automatically updated to reflect the committed changes.