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 /drivers/usb/serial/pl2303.c | |
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>
Diffstat (limited to 'drivers/usb/serial/pl2303.c')
-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 || |