On Mon, Nov 04, 2013 at 02:11:50AM +0300, Dan Carpenter wrote:
> On Sun, Nov 03, 2013 at 10:28:02AM -0800, Josh Triplett wrote:
> > On Tue, Oct 29, 2013 at 11:01:43PM +0300, Dan Carpenter wrote:
> > > The icount.reserved[] array isn't initialized so it leaks stack
> > > information to userspace.
> > > 
> > > Reported-by: Nico Golde <n...@ngolde.de>
> > > Reported-by: Fabian Yamaguchi <f...@goesec.de>
> > > Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
> > 
> > Reviewed-by: Josh Triplett <j...@joshtriplett.org>
> > 
> > Also, you don't quite have the patch format right here; you should have
> > a --- line after the commit mesage, followed by a diffstat.  Did you use
> > git format-patch to generate this patch?
> 
> I normally don't include the diffstat.  Which tools care about that?

Human wetware. :)

It isn't required by any tools.  The --- is, though, to produce
something applicable by git.

- Josh Triplett
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to