Commit 3383427a authored by Jiri Slaby's avatar Jiri Slaby Committed by Greg Kroah-Hartman

TTY: n_tty, remove bogus checks

* BUG_ON(!tty) in n_tty_set_termios -- it cannot be called with tty ==
  NULL. It is called from two call sites. First, from n_tty_open where
  we have a valid tty. Second, as ld->ops->set_termios from
  tty_set_termios. But there we have a valid tty too.
* if (!tty) in n_tty_open -- why would the TTY layer call ldisc's
  open with an invalid TTY? No it indeed does not. All call sites have
  a tty and dereference that.
* BUG_ON(!tty->read_buf) in n_tty_read -- this used to be a valid
  check. The ldisc handling was broken some time ago when I added the
  check to ensure everything is OK. It still can catch the case, but
  no later than we move the buffer to ldisc data. Then there will be
  no read_buf in tty_struct, i.e. nothing to check for.
* if (!tty->read_buf) in n_tty_receive_buf -- this should never
  happen. All callers of ldisc->ops->receive_ops should hold a
  reference to an ldisc and close (which frees read_buf) cannot be
  called until the reference is dropped.
* if (WARN_ON(!tty->read_buf)) in n_tty_read -- the same as in the
  previous case.
Signed-off-by: default avatarJiri Slaby <jslaby@suse.cz>
Acked-by: default avatarAlan Cox <alan@linux.intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent b91939f5
...@@ -1375,9 +1375,6 @@ static void n_tty_receive_buf(struct tty_struct *tty, const unsigned char *cp, ...@@ -1375,9 +1375,6 @@ static void n_tty_receive_buf(struct tty_struct *tty, const unsigned char *cp,
char buf[64]; char buf[64];
unsigned long cpuflags; unsigned long cpuflags;
if (!tty->read_buf)
return;
if (tty->real_raw) { if (tty->real_raw) {
spin_lock_irqsave(&tty->read_lock, cpuflags); spin_lock_irqsave(&tty->read_lock, cpuflags);
i = min(N_TTY_BUF_SIZE - tty->read_cnt, i = min(N_TTY_BUF_SIZE - tty->read_cnt,
...@@ -1471,7 +1468,6 @@ int is_ignored(int sig) ...@@ -1471,7 +1468,6 @@ int is_ignored(int sig)
static void n_tty_set_termios(struct tty_struct *tty, struct ktermios *old) static void n_tty_set_termios(struct tty_struct *tty, struct ktermios *old)
{ {
int canon_change = 1; int canon_change = 1;
BUG_ON(!tty);
if (old) if (old)
canon_change = (old->c_lflag ^ tty->termios.c_lflag) & ICANON; canon_change = (old->c_lflag ^ tty->termios.c_lflag) & ICANON;
...@@ -1579,9 +1575,6 @@ static void n_tty_close(struct tty_struct *tty) ...@@ -1579,9 +1575,6 @@ static void n_tty_close(struct tty_struct *tty)
static int n_tty_open(struct tty_struct *tty) static int n_tty_open(struct tty_struct *tty)
{ {
if (!tty)
return -EINVAL;
/* These are ugly. Currently a malloc failure here can panic */ /* These are ugly. Currently a malloc failure here can panic */
tty->read_buf = kzalloc(N_TTY_BUF_SIZE, GFP_KERNEL); tty->read_buf = kzalloc(N_TTY_BUF_SIZE, GFP_KERNEL);
tty->echo_buf = kzalloc(N_TTY_BUF_SIZE, GFP_KERNEL); tty->echo_buf = kzalloc(N_TTY_BUF_SIZE, GFP_KERNEL);
...@@ -1736,10 +1729,6 @@ static ssize_t n_tty_read(struct tty_struct *tty, struct file *file, ...@@ -1736,10 +1729,6 @@ static ssize_t n_tty_read(struct tty_struct *tty, struct file *file,
int packet; int packet;
do_it_again: do_it_again:
if (WARN_ON(!tty->read_buf))
return -EAGAIN;
c = job_control(tty, file); c = job_control(tty, file);
if (c < 0) if (c < 0)
return c; return c;
...@@ -1825,7 +1814,6 @@ static ssize_t n_tty_read(struct tty_struct *tty, struct file *file, ...@@ -1825,7 +1814,6 @@ static ssize_t n_tty_read(struct tty_struct *tty, struct file *file,
/* FIXME: does n_tty_set_room need locking ? */ /* FIXME: does n_tty_set_room need locking ? */
n_tty_set_room(tty); n_tty_set_room(tty);
timeout = schedule_timeout(timeout); timeout = schedule_timeout(timeout);
BUG_ON(!tty->read_buf);
continue; continue;
} }
__set_current_state(TASK_RUNNING); __set_current_state(TASK_RUNNING);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment