diff options
| author | Stanislav Kozina <skozina@redhat.com> | 2012-08-16 12:01:47 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-10-07 08:35:52 -0700 |
| commit | 7e67dd66193c2325cbd2739522db3334413c3aea (patch) | |
| tree | cf15c3f18e1f1c6250a854faccb7fc183cef06d6 | |
| parent | e83d31545f97dd5bf24ad77254afc38e676415c2 (diff) | |
| download | lwn-7e67dd66193c2325cbd2739522db3334413c3aea.tar.gz lwn-7e67dd66193c2325cbd2739522db3334413c3aea.zip | |
Remove BUG_ON from n_tty_read()
commit e9490e93c1978b6669f3e993caa3189be13ce459 upstream.
Change the BUG_ON to WARN_ON and return in case of tty->read_buf==NULL. We want to track a
couple of long standing reports of this but at the same time we can avoid killing the box.
Signed-off-by: Stanislav Kozina <skozina@redhat.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/tty/n_tty.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/tty/n_tty.c b/drivers/tty/n_tty.c index ee1c268f5f9d..26f0d0e4484b 100644 --- a/drivers/tty/n_tty.c +++ b/drivers/tty/n_tty.c @@ -1728,7 +1728,8 @@ static ssize_t n_tty_read(struct tty_struct *tty, struct file *file, do_it_again: - BUG_ON(!tty->read_buf); + if (WARN_ON(!tty->read_buf)) + return -EAGAIN; c = job_control(tty, file); if (c < 0) |