Re: [patch V2 2/6] posix-cpu-timers: Fix permission check regression

Frederic Weisbecker
2019-09-09T15:13:47Z

On Thu, Sep 05, 2019 at 11:15:08PM +0200, Thomas Gleixner wrote:
> + if (gettime) {
> + /*
> + * For clock_gettime(PROCESS) the task does not need to be
> + * the actual group leader. tsk->sighand gives
> + * access to the group's clock.
> + *
> + * Timers need the group leader because they take a
> + * reference on it and store the task pointer until the
> + * timer is destroyed.

Well, that would work with a non group leader as well but anyway that wouldn't
be pretty.

Reviewed-by: Frederic Weisbecker <frederic@kernel.org>