Commit c6e42c40 authored by Nathan Fontenot's avatar Nathan Fontenot Committed by Kamal Mostafa

powerpc/pseries: Fix possible leaked device node reference

commit 2222ce0f upstream.

Failure return from dlpar_configure_connector when dlpar adding cpus
results in leaking references to the cpus parent device node. Move the
call to of_node_put() prior to checking the result of
dlpar_configure_connector.

Fixes: 8d5ff320 ("powerpc/pseries: Make dlpar_configure_connector parent node aware")
Signed-off-by: default avatarNathan Fontenot <nfont@linux.vnet.ibm.com>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
Signed-off-by: default avatarKamal Mostafa <kamal@canonical.com>
parent 8822ef23
...@@ -419,11 +419,10 @@ static ssize_t dlpar_cpu_probe(const char *buf, size_t count) ...@@ -419,11 +419,10 @@ static ssize_t dlpar_cpu_probe(const char *buf, size_t count)
return -ENODEV; return -ENODEV;
dn = dlpar_configure_connector(cpu_to_be32(drc_index), parent); dn = dlpar_configure_connector(cpu_to_be32(drc_index), parent);
of_node_put(parent);
if (!dn) if (!dn)
return -EINVAL; return -EINVAL;
of_node_put(parent);
rc = dlpar_attach_node(dn); rc = dlpar_attach_node(dn);
if (rc) { if (rc) {
dlpar_release_drc(drc_index); dlpar_release_drc(drc_index);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment