diff options
author | Wolfgang Kroworsch <wolfgang@kroworsch.de> | 2008-11-06 21:53:16 +0100 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-11-07 00:41:17 +0100 |
commit | a564738c1c9c7b9ed696bf4116267789201ac8ac (patch) | |
tree | e5bd2ee7c28e2831bea5c1727452978c76c1c108 /drivers/rtc/interface.c | |
parent | cpqarry: fix return value of cpqarray_init() (diff) | |
download | linux-a564738c1c9c7b9ed696bf4116267789201ac8ac.tar.xz linux-a564738c1c9c7b9ed696bf4116267789201ac8ac.zip |
vt: incomplete initialization of vc_tab_stop
Problem 1 (see patch below):
vc_tab_stop is declared as an array of 8 unsigned ints in struct
vc_data in include/linux/console_struct.h .
In drivers/char/vt.c only 5 of these 8 unsigned ints get initialized
leading to unintended tabulator placement on displays with more than
160 columns text.
Problem 2 (open):
Upcoming displays will have more than 256 columns of text leading to
invalid memory access in drivers/char/vt.c during tabulator
calculations:
if (vc->vc_tab_stop[vc->vc_x >> 5] & (1 << (vc->vc_x & 31)))
break;
Signed-off-by: Wolfgang Kroworsch <wolfgang@kroworsch.de>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Christoph Hellwig <hch@lst.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/rtc/interface.c')
0 files changed, 0 insertions, 0 deletions