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

[Dan Carpenter]

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 at the-brannons.com>

Acked-by: Dan Carpenter <error27 at gmail.com>

Thanks again.

regards,
dan carpenter