crypto: ixp4xx - fix OF node reference leaks in init_ixp_crypto()

[ Upstream commit 472a989029aac2b78ef2f0b18b27c568bf76d104 ]

init_ixp_crypto() calls of_parse_phandle_with_fixed_args() multiple
times, but does not release all the obtained refcounts. Fix it by adding
of_node_put() calls.

This bug was found by an experimental static analysis tool that I am
developing.

Fixes: 76f24b4f46b8 ("crypto: ixp4xx - Add device tree support")
Signed-off-by: Joe Hattori <joe@pf.is.s.u-tokyo.ac.jp>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/drivers/crypto/ixp4xx_crypto.c b/drivers/crypto/ixp4xx_crypto.c
index 98730aab287c3d4443576d7089ad9b2f22d6c070..2bf315554f02a2bcafdd458eabceea49991afe36 100644 (file)
--- a/drivers/crypto/ixp4xx_crypto.c
+++ b/drivers/crypto/ixp4xx_crypto.c
@@ -470,6 +470,7 @@ static int init_ixp_crypto(struct device *dev)
                        return -ENODEV;
                }
                npe_id = npe_spec.args[0];
+               of_node_put(npe_spec.np);
 
                ret = of_parse_phandle_with_fixed_args(np, "queue-rx", 1, 0,
                                                       &queue_spec);
@@ -478,6 +479,7 @@ static int init_ixp_crypto(struct device *dev)
                        return -ENODEV;
                }
                recv_qid = queue_spec.args[0];
+               of_node_put(queue_spec.np);
 
                ret = of_parse_phandle_with_fixed_args(np, "queue-txready", 1, 0,
                                                       &queue_spec);
@@ -486,6 +488,7 @@ static int init_ixp_crypto(struct device *dev)
                        return -ENODEV;
                }
                send_qid = queue_spec.args[0];
+               of_node_put(queue_spec.np);
        } else {
                /*
                 * Hardcoded engine when using platform data, this goes away