Re: [PATCH v2] staging : rtl8712: Free memory when kmalloc fails

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux