Page MenuHomeFreeBSD
Differential D10273

Don't leak a session and lock if a GMAC key has an invalid length.
ClosedPublic
Actions

Authored by jhb on Apr 4 2017, 4:33 PM.
Tags
None
Referenced Files
F110127711: D10273.diff
Fri, Feb 14, 12:02 AM
Unknown Object (File)
Sun, Jan 26, 6:20 PM
Unknown Object (File)
Sat, Jan 25, 7:11 PM
Unknown Object (File)
Fri, Jan 24, 5:08 PM
Unknown Object (File)
Thu, Jan 23, 2:43 AM
Unknown Object (File)
Dec 13 2024, 5:57 PM
Unknown Object (File)
Dec 7 2024, 12:35 PM
Unknown Object (File)
Oct 27 2024, 2:35 PM
View All 74 Files
Subscribers
delphij
imp

Details

Reviewers
jmg
delphij
Group Reviewers
secteam
Commits
rS316510: Don't leak a session and lock if a GMAC key has an invalid length.
Test Plan
  • Run a buggy gcm test that did not set the hash key only the cipher key (so the hash key length was 0). Observe panic on return to userland due to leaked lock.

Diff Detail

Event Timeline

jhb created this revision.Apr 4 2017, 4:33 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 4 2017, 4:33 PM
delphij accepted this revision.Apr 4 2017, 5:06 PM
delphij added a subscriber: delphij.

Looks good to me.

This revision is now accepted and ready to land.Apr 4 2017, 5:06 PM
Harbormaster completed remote builds in B8498: Diff 27040.Apr 4 2017, 5:18 PM
Closed by commit rS316510: Don't leak a session and lock if a GMAC key has an invalid length. (authored by jhb). · Explain WhyApr 5 2017, 1:46 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
opencrypto/
5 lines

Diff 27040

View Options

sys/opencrypto/cryptosoft.c

Loading...