aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/usb/serial/opticon.c
diff options
context:
space:
mode:
authorJohan Hovold <johan@kernel.org>2016-11-08 13:26:50 +0100
committerJohan Hovold <johan@kernel.org>2016-11-11 17:54:04 +0100
commit2fbd69c4e33360383907cf0abb245440e62a6f37 (patch)
tree71778f45768c74d97deffbcaac7be37d7f083faf /drivers/usb/serial/opticon.c
parentUSB: serial: ch341: add debug output for chip version (diff)
downloadlinux-dev-2fbd69c4e33360383907cf0abb245440e62a6f37.tar.xz
linux-dev-2fbd69c4e33360383907cf0abb245440e62a6f37.zip
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.c3
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 */