Messages in this thread | | | From | Varun Sethi <> | Subject | RE: [PATCH] iommu: fsl_pamu.c: Fix for possible null pointer dereference | Date | Wed, 21 May 2014 07:15:28 +0000 |
| |
> -----Original Message----- > From: iommu-bounces@lists.linux-foundation.org [mailto:iommu- > bounces@lists.linux-foundation.org] On Behalf Of > Bharat.Bhushan@freescale.com > Sent: Wednesday, May 21, 2014 9:28 AM > To: Rickard Strandqvist; Joerg Roedel; Grant Likely > Cc: devicetree@vger.kernel.org; iommu@lists.linux-foundation.org; Rob > Herring; linux-kernel@vger.kernel.org > Subject: RE: [PATCH] iommu: fsl_pamu.c: Fix for possible null pointer > dereference > > > > > -----Original Message----- > > From: iommu-bounces@lists.linux-foundation.org [mailto:iommu- > > bounces@lists.linux-foundation.org] On Behalf Of Rickard Strandqvist > > Sent: Saturday, May 17, 2014 10:47 PM > > To: Joerg Roedel; Grant Likely > > Cc: devicetree@vger.kernel.org; iommu@lists.linux-foundation.org; Rob > > Herring; linux-kernel@vger.kernel.org; Rickard Strandqvist > > Subject: [PATCH] iommu: fsl_pamu.c: Fix for possible null pointer > > dereference > > > > There is otherwise a risk of a possible null pointer dereference. > > You are right. [Sethi Varun-B16395] Thanks for pointing out the issue. > > > > > 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> Acknowledged-by: Varun Sethi <Varun.Sethi@freescale.com>
> > --- > > drivers/iommu/fsl_pamu.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/drivers/iommu/fsl_pamu.c b/drivers/iommu/fsl_pamu.c index > > cba0498..b99dd88 100644 > > --- a/drivers/iommu/fsl_pamu.c > > +++ b/drivers/iommu/fsl_pamu.c > > @@ -592,8 +592,7 @@ found_cpu_node: > > /* advance to next node in cache hierarchy */ > > node = of_find_node_by_phandle(*prop); > > if (!node) { > > - pr_debug("Invalid node for cache hierarchy %s\n", > > - node->full_name); > > + pr_debug("Invalid node for cache hierarchy\n"); > > return ~(u32)0; > > } > > } > > -- > > 1.7.10.4 > > > > _______________________________________________ > > iommu mailing list > > iommu@lists.linux-foundation.org > > https://lists.linuxfoundation.org/mailman/listinfo/iommu > _______________________________________________ > iommu mailing list > iommu@lists.linux-foundation.org > https://lists.linuxfoundation.org/mailman/listinfo/iommu
| |