summaryrefslogtreecommitdiffstats
path: root/drivers/usb/serial/quatech2.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/quatech2.c
parenta98b69002a1656f1d50e24d6c189a66a5f55f783 (diff)
downloadop-kernel-dev-2fbd69c4e33360383907cf0abb245440e62a6f37.zip
op-kernel-dev-2fbd69c4e33360383907cf0abb245440e62a6f37.tar.gz
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/quatech2.c')
-rw-r--r--drivers/usb/serial/quatech2.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/usb/serial/quatech2.c b/drivers/usb/serial/quatech2.c
index 85acb50..659cb86 100644
--- a/drivers/usb/serial/quatech2.c
+++ b/drivers/usb/serial/quatech2.c
@@ -463,9 +463,6 @@ static int get_serial_info(struct usb_serial_port *port,
{
struct serial_struct tmp;
- if (!retinfo)
- return -EFAULT;
-
memset(&tmp, 0, sizeof(tmp));
tmp.line = port->minor;
tmp.port = 0;
OpenPOWER on IntegriCloud