diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2023-10-12 13:11:25 +0800 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2023-10-20 13:39:26 +0800 |
commit | 87d6621c07d230d9168a021f2760062f2262e8b9 (patch) | |
tree | 035abdb5ae2ebca42e901d31bbd5196047291be1 /crypto | |
parent | d4f5bfe20da9fa54024a73a9c60aea45e572d786 (diff) | |
download | linux-87d6621c07d230d9168a021f2760062f2262e8b9.tar.gz linux-87d6621c07d230d9168a021f2760062f2262e8b9.tar.bz2 linux-87d6621c07d230d9168a021f2760062f2262e8b9.zip |
crypto: lskcipher - Return EINVAL when ecb_name fails sanity checks
Set the error value to -EINVAL instead of zero when the underlying
name (within "ecb()") fails basic sanity checks.
Fixes: 8aee5d4ebd11 ("crypto: lskcipher - Add compatibility wrapper around ECB")
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Closes: https://lore.kernel.org/r/202310111323.ZjK7bzjw-lkp@intel.com/
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto')
-rw-r--r-- | crypto/lskcipher.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/crypto/lskcipher.c b/crypto/lskcipher.c index 9be3c04bc62a..cb6170ebcaa3 100644 --- a/crypto/lskcipher.c +++ b/crypto/lskcipher.c @@ -583,6 +583,7 @@ struct lskcipher_instance *lskcipher_alloc_instance_simple( if (ecb_name[0]) { int len; + err = -EINVAL; len = strscpy(ecb_name, &cipher_alg->co.base.cra_name[4], sizeof(ecb_name)); if (len < 2) |