On Wed, Sep 17, 2014 at 10:47:25PM +0200, Abel Moyo wrote:
> Added error checking for alloc_tx_struct in do_tx()
> 
> Signed-off-by: Abel Moyo <abelmoyo...@gmail.com>
> ---
>  drivers/staging/gdm724x/gdm_usb.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/staging/gdm724x/gdm_usb.c 
> b/drivers/staging/gdm724x/gdm_usb.c
> index 483185b..0c20554 100644
> --- a/drivers/staging/gdm724x/gdm_usb.c
> +++ b/drivers/staging/gdm724x/gdm_usb.c
> @@ -682,6 +682,10 @@ static void do_tx(struct work_struct *work)
>               }
>  
>               t = alloc_tx_struct(TX_BUF_SIZE);
> +             if (t == NULL) {
> +                     pr_err("alloc_tx_struct - out of memory");
> +                     return;

There is a missing spin_unlock.

Don't print an error message.  There is already a much more useful error
message at the lower levels.  This one is just a waste of RAM.

regards,
dan carpenter
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to