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

2019-09-05 Thread Thomas Gleixner
On Thu, 5 Sep 2019, Frederic Weisbecker wrote: > On Thu, Sep 05, 2019 at 02:03:41PM +0200, Thomas Gleixner wrote: > > + if (gettime) { > > + /* > > +* For clock_gettime() the task does not need to be the > > +* actual group leader. tsk->sighand gives access to th

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

2019-09-05 Thread Frederic Weisbecker
On Thu, Sep 05, 2019 at 02:03:41PM +0200, Thomas Gleixner wrote: > The recent consolidation of the three permission checks introduced a subtle > regression. For timer_create() with a process wide timer it returns the > current task if the lookup through the PID which is encoded into the > clockid r

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

2019-09-05 Thread Thomas Gleixner
The recent consolidation of the three permission checks introduced a subtle regression. For timer_create() with a process wide timer it returns the current task if the lookup through the PID which is encoded into the clockid results in returning current. That's broken because it does not validate