diff options
| author | Oleg Nesterov <oleg@tv-sign.ru> | 2006-03-28 16:11:30 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-28 18:36:44 -0800 |
| commit | 547679087bc60277d11b11631d826895762c4505 (patch) | |
| tree | 91759a5584b9d42302d4b54ecdde644bc47e781d /mm/msync.c | |
| parent | a1d5e21e3e388fb2c16463d007e788b1e41b74f1 (diff) | |
[PATCH] send_sigqueue: simplify and fix the race
send_sigqueue() checks PF_EXITING, then locks p->sighand->siglock. This is
unsafe: 'p' can exit in between and set ->sighand = NULL. The race is
theoretical, the window is tiny and irqs are disabled by the caller, so I
don't think we need the fix for -stable tree.
Convert send_sigqueue() to use lock_task_sighand() helper.
Also, delete 'p->flags & PF_EXITING' re-check, it is unneeded and the
comment is wrong.
Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'mm/msync.c')
0 files changed, 0 insertions, 0 deletions