Page MenuHomeFreeBSD

Don't panic for empty CCM requests.
ClosedPublic

Authored by jhb on Apr 24 2019, 10:45 PM.
Tags
None
Referenced Files
F153450428: D20043.id56613.diff
Tue, Apr 21, 6:07 AM
F153431085: D20043.id56612.diff
Tue, Apr 21, 3:25 AM
Unknown Object (File)
Mon, Apr 20, 7:52 PM
Unknown Object (File)
Sun, Apr 19, 1:31 PM
Unknown Object (File)
Thu, Apr 16, 12:25 AM
Unknown Object (File)
Wed, Apr 15, 6:04 AM
Unknown Object (File)
Fri, Apr 10, 1:46 PM
Unknown Object (File)
Fri, Apr 10, 3:44 AM
Subscribers

Details

Summary

A request to encrypt an empty payload without any AAD is unusual, but
it is defined behavior. Removing this assertion removes a panic and
instead returns the correct tag for an empty buffer.

Test Plan
  • 'cryptocheck -d soft -va aes-ccm 0' now gets a result matching OpenSSL instead of triggering a panic

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Something made me think that it was not acceptable, but now I can't remember what it was, so it's clearly wrong.

This revision is now accepted and ready to land.Apr 24 2019, 10:46 PM

Yeah, I think we talked about this in one of sef@'s reviews. It might've been a later one, though.

This revision was automatically updated to reflect the committed changes.