diff options
author | Johan Hovold <jhovold@gmail.com> | 2013-06-26 10:47:31 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-07-23 19:24:33 -0400 |
commit | dc43ff924d0d29e53e3f4d26db2c4265b1aa5a2a (patch) | |
tree | e339833acb19fcf16a2b62ec45e63a1bb6dd96b9 /drivers/usb/serial/oti6858.c | |
parent | 3ba19fe31b726371b45f91a8de9341f5625e75e4 (diff) |
USB: oti6858: do not call set_termios with uninitialised data
Make sure set_termios is not called with uninitialised data at open. The
old termios struct is currently not used, but pass NULL instead to avoid
future problems (e.g. stack data leak).
Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/serial/oti6858.c')
-rw-r--r-- | drivers/usb/serial/oti6858.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/usb/serial/oti6858.c b/drivers/usb/serial/oti6858.c index baa2ea59b921..a2080ac7b7e5 100644 --- a/drivers/usb/serial/oti6858.c +++ b/drivers/usb/serial/oti6858.c | |||
@@ -508,7 +508,6 @@ static void oti6858_set_termios(struct tty_struct *tty, | |||
508 | static int oti6858_open(struct tty_struct *tty, struct usb_serial_port *port) | 508 | static int oti6858_open(struct tty_struct *tty, struct usb_serial_port *port) |
509 | { | 509 | { |
510 | struct oti6858_private *priv = usb_get_serial_port_data(port); | 510 | struct oti6858_private *priv = usb_get_serial_port_data(port); |
511 | struct ktermios tmp_termios; | ||
512 | struct usb_serial *serial = port->serial; | 511 | struct usb_serial *serial = port->serial; |
513 | struct oti6858_control_pkt *buf; | 512 | struct oti6858_control_pkt *buf; |
514 | unsigned long flags; | 513 | unsigned long flags; |
@@ -559,7 +558,7 @@ static int oti6858_open(struct tty_struct *tty, struct usb_serial_port *port) | |||
559 | 558 | ||
560 | /* setup termios */ | 559 | /* setup termios */ |
561 | if (tty) | 560 | if (tty) |
562 | oti6858_set_termios(tty, port, &tmp_termios); | 561 | oti6858_set_termios(tty, port, NULL); |
563 | 562 | ||
564 | return 0; | 563 | return 0; |
565 | } | 564 | } |