diff options
author | Ray Lee <ray-lk@madrabbit.org> | 2008-03-04 18:25:12 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-04-25 00:16:40 -0400 |
commit | b56394bf325820e9f338eaef2941f18b17b98098 (patch) | |
tree | 46a38ee7583d5341ec69d257029218e805fb3ccb | |
parent | dfa5ec79d28300b0d1fdeafbeebf0a6b721edc38 (diff) |
USB: io_ti.c: remove unneeded null tty check
The Coverity checker (and Adrian Bunk) spotted an inconsistent NULL check of
port->tty (it's blindly dereferenced later without the check).
Alan Cox confirmed the check can go.
Signed-off-by: Ray Lee <ray-lk@madrabbit.org>
Cc: Adrian Bunk <bunk@kernel.org>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/usb/serial/io_ti.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/usb/serial/io_ti.c b/drivers/usb/serial/io_ti.c index 6afee30a6a5c..316467ecd778 100644 --- a/drivers/usb/serial/io_ti.c +++ b/drivers/usb/serial/io_ti.c | |||
@@ -1941,8 +1941,7 @@ static int edge_open (struct usb_serial_port *port, struct file * filp) | |||
1941 | if (edge_port == NULL) | 1941 | if (edge_port == NULL) |
1942 | return -ENODEV; | 1942 | return -ENODEV; |
1943 | 1943 | ||
1944 | if (port->tty) | 1944 | port->tty->low_latency = low_latency; |
1945 | port->tty->low_latency = low_latency; | ||
1946 | 1945 | ||
1947 | port_number = port->number - port->serial->minor; | 1946 | port_number = port->number - port->serial->minor; |
1948 | switch (port_number) { | 1947 | switch (port_number) { |