diff options
| author | Horst Schirmeier <horst@schirmeier.com> | 2005-07-28 09:32:20 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-01-04 16:51:41 -0500 |
| commit | 95f209f93663103db2a8fb989e226ac68a98b036 (patch) | |
| tree | 459d85d3012707d2d7d588cfff88a67c3485db48 | |
| parent | 17fa6e552f2fc6bb06af767b0abf9cb642e13404 (diff) | |
[PATCH] USB: pl2303_update_line_status data length fix
Minimum data length must be UART_STATE + 1, as data[UART_STATE] is being
accessed for the new line_state. Although PL-2303 hardware is not
expected to send data with exactly UART_STATE length, this keeps it on
the safe side.
Signed-off-by: Horst Schirmeier <horst@schirmeier.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | drivers/usb/serial/pl2303.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/serial/pl2303.c b/drivers/usb/serial/pl2303.c index c96ba9fc19e0..f03721056190 100644 --- a/drivers/usb/serial/pl2303.c +++ b/drivers/usb/serial/pl2303.c | |||
| @@ -810,7 +810,7 @@ static void pl2303_update_line_status(struct usb_serial_port *port, | |||
| 810 | struct pl2303_private *priv = usb_get_serial_port_data(port); | 810 | struct pl2303_private *priv = usb_get_serial_port_data(port); |
| 811 | unsigned long flags; | 811 | unsigned long flags; |
| 812 | u8 status_idx = UART_STATE; | 812 | u8 status_idx = UART_STATE; |
| 813 | u8 length = UART_STATE; | 813 | u8 length = UART_STATE + 1; |
| 814 | 814 | ||
| 815 | if ((le16_to_cpu(port->serial->dev->descriptor.idVendor) == SIEMENS_VENDOR_ID) && | 815 | if ((le16_to_cpu(port->serial->dev->descriptor.idVendor) == SIEMENS_VENDOR_ID) && |
| 816 | (le16_to_cpu(port->serial->dev->descriptor.idProduct) == SIEMENS_PRODUCT_ID_X65 || | 816 | (le16_to_cpu(port->serial->dev->descriptor.idProduct) == SIEMENS_PRODUCT_ID_X65 || |