[PATCH] staging: dgnc: check return value of kzalloc

Gujulan Elango, Hari Prasath (H.) hgujulan at visteon.com
Fri May 29 10:06:48 UTC 2015


On Fri, May 29, 2015 at 11:54:09AM +0300, Dan Carpenter wrote:
> On Fri, May 29, 2015 at 07:44:11AM +0000, Gujulan Elango, Hari Prasath (H.) wrote:
> > check return value of kzalloc and return error if it fails.
> > 
> 
> There is a problem with your patch.  See if you can spot it.  Also can
> you add to the changelog what are the user visible effects of not
> checking for kzalloc() failure?  Last time with ->flipbuf there were no
> user visible effects.
> 
> Also return -ENOMEM seems like the wrong thing.  Shouldn't we do some
> error handling.  Also you are not working on linux-next?  Some of this
> stuff is done correctly in linux-next.
> 
> regards,
> dan carpenter
> 

Dan,I am working on the staging-testing git tree.I was not aware that
someone has submitted a similar patch which was reviewed by you &
sudip.If a patch to address this has already gone in,why am I not seeing
that change here.Am I missing something ?


More information about the devel mailing list