diff options
author | Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> | 2014-05-17 13:16:44 -0400 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2014-05-26 05:44:29 -0400 |
commit | d6a71bf79da03b835f8df5430eaee6fc46df1038 (patch) | |
tree | aec4b2246b0af970cf6dd356d4a1804c7ad30d42 | |
parent | 4b660a7f5c8099d88d1a43d8ae138965112592c7 (diff) |
iommu: fsl_pamu.c: Fix for possible null pointer dereference
There is otherwise a risk of a possible null pointer dereference.
Was largely found by using a static code analysis program called cppcheck.
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Reviewed-by: Bharat Bhushan <bharat.bhushan@freescale.com>
Acked-by: Varun Sethi <Varun.Sethi@freescale.com>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
-rw-r--r-- | drivers/iommu/fsl_pamu.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/iommu/fsl_pamu.c b/drivers/iommu/fsl_pamu.c index cba0498eb011..b99dd88e31b9 100644 --- a/drivers/iommu/fsl_pamu.c +++ b/drivers/iommu/fsl_pamu.c | |||
@@ -592,8 +592,7 @@ found_cpu_node: | |||
592 | /* advance to next node in cache hierarchy */ | 592 | /* advance to next node in cache hierarchy */ |
593 | node = of_find_node_by_phandle(*prop); | 593 | node = of_find_node_by_phandle(*prop); |
594 | if (!node) { | 594 | if (!node) { |
595 | pr_debug("Invalid node for cache hierarchy %s\n", | 595 | pr_debug("Invalid node for cache hierarchy\n"); |
596 | node->full_name); | ||
597 | return ~(u32)0; | 596 | return ~(u32)0; |
598 | } | 597 | } |
599 | } | 598 | } |