perf: Check permission only for parent tracepoint event
authorJiri Olsa <jolsa@kernel.org>
Wed, 16 Jul 2014 12:33:29 +0000 (14:33 +0200)
committerIngo Molnar <mingo@kernel.org>
Mon, 28 Jul 2014 08:01:38 +0000 (10:01 +0200)
commitf4be073db878d0e79f74bc36f1642847781791a0
tree27af2c38ab25f889f0e850ce856aaf41a15add57
parent5030c69755416d19516c0a61cd988a0e0062e041
perf: Check permission only for parent tracepoint event

There's no need to check cloned event's permission once the
parent was already checked.

Also the code is checking 'current' process permissions, which
is not owner process for cloned events, thus could end up with
wrong permission check result.

Reported-by: Alexander Yarygin <yarygin@linux.vnet.ibm.com>
Tested-by: Alexander Yarygin <yarygin@linux.vnet.ibm.com>
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Signed-off-by: Peter Zijlstra <peterz@infradead.org>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Steven Rostedt <rostedt@goodmis.org>
Link: http://lkml.kernel.org/r/1405079782-8139-1-git-send-email-jolsa@kernel.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
kernel/trace/trace_event_perf.c