Commit 18a11b5e authored Jun 03, 2015 by Stephen Boyd Committed by Catalin Marinas Jun 30, 2015

arm64: perf: Don't use of_node after putting it



It's possible, albeit unlikely, that using the of_node here will
reference freed memory. Call of_node_put() after printing the
name to be safe.

Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Will Deacon <will.deacon@arm.com>

parent 1e43ba9c

arch/arm64/kernel/perf_event.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -1340,12 +1340,13 @@ static int armpmu_device_probe(struct platform_device *pdev)
		if (arch_find_n_match_cpu_physical_id(dn, cpu, NULL))
		break;

		of_node_put(dn);
		if (cpu >= nr_cpu_ids) {
		pr_warn("Failed to find logical CPU for %s\n",
		dn->name);
		of_node_put(dn);
		break;
		}
		of_node_put(dn);

		irqs[i] = cpu;
		}