]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
powerpc/pseries: Fix node leak in update_lmb_associativity_index()
authorMichael Ellerman <mpe@ellerman.id.au>
Tue, 27 Nov 2018 08:16:44 +0000 (19:16 +1100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 5 Dec 2019 08:20:47 +0000 (09:20 +0100)
[ Upstream commit 47918bc68b7427e961035949cc1501a864578a69 ]

In update_lmb_associativity_index() we lookup dr_node using
of_find_node_by_path() which takes a reference for us. In the
non-error case we forget to drop the reference. Note that
find_aa_index() does modify properties of the node, but doesn't need
an extra reference held once it's returned.

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Sasha Levin <sashal@kernel.org>
arch/powerpc/platforms/pseries/hotplug-memory.c

index d93ff494e7781fcc6cbd38a64e31b681e928e013..7f86bc3eaadecc2e2496b400e790dd9fbcbe7b27 100644 (file)
@@ -202,6 +202,7 @@ static int update_lmb_associativity_index(struct drmem_lmb *lmb)
 
        aa_index = find_aa_index(dr_node, ala_prop, lmb_assoc);
 
+       of_node_put(dr_node);
        dlpar_free_cc_nodes(lmb_node);
 
        if (aa_index < 0) {