Re: [PATCH] staging: speakup: more fixes for init-failure handling.

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

 



On Sun, Dec 19, 2010 at 10:50:24PM +0000, Christopher Brannon wrote:
> We still leaked many resources when Speakup failed to initialize.
> Examples of leaked resources include:
> /dev/synth, keyboard or VT notifiers, and heap-allocated st_spk_t
> structs.
> This is fixed.
> 
> * We now use PTR_ERR to detect kthread_create failure
> (thank you Dan Carpenter).
> 
> * The loop which frees members of the speakup_console array now iterates
> over the whole array, not stopping at the first NULL value.  Fixes
> a possible memory leak.  Safe because kfree(NULL) is a no-op.
> 
> * The order of some initializations was changed.  The safe ones, which
> will never fail, are performed first.
> 
> Signed-off-by: Christopher Brannon <chris@xxxxxxxxxxxxxxxx>

Acked-by: Dan Carpenter <error27@xxxxxxxxx>

Thanks again.

regards,
dan carpenter

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/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