lkml.org 
[lkml]   [2014]   [Jul]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[PATCH 0/5] perf: Fix tracepoint events permissions check
Date
hi,
sending fix for bug reported by Alexander Yarygin in here:
http://marc.info/?l=linux-kernel&m=140475133707722&w=2

The main problem was, that the event_init tracepoint callback
checked permission of the 'current' task instead of the event
owner task.

While this is ok for perf_event_open syscall check, it is wrong
once event_init is called during fork to create child events.
In this case the permission of the forked task is checked instead
of the owner task of the parent event.

Changing tracepoint permission code to check event's owner task,
plus some other changes I needed for this.

thanks,
jirka


---
Jiri Olsa (5):
perf: Make perf_init_event function static
perf: Destroy event's children on task exit
perf: Initialize owner before calling event_init callback
perf: Move event owner retrieval into perf_event_get_owner
perf: Check event's owner permission in tracepoint init callback

include/linux/perf_event.h | 1 +
kernel/events/core.c | 72 +++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------------
kernel/trace/trace_event_perf.c | 19 +++++++++++++++++--
3 files changed, 73 insertions(+), 19 deletions(-)


\
 
 \ /
  Last update: 2014-07-11 14:41    [W:0.181 / U:0.252 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site