bus: arm-integrator-lm: Add of_node_put() before return statement
authorSumera Priyadarsini <sylphrenadin@gmail.com>
Tue, 12 Jan 2021 09:25:49 +0000 (10:25 +0100)
committerArnd Bergmann <arnd@arndb.de>
Fri, 15 Jan 2021 16:53:05 +0000 (17:53 +0100)
commit1740e6736922cc1a5d061cc4240d08eacfbbaa71
treedede80bcc279375dd50e9ffe1488575b88519010
parent8880bf6de400346600cb334f236cce924558690f
bus: arm-integrator-lm: Add of_node_put() before return statement

Every iteration of for_each_available_child_of_node() decrements
the reference count of the previous node, however when control is
transferred from the middle of the loop, as in the case of a return
or break or goto, there is no decrement thus ultimately resulting in
a memory leak.

Fix a potential memory leak in arm-integrator-lm.c by inserting
of_node_put() before a return statement.

Issue found with Coccinelle.

Signed-off-by: Sumera Priyadarsini <sylphrenadin@gmail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Link: https://lore.kernel.org/r/20200829174154.GA9319@Kaladin
Link: https://lore.kernel.org/r/20210112092549.251548-1-linus.walleij@linaro.org'
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
drivers/bus/arm-integrator-lm.c