diff options
author | Nishka Dasgupta <nishkadg.linux@gmail.com> | 2019-07-24 03:54:33 -0400 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2019-08-02 00:44:32 -0400 |
commit | 3bd4952853d69659e58d1d4a4964383cb536b5be (patch) | |
tree | e4aee889a1d4cf61c7e27bf1d8b8421283d56e71 | |
parent | 939f146b89c22c387d51f69b27ef031664f64cbe (diff) |
crypto: nx - Add of_node_put() before return in 842
Each iteration of for_each_compatible_node puts the previous node, but
in the case of a return from the middle of the loop, there is no put,
thus causing a memory leak. Add an of_node_put before the return.
Issue found with Coccinelle.
Acked-by: Stewart Smith <stewart@linux.ibm.com>
Signed-off-by: Nishka Dasgupta <nishkadg.linux@gmail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
-rw-r--r-- | drivers/crypto/nx/nx-842-powernv.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/crypto/nx/nx-842-powernv.c b/drivers/crypto/nx/nx-842-powernv.c index e78ff5c65ed6..c037a2403b82 100644 --- a/drivers/crypto/nx/nx-842-powernv.c +++ b/drivers/crypto/nx/nx-842-powernv.c | |||
@@ -1020,6 +1020,7 @@ static __init int nx842_powernv_init(void) | |||
1020 | ret = nx842_powernv_probe_vas(dn); | 1020 | ret = nx842_powernv_probe_vas(dn); |
1021 | if (ret) { | 1021 | if (ret) { |
1022 | nx842_delete_coprocs(); | 1022 | nx842_delete_coprocs(); |
1023 | of_node_put(dn); | ||
1023 | return ret; | 1024 | return ret; |
1024 | } | 1025 | } |
1025 | } | 1026 | } |