From: kernel test robot <l...@intel.com>

for_each_node_by_type should have of_node_put() before return.

Generated by: scripts/coccinelle/iterators/for_each_child.cocci

CC: Sumera Priyadarsini <sylphrena...@gmail.com>
Reported-by: kernel test robot <l...@intel.com>
Signed-off-by: kernel test robot <l...@intel.com>
---

The code seems to have been this way since the beginning of time.  Perhaps
the of_node_put is a no op for this code?

 core_64.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

--- a/arch/powerpc/kexec/core_64.c
+++ b/arch/powerpc/kexec/core_64.c
@@ -64,8 +64,10 @@ int default_machine_kexec_prepare(struct
                        begin = image->segment[i].mem;
                        end = begin + image->segment[i].memsz;

-                       if ((begin < high) && (end > low))
+                       if ((begin < high) && (end > low)) {
+                               of_node_put(node);
                                return -ETXTBSY;
+                       }
                }
        }

Reply via email to