The loops for Chacha20 and Chacha20+Poly1305 which encrypted/decrypted full blocks of data used the minimum of the input and output segment lengths to determine the size of the next chunk ('todo') to pass to Chacha20_ctr32(). However, the input and output segments could extend past the end of the ciphertext region into the tag (e.g. if a "plain" single mbuf contained an entire TLS record). If the length of the tag plus the length of the last partial block together were at least as large as a full Chacha20 block (64 bytes), then an extra block was encrypted/decrypted overlapping with the tag. Fix this by also capping the amount of data to encrypt/decrypt by the amount of remaining data in the ciphertext region ('resid'). Reported by: gallatin Reviewed by: cem, gallatin, markj Sponsored by: Netflix Differential Revision: https://reviews.freebsd.org/D29517 |
||
---|---|---|
.. | ||
aesni | ||
armv8 | ||
blake2 | ||
camellia | ||
ccp | ||
chacha20 | ||
des | ||
libsodium | ||
openssl | ||
rc4 | ||
rijndael | ||
sha2 | ||
siphash | ||
skein | ||
via | ||
intake.h | ||
sha1.c | ||
sha1.h |