Hi Russell,

From: ext Russell King - ARM Linux <li...@arm.linux.org.uk>
Subject: Re: [PATCH 02/10] OMAP: iommu: add initial debugfs support
Date: Thu, 13 Aug 2009 11:23:59 +0200

> On Wed, Aug 12, 2009 at 03:13:24PM +0300, Tony Lindgren wrote:
> > +static DEFINE_MUTEX(iommu_debug_lock);
> > +static char local_buffer[SZ_4K];
> 
> I don't like this - what if the data you're sprintf'ing into this
> buffer overflows it?

Right.

I have attached the updated version which limits max write counts to
avoid the above buffer overflow.



Attachment: 0001-OMAP-iommu-add-initial-debugfs-support.patch
Description: Binary data

Reply via email to