]> bbs.cooldavid.org Git - net-next-2.6.git/commit - kernel/signal.c
[PATCH] send_sigqueue: simplify and fix the race
authorOleg Nesterov <oleg@tv-sign.ru>
Wed, 29 Mar 2006 00:11:30 +0000 (16:11 -0800)
committerLinus Torvalds <torvalds@g5.osdl.org>
Wed, 29 Mar 2006 02:36:44 +0000 (18:36 -0800)
commit547679087bc60277d11b11631d826895762c4505
tree91759a5584b9d42302d4b54ecdde644bc47e781d
parenta1d5e21e3e388fb2c16463d007e788b1e41b74f1
[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>
kernel/signal.c