Re: [irda-users] [PATCH] mcs7780 needs to free allocated rx buffer

[Samuel Ortiz <samuel@sortiz.org>]

Hi Hinko,

On Fri, Oct 12, 2007 at 02:36:46PM +0200, hinko.kocevar@cetrtapot.si wrote:
> Hi,
> 
> While testing the mcs7780 based IrDA USB dongle I've stumbled upon
> memory leak in mcs_net_close(). Patch below fixes it.
Thanks for the fix. Applied to my local git, will push forward.

Cheers,
Samuel.


> Signed-off-by: Hinko Kocevar <hinko.kocevar@cetrtapot.si>
> 
> Best regards,
> Hinko
> 
> ----
> 
> 

> --- linux-2.6.23/drivers/net/irda/mcs7780.c.orig	2007-10-12 14:02:55.000000000 +0200
> +++ linux-2.6.23/drivers/net/irda/mcs7780.c	2007-10-12 14:05:03.000000000 +0200
> @@ -678,6 +678,8 @@ static int mcs_net_close(struct net_devi
>  	/* Stop transmit processing */
>  	netif_stop_queue(netdev);
>  
> +	kfree_skb(mcs->rx_buff.skb);
> +
>  	/* kill and free the receive and transmit URBs */
>  	usb_kill_urb(mcs->rx_urb);
>  	usb_free_urb(mcs->rx_urb);

> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc.
> Still grepping through log files to find problems?  Stop.
> Now Search log events and configuration files using AJAX and a browser.
> Download your FREE copy of Splunk now >> http://get.splunk.com/
> _______________________________________________
> irda-users mailing list
> irda-users@lists.sourceforge.net
> http://lists.sourceforge.net/lists/listinfo/irda-users


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/