Re: [PATCH] USB: EHCI: Fix isochronous URB leak

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

 



Hi Greg,

On Tue, 24 Feb 2009 22:04:48 -0800
Greg KH <greg@xxxxxxxxx> wrote:

> On Mon, Feb 23, 2009 at 07:29:20PM +0100, Karsten Wiese wrote:
> > 
> > ehci-hcd uses usb_get_urb() and usb_put_urb() in an unbalanced way causing
> > isochronous URB's kref.counts incrementing once per usb_submit_urb() call.
> > There is no need for ehci-hcd to deal with usb_get_urb() nor usb_put_urb()
> > anyhow, so patch removes their usages in ehci-hcd.
> > It also removes NULL assignments that have no effect.
> > 
> > Signed-off-by: Karsten Wiese <fzu@xxxxxxxxxxxxxxxxxxxxx>
> > ---
> > 
> > Made against 2.6.29-rc6. Suitable to go there and stable ASAP, no?
> 
> Are you sure this is still needed?

This patch is needed. It fixes a bug with USB v4l/dvb drivers, where memory
leaks are sometimes generated when calling usb_kill_urb().

By using this patch against 2.6.29-rc-git2, it fixed the issue.

I suspect that this patch should also be backported to -stable.


Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux