On Thu, Oct 27, 2016 at 6:40 PM, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > On Thu, Oct 27, 2016 at 01:46:13PM +0530, Souptick Joarder wrote: >> There are few functions where we need to free previously allocated memory >> when kmalloc fails.Else it may lead to memory leakage.In _init_cmd_priv() >> and _r8712_init_xmit_priv(),in few places we are not freeing previously >> allocated memory when kmalloc fails. >> >> Signed-off-by: Souptick joarder <jrdr.linux@xxxxxxxxx> >> --- >> Changes in v2: >> -Make the commit message more cleaner. > > I'll be pedantic here, but please always put a ' ' after a '.' or a ',' I didn't get which part exactly you are referring. Are you referring to the patch description and asking me to put ' ' after a '.' or ',' > It's the copy-editor in me coming out... > > You couldn't do something like that on a term-paper and expect to not > get it marked down, right? My apology here. > > another try? > > thanks, > > greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel