diff options
| author | Johan Hovold <johan@kernel.org> | 2019-10-29 11:23:53 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-11-06 12:43:31 +0100 |
| commit | 906f9f252be650eee94f2f1f7d9ac1efdb136044 (patch) | |
| tree | 637dfbd9855c315130f3e9e759112c59c53db607 | |
| parent | 0675c81abe0424be839cbc64a4dd233de8297170 (diff) | |
| download | linux-stable-906f9f252be650eee94f2f1f7d9ac1efdb136044.tar.gz linux-stable-906f9f252be650eee94f2f1f7d9ac1efdb136044.tar.bz2 linux-stable-906f9f252be650eee94f2f1f7d9ac1efdb136044.zip | |
USB: serial: whiteheat: fix potential slab corruption
commit 1251dab9e0a2c4d0d2d48370ba5baa095a5e8774 upstream.
Fix a user-controlled slab buffer overflow due to a missing sanity check
on the bulk-out transfer buffer used for control requests.
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Johan Hovold <johan@kernel.org>
Link: https://lore.kernel.org/r/20191029102354.2733-2-johan@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/usb/serial/whiteheat.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/usb/serial/whiteheat.c b/drivers/usb/serial/whiteheat.c index 55cebc1e6fec..d4951f9a8459 100644 --- a/drivers/usb/serial/whiteheat.c +++ b/drivers/usb/serial/whiteheat.c @@ -575,6 +575,10 @@ static int firm_send_command(struct usb_serial_port *port, __u8 command, command_port = port->serial->port[COMMAND_PORT]; command_info = usb_get_serial_port_data(command_port); + + if (command_port->bulk_out_size < datasize + 1) + return -EIO; + mutex_lock(&command_info->mutex); command_info->command_finished = false; |