perf: Disable monitoring on setuid processes for regular users
authorStephane Eranian <eranian@google.com>
Thu, 20 Jun 2013 09:36:28 +0000 (11:36 +0200)
committerBen Hutchings <ben@decadent.org.uk>
Sat, 27 Jul 2013 04:34:19 +0000 (05:34 +0100)
commite79c0b93200a2dc945e853591002b6e5bc529b91
tree23481695d482d30d8a1d89138239fa3f2e7c03db
parent809feb1735e5f4210ba455108ca91cb55b7b8610
perf: Disable monitoring on setuid processes for regular users

commit 2976b10f05bd7f6dab9f9e7524451ddfed656a89 upstream.

There was a a bug in setup_new_exec(), whereby
the test to disabled perf monitoring was not
correct because the new credentials for the
process were not yet committed and therefore
the get_dumpable() test was never firing.

The patch fixes the problem by moving the
perf_event test until after the credentials
are committed.

Signed-off-by: Stephane Eranian <eranian@google.com>
Tested-by: Jiri Olsa <jolsa@redhat.com>
Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
fs/exec.c