diff options
author | Bruno Prémont <[email protected]> | 2009-01-04 13:11:54 -0800 |
---|---|---|
committer | Linus Torvalds <[email protected]> | 2009-01-04 13:33:20 -0800 |
commit | e687d691cb3790d25e31c74f5941fd7c565e9df5 (patch) | |
tree | d606511c734e99fb6e4786d20ab42a8cf1b2232f /fs/jbd2/commit.c | |
parent | c644f0e4b56f9a2fc066cd0d75a18074d130e4a3 (diff) |
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 <[email protected]>
Cc: <[email protected]>
Cc: Krzysztof Helt <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'fs/jbd2/commit.c')
0 files changed, 0 insertions, 0 deletions