diff options
author | Pascal van Leeuwen <pascalvanl@gmail.com> | 2019-08-09 17:51:07 +0200 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2019-08-15 21:52:14 +1000 |
commit | a62084d299d950f2ad0649caf9a9b9a431346996 (patch) | |
tree | f5dde31132c5ed33931bfe8494b718647b85b59c /crypto | |
parent | 440dc9aab5286aec46b6bdf061c6eec4fd53887f (diff) | |
download | linux-a62084d299d950f2ad0649caf9a9b9a431346996.tar.gz linux-a62084d299d950f2ad0649caf9a9b9a431346996.tar.bz2 linux-a62084d299d950f2ad0649caf9a9b9a431346996.zip |
crypto: aead - Do not allow authsize=0 if auth. alg has digestsize>0
Return -EINVAL on an attempt to set the authsize to 0 with an auth.
algorithm with a non-zero digestsize (i.e. anything but digest_null)
as authenticating the data and then throwing away the result does not
make any sense at all.
The digestsize zero exception is for use with digest_null for testing
purposes only.
Signed-off-by: Pascal van Leeuwen <pvanleeuwen@verimatrix.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto')
-rw-r--r-- | crypto/aead.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/crypto/aead.c b/crypto/aead.c index fbf0ec93bc8e..ce035589cf57 100644 --- a/crypto/aead.c +++ b/crypto/aead.c @@ -70,7 +70,8 @@ int crypto_aead_setauthsize(struct crypto_aead *tfm, unsigned int authsize) { int err; - if (authsize > crypto_aead_maxauthsize(tfm)) + if ((!authsize && crypto_aead_maxauthsize(tfm)) || + authsize > crypto_aead_maxauthsize(tfm)) return -EINVAL; if (crypto_aead_alg(tfm)->setauthsize) { |