[gnutls-devel] [PATCH] Don't call _gnutls_cipher_encrypt2 with textlen = 0 in _gnutls_auth_cipher_encrypt2_tag
Matthias-Christian Ott
ott at mirix.org
Tue Dec 30 03:21:55 CET 2014
On 2014-12-30 02:15, Matthias-Christian Ott wrote:
> If the plaintext is shorter than the block size of the used cipher,
> _gnutls_auth_cipher_encrypt2_tag calls _gnutls_cipher_encrypt2 with
> textlen = 0. By definition _gnutls_cipher_encrypt2 does nothing in this
> case and thus does not need to be called.
There are more uses of _gnutls_cipher_encrypt2 where textlen could be
zero. Probably this needs some more thought and GnuTLS needs to make the
contracts between the functions explicit, especially the preconditions.
Please review the patch thoroughly. I'm not sure whether it introduces a
timing side channel.
Regards,
Matthias-Christian
More information about the Gnutls-devel
mailing list