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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20071013005959.GB8187@sortiz.org>
Date:	Sat, 13 Oct 2007 03:59:59 +0300
From:	Samuel Ortiz <samuel@...tiz.org>
To:	"hinko.kocevar@...rtapot.si" <hinko.kocevar@...rtapot.si>
Cc:	irda-users@...ts.sourceforge.net, linux-kernel@...r.kernel.org
Subject: Re: [irda-users] [PATCH] mcs7780 needs to free allocated rx buffer

Hi Hinko,

On Fri, Oct 12, 2007 at 02:36:46PM +0200, hinko.kocevar@...rtapot.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@...rtapot.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@...ts.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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/