aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Daney <david.daney@cavium.com>2015-08-19 16:17:47 -0400
committerRob Herring <robh@kernel.org>2015-08-25 12:29:56 -0400
commit3a496b00b6f90c41bd21a410871dfc97d4f3c7ab (patch)
treeb633747f5c4608e1f7122be4f96524a86bd2a25a
parent068627a5978a885b33f0f49da81860ce78c3ac6f (diff)
of/address: Don't loop forever in of_find_matching_node_by_address().
If the internal call to of_address_to_resource() fails, we end up looping forever in of_find_matching_node_by_address(). This can be caused by a defective device tree, or calling with an incorrect matches argument. Fix by calling of_find_matching_node() unconditionally at the end of the loop. Signed-off-by: David Daney <david.daney@cavium.com> Cc: stable@vger.kernel.org Signed-off-by: Rob Herring <robh@kernel.org>
-rw-r--r--drivers/of/address.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/of/address.c b/drivers/of/address.c
index 8bfda6ade2c0..384574c3987c 100644
--- a/drivers/of/address.c
+++ b/drivers/of/address.c
@@ -845,10 +845,10 @@ struct device_node *of_find_matching_node_by_address(struct device_node *from,
845 struct resource res; 845 struct resource res;
846 846
847 while (dn) { 847 while (dn) {
848 if (of_address_to_resource(dn, 0, &res)) 848 if (!of_address_to_resource(dn, 0, &res) &&
849 continue; 849 res.start == base_address)
850 if (res.start == base_address)
851 return dn; 850 return dn;
851
852 dn = of_find_matching_node(dn, matches); 852 dn = of_find_matching_node(dn, matches);
853 } 853 }
854 854