From: Oliver Neukum <[EMAIL PROTECTED]>

we report errors to the caller. THis patch adds error handling to the driver.

Signed-off-by: Oliver Neukum <[EMAIL PROTECTED]>
Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]>
---
 drivers/usb/serial/mct_u232.c |   12 ++++++++----
 1 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/drivers/usb/serial/mct_u232.c b/drivers/usb/serial/mct_u232.c
index 4cd839b..3db1adc 100644
--- a/drivers/usb/serial/mct_u232.c
+++ b/drivers/usb/serial/mct_u232.c
@@ -438,17 +438,21 @@ static int  mct_u232_open (struct usb_serial_port *port, 
struct file *filp)
        if (retval) {
                err("usb_submit_urb(read bulk) failed pipe 0x%x err %d",
                    port->read_urb->pipe, retval);
-               goto exit;
+               goto error;
        }
 
        port->interrupt_in_urb->dev = port->serial->dev;
        retval = usb_submit_urb(port->interrupt_in_urb, GFP_KERNEL);
-       if (retval)
+       if (retval) {
+               usb_kill_urb(port->read_urb);
                err(" usb_submit_urb(read int) failed pipe 0x%x err %d",
                    port->interrupt_in_urb->pipe, retval);
-
-exit:
+               goto error;
+       }
        return 0;
+
+error:
+       return retval;
 } /* mct_u232_open */
 
 
-- 
1.5.1.2


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to