diff options
author | Bruno Prémont <bonbons@linux-vserver.org> | 2009-01-04 22:11:54 +0100 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-01-04 22:33:20 +0100 |
commit | e687d691cb3790d25e31c74f5941fd7c565e9df5 (patch) | |
tree | d606511c734e99fb6e4786d20ab42a8cf1b2232f /mm/filemap.c | |
parent | fs: introduce bgl_lock_ptr() (diff) | |
download | linux-e687d691cb3790d25e31c74f5941fd7c565e9df5.tar.xz linux-e687d691cb3790d25e31c74f5941fd7c565e9df5.zip |
viafb: fix crashes due to 4k stack overflow
The function viafb_cursor() uses 2 stack-variables of CURSOR_SIZE bits;
CURSOR_SIZE is defined as (8 * 1024). Using up twice 1k on stack is too
much for 4k-stack (though it works with 8k-stacks). Make those two
variables kzalloc'ed to preserve stack space.
Also merge the whole lot of local struct's in viafb_ioctl into a union so
the stack usage gets minimized here as well. (struct's are only accessed
in their indicidual IOCTL case) This second part is only compile-tested as
I know of no userspace app using the IOCTLs.
Signed-off-by: Bruno Prémont <bonbons@linux-vserver.org>
Cc: <JosephChan@via.com.tw>
Cc: Krzysztof Helt <krzysztof.h1@poczta.fm>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/filemap.c')
0 files changed, 0 insertions, 0 deletions