diff options
| author | Johan Hovold <johan@kernel.org> | 2016-11-08 15:26:50 +0300 |
|---|---|---|
| committer | Johan Hovold <johan@kernel.org> | 2016-11-11 19:54:04 +0300 |
| commit | 2fbd69c4e33360383907cf0abb245440e62a6f37 (patch) | |
| tree | 71778f45768c74d97deffbcaac7be37d7f083faf /drivers/usb/serial/opticon.c | |
| parent | a98b69002a1656f1d50e24d6c189a66a5f55f783 (diff) | |
| download | linux-2fbd69c4e33360383907cf0abb245440e62a6f37.tar.xz | |
USB: serial: fix invalid user-pointer checks
Drop invalid user-pointer checks from ioctl handlers.
A NULL-pointer can be valid in user space and copy_to_user() takes care
of sanity checking.
Signed-off-by: Johan Hovold <johan@kernel.org>
Diffstat (limited to 'drivers/usb/serial/opticon.c')
| -rw-r--r-- | drivers/usb/serial/opticon.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/usb/serial/opticon.c b/drivers/usb/serial/opticon.c index 4b7bfb394a32..5ded6f524d59 100644 --- a/drivers/usb/serial/opticon.c +++ b/drivers/usb/serial/opticon.c @@ -336,9 +336,6 @@ static int get_serial_info(struct usb_serial_port *port, { struct serial_struct tmp; - if (!serial) - return -EFAULT; - memset(&tmp, 0x00, sizeof(tmp)); /* fake emulate a 16550 uart to make userspace code happy */ |