Re: [PATCH 1/2] signals: kill(-1) should only signal processes in the same namespace

From: Eric W. Biederman
Date: Thu Jul 17 2008 - 13:52:33 EST


Oleg Nesterov <oleg@xxxxxxxxxx> writes:

> kill_something_info() can't walk threads, think about the realtime signals.

walking threads is fine delivering signals to non thread group leaders is
a problem.

> Anyway, I think we should change kill_something_info(-1) to use rcu_read_lock()
> instead of tasklist.

Being dense I think the locking implications of a correct implementation are more
then we are ready to deal with to fix this bug. Although I remember discussing
it and seeing something reasonable.

Eric

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/