Commit b631433b authored by Johan Hovold's avatar Johan Hovold

USB: serial: ark3116: fix open error handling

Fix open error handling which failed to detect errors when reading the
MSR and LSR registers, something which could lead to the shadow
registers being initialised from errnos.

Note that calling the generic close implementation is sufficient in the
error paths as the interrupt urb has not yet been submitted and the
register updates have not been made.

Fixes: f4c1e8d5 ("USB: ark3116: Make existing functions 16450-aware
and add close and release functions.")
Reviewed-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarJohan Hovold <johan@kernel.org>
parent 9fef37d7
...@@ -373,23 +373,29 @@ static int ark3116_open(struct tty_struct *tty, struct usb_serial_port *port) ...@@ -373,23 +373,29 @@ static int ark3116_open(struct tty_struct *tty, struct usb_serial_port *port)
dev_dbg(&port->dev, dev_dbg(&port->dev,
"%s - usb_serial_generic_open failed: %d\n", "%s - usb_serial_generic_open failed: %d\n",
__func__, result); __func__, result);
goto err_out; goto err_free;
} }
/* remove any data still left: also clears error state */ /* remove any data still left: also clears error state */
ark3116_read_reg(serial, UART_RX, buf); ark3116_read_reg(serial, UART_RX, buf);
/* read modem status */ /* read modem status */
priv->msr = ark3116_read_reg(serial, UART_MSR, buf); result = ark3116_read_reg(serial, UART_MSR, buf);
if (result < 0)
goto err_close;
priv->msr = *buf;
/* read line status */ /* read line status */
priv->lsr = ark3116_read_reg(serial, UART_LSR, buf); result = ark3116_read_reg(serial, UART_LSR, buf);
if (result < 0)
goto err_close;
priv->lsr = *buf;
result = usb_submit_urb(port->interrupt_in_urb, GFP_KERNEL); result = usb_submit_urb(port->interrupt_in_urb, GFP_KERNEL);
if (result) { if (result) {
dev_err(&port->dev, "submit irq_in urb failed %d\n", dev_err(&port->dev, "submit irq_in urb failed %d\n",
result); result);
ark3116_close(port); goto err_close;
goto err_out;
} }
/* activate interrupts */ /* activate interrupts */
...@@ -402,8 +408,15 @@ static int ark3116_open(struct tty_struct *tty, struct usb_serial_port *port) ...@@ -402,8 +408,15 @@ static int ark3116_open(struct tty_struct *tty, struct usb_serial_port *port)
if (tty) if (tty)
ark3116_set_termios(tty, port, NULL); ark3116_set_termios(tty, port, NULL);
err_out:
kfree(buf); kfree(buf);
return 0;
err_close:
usb_serial_generic_close(port);
err_free:
kfree(buf);
return result; return result;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment