diff options
| author |   Philippe Elie <phil.el@wanadoo.fr> | 2007-11-14 16:58:48 -0800 |
|---|---|---|
| committer |   Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-11-14 18:45:37 -0800 |
| commit | df9d177aa28d50e64bae6fbd6b263833079e3571 (patch) | |
| tree | b265158232a39d9d45cac0a97d5fdc17e7ca237d /arch/um/os-Linux/file.c | |
| parent | uvesafb: fix warnings about unused variables on non-x86 (diff) | |
| download | linux-dev-df9d177aa28d50e64bae6fbd6b263833079e3571.tar.xz linux-dev-df9d177aa28d50e64bae6fbd6b263833079e3571.zip | |
oProfile: oops when profile_pc() returns ~0LU
Instruction pointer returned by profile_pc() can be a random value. This
break the assumption than we can safely set struct op_sample.eip field to a
magic value to signal to the per-cpu buffer reader side special event like
task switch ending up in a segfault in get_task_mm() when profile_pc()
return ~0UL. Fixed by sanitizing the sampled eip and reject/log invalid
eip.
Problem reported by Sami Farin, patch tested by him.
Signed-off-by: Philippe Elie <phil.el@wanadoo.fr>
Tested-by: Sami Farin <safari-kernel@safari.iki.fi>
Cc: <stable@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/um/os-Linux/file.c')
0 files changed, 0 insertions, 0 deletions