diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-10-29 04:43:25 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-11-05 19:12:45 -0500 |
commit | fda2b418a1945bebd5c8f670768fe75c515816ec (patch) | |
tree | 66f1ddc76c66ea3bd5e7bf6cb093d019ff3f4fcf | |
parent | a4dc923640418120fe7bbdfac149397bc761a249 (diff) |
goldfish: off by one in goldfish_tty_console_setup()
The goldfish_ttys[] array has "goldfish_tty_line_count" number of
elements. It's allocated in goldfish_tty_create_driver(). This test
should be >= instead of >.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/tty/goldfish.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/tty/goldfish.c b/drivers/tty/goldfish.c index c24b9633ae19..967b2c2b7cf1 100644 --- a/drivers/tty/goldfish.c +++ b/drivers/tty/goldfish.c | |||
@@ -155,7 +155,7 @@ static struct tty_driver *goldfish_tty_console_device(struct console *c, | |||
155 | 155 | ||
156 | static int goldfish_tty_console_setup(struct console *co, char *options) | 156 | static int goldfish_tty_console_setup(struct console *co, char *options) |
157 | { | 157 | { |
158 | if ((unsigned)co->index > goldfish_tty_line_count) | 158 | if ((unsigned)co->index >= goldfish_tty_line_count) |
159 | return -ENODEV; | 159 | return -ENODEV; |
160 | if (!goldfish_ttys[co->index].base) | 160 | if (!goldfish_ttys[co->index].base) |
161 | return -ENODEV; | 161 | return -ENODEV; |