diff options
| author | Wolfram Sang <wsa@the-dreams.de> | 2013-04-19 21:12:17 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-04-22 10:22:21 -0700 |
| commit | 4829e7650f8a40645e4e32b26a37fb833a5e75f0 (patch) | |
| tree | 5f04c4596c310c031d2f4e5c9d8a673e0435483a /drivers/tty | |
| parent | 78506f223a7bb7d47b1772592932921689adc6d8 (diff) | |
| download | linux-4829e7650f8a40645e4e32b26a37fb833a5e75f0.tar.bz2 | |
serial: mxs: fix buffer overflow
SMATCH correctly found an off-by-one error:
drivers/tty/serial/mxs-auart.c:889 auart_console_write() error: buffer overflow 'auart_port' 5 <= 5
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty')
| -rw-r--r-- | drivers/tty/serial/mxs-auart.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/tty/serial/mxs-auart.c b/drivers/tty/serial/mxs-auart.c index d549fe1fa42a..9d357da817de 100644 --- a/drivers/tty/serial/mxs-auart.c +++ b/drivers/tty/serial/mxs-auart.c @@ -883,7 +883,7 @@ auart_console_write(struct console *co, const char *str, unsigned int count) unsigned int old_ctrl0, old_ctrl2; unsigned int to = 1000; - if (co->index > MXS_AUART_PORTS || co->index < 0) + if (co->index >= MXS_AUART_PORTS || co->index < 0) return; s = auart_port[co->index]; |