diff options
author | Horia Geant? <horia.geanta@freescale.com> | 2015-05-11 13:03:24 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-07-10 12:49:33 -0400 |
commit | 0e566fe9dea0750713fa267b5a0b23f0b78b26ea (patch) | |
tree | 9b59b2b7f0bcda8a56828ed4da938c1898e49d68 | |
parent | 1e4205d4e0b0579b906de8ec7c5920968bfb32a9 (diff) |
crypto: talitos - avoid memleak in talitos_alg_alloc()
commit 5fa7dadc898567ce14d6d6d427e7bd8ce6eb5d39 upstream.
Fixes: 1d11911a8c57 ("crypto: talitos - fix warning: 'alg' may be used uninitialized in this function")
Signed-off-by: Horia Geanta <horia.geanta@freescale.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/crypto/talitos.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/crypto/talitos.c b/drivers/crypto/talitos.c index 857414afa29a..c52b4b32bcbd 100644 --- a/drivers/crypto/talitos.c +++ b/drivers/crypto/talitos.c | |||
@@ -2561,6 +2561,7 @@ static struct talitos_crypto_alg *talitos_alg_alloc(struct device *dev, | |||
2561 | break; | 2561 | break; |
2562 | default: | 2562 | default: |
2563 | dev_err(dev, "unknown algorithm type %d\n", t_alg->algt.type); | 2563 | dev_err(dev, "unknown algorithm type %d\n", t_alg->algt.type); |
2564 | kfree(t_alg); | ||
2564 | return ERR_PTR(-EINVAL); | 2565 | return ERR_PTR(-EINVAL); |
2565 | } | 2566 | } |
2566 | 2567 | ||