Oleg Nesterov b95c35e76b oom: fix the unsafe usage of badness() in proc_oom_score()
proc_oom_score(task) has a reference to task_struct, but that is all.
If this task was already released before we take tasklist_lock

	- we can't use task->group_leader, it points to nowhere

	- it is not safe to call badness() even if this task is
	  ->group_leader, has_intersects_mems_allowed() assumes
	  it is safe to iterate over ->thread_group list.

	- even worse, badness() can hit ->signal == NULL

Add the pid_alive() check to ensure __unhash_process() was not called.

Also, use "task" instead of task->group_leader. badness() should return
the same result for any sub-thread. Currently this is not true, but
this should be changed anyway.

Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Cc: stable@kernel.org
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2010-04-01 08:50:21 -07:00
..
2010-03-06 11:26:29 -08:00
2010-03-24 16:31:22 -07:00
2009-05-15 15:32:45 +02:00
2009-10-29 07:39:25 -07:00
2009-09-23 07:39:29 -07:00
2009-12-16 12:19:59 +01:00
2009-01-05 12:27:44 +03:00
2010-03-03 13:00:18 -05:00
2010-03-06 11:26:24 -08:00