diff options
author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2020-07-12 23:14:04 +0200 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2020-07-16 13:49:14 +0200 |
commit | d110cf0ac1bf24ac098b8ea5b0188c717acf7a02 (patch) | |
tree | 1c0c81e213f2dd4a2de933109c65ee605873c74f /drivers/crypto | |
parent | crypto: lrw - prefix function and struct names with "lrw" (diff) | |
download | linux-d110cf0ac1bf24ac098b8ea5b0188c717acf7a02.tar.xz linux-d110cf0ac1bf24ac098b8ea5b0188c717acf7a02.zip |
crypto: chelsio - Avoid some code duplication
The error handling path of 'chcr_authenc_setkey()' is the same as this
error handling code.
So just 'goto out' as done everywhere in the function to simplify the code.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'drivers/crypto')
-rw-r--r-- | drivers/crypto/chelsio/chcr_algo.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/crypto/chelsio/chcr_algo.c b/drivers/crypto/chelsio/chcr_algo.c index 9b68d62149ad..f37744f76b0f 100644 --- a/drivers/crypto/chelsio/chcr_algo.c +++ b/drivers/crypto/chelsio/chcr_algo.c @@ -3598,9 +3598,7 @@ static int chcr_authenc_setkey(struct crypto_aead *authenc, const u8 *key, base_hash = chcr_alloc_shash(max_authsize); if (IS_ERR(base_hash)) { pr_err("chcr : Base driver cannot be loaded\n"); - aeadctx->enckey_len = 0; - memzero_explicit(&keys, sizeof(keys)); - return -EINVAL; + goto out; } { SHASH_DESC_ON_STACK(shash, base_hash); |