diff options
| author | Alan Cox <alan@redhat.com> | 2008-10-13 10:44:57 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-13 09:51:44 -0700 |
| commit | 51383f69ec56fe3f425d5255fb43e18c445e061c (patch) | |
| tree | f9d9e280892702c3a47ffa807c0937ea9c80e315 /drivers/char/mxser.c | |
| parent | ea1afd256258f04a290aaa7fd5c2d4deb2e79e26 (diff) | |
| download | op-kernel-dev-51383f69ec56fe3f425d5255fb43e18c445e061c.zip op-kernel-dev-51383f69ec56fe3f425d5255fb43e18c445e061c.tar.gz | |
tty: Remove lots of NULL checks
Many tty drivers contain 'can't happen' checks against NULL pointers passed
in by the tty layer. These have never been possible to occur. Even more
importantly if they ever do occur we want to know as it would be a serious
bug.
Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/char/mxser.c')
| -rw-r--r-- | drivers/char/mxser.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/char/mxser.c b/drivers/char/mxser.c index 308cb60..8beef50 100644 --- a/drivers/char/mxser.c +++ b/drivers/char/mxser.c @@ -616,9 +616,6 @@ static int mxser_set_baud(struct tty_struct *tty, long newspd) int quot = 0, baud; unsigned char cval; - if (!tty->termios) - return -1; - if (!info->ioaddr) return -1; @@ -688,8 +685,6 @@ static int mxser_change_speed(struct tty_struct *tty, int ret = 0; unsigned char status; - if (!tty->termios) - return ret; cflag = tty->termios->c_cflag; if (!info->ioaddr) return ret; |
