Patchwork [3.5.yuz,extended,stable] Patch "USB: quatech2: fix memory leak in error path" has been added to staging queue

mail settings
Submitter Herton Ronaldo Krzesinski
Date Nov. 22, 2012, 4:49 a.m.
Message ID <>
Download mbox | patch
Permalink /patch/200996/
State New
Headers show


Herton Ronaldo Krzesinski - Nov. 22, 2012, 4:49 a.m.
This is a note to let you know that I have just added a patch titled

    USB: quatech2: fix memory leak in error path

to the linux-3.5.y-queue branch of the 3.5.yuz extended stable tree 
which can be found at:;a=shortlog;h=refs/heads/linux-3.5.y-queue

If you, or anyone else, feels it should not be added to this tree, please 
reply to this email.

For more information about the 3.5.yuz tree, see



From 21c54f67678307093acd2254502f8f2c31c53e6f Mon Sep 17 00:00:00 2001
From: Johan Hovold <>
Date: Thu, 25 Oct 2012 10:29:07 +0200
Subject: [PATCH] USB: quatech2: fix memory leak in error path

commit b8a0055050b6294826171641b182c09f78f4cc63 upstream.

Fix memory leak in attach error path where the read urb was never freed.

Cc: Bill Pemberton <>
Signed-off-by: Johan Hovold <>
Signed-off-by: Greg Kroah-Hartman <>
Signed-off-by: Herton Ronaldo Krzesinski <>
 drivers/usb/serial/quatech2.c |    1 +
 1 file changed, 1 insertion(+)



diff --git a/drivers/usb/serial/quatech2.c b/drivers/usb/serial/quatech2.c
index 8dd88eb..d170271 100644
--- a/drivers/usb/serial/quatech2.c
+++ b/drivers/usb/serial/quatech2.c
@@ -829,6 +829,7 @@  static int qt2_setup_urbs(struct usb_serial *serial)
 	if (status != 0) {
 			"%s - submit read urb failed %i\n", __func__, status);
+		usb_free_urb(serial_priv->read_urb);
 		return status;