tequilaOS/platform_system_sepolicy
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Revert "Revert "allow simpleperf to profile more app types.""

Browse source 
This reverts commit dd2079d7f0.

Bug: 199086135
Test: run simpleperf to record systemui.
Change-Id: Ibc6017d53a9835a2f8ff5409c825c0d70ef23e25
This commit is contained in:
Yabin Cui 2021-10-27 11:05:01 -07:00
parent 388bbbccb3
commit 3696a20b11
3 changed files with 18 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
private/simpleperf.te
View file

@ -5,7 +5,16 @@
typeattribute simpleperf coredomain; typeattribute simpleperf coredomain;
type simpleperf_exec, system_file_type, exec_type, file_type; type simpleperf_exec, system_file_type, exec_type, file_type;
domain_auto_trans({ untrusted_app_all -runas_app }, simpleperf_exec, simpleperf) # Define apps that can be marked debuggable/profileable and be profiled by simpleperf.
define(`simpleperf_profileable_apps', `{
ephemeral_app
isolated_app
platform_app
priv_app
untrusted_app_all
}')
domain_auto_trans({ simpleperf_profileable_apps -runas_app }, simpleperf_exec, simpleperf)
# When running in this domain, simpleperf is scoped to profiling an individual # When running in this domain, simpleperf is scoped to profiling an individual
# app. The necessary MAC permissions for profiling are more maintainable and # app. The necessary MAC permissions for profiling are more maintainable and
@ -16,14 +25,19 @@ untrusted_app_domain(simpleperf)
# Allow ptrace attach to the target app, for reading JIT debug info (using # Allow ptrace attach to the target app, for reading JIT debug info (using
# process_vm_readv) during unwinding and symbolization. # process_vm_readv) during unwinding and symbolization.
allow simpleperf untrusted_app_all:process ptrace; allow simpleperf simpleperf_profileable_apps:process ptrace;
# Allow using perf_event_open syscall for profiling the target app. # Allow using perf_event_open syscall for profiling the target app.
allow simpleperf self:perf_event { open read write kernel }; allow simpleperf self:perf_event { open read write kernel };
# Allow /proc/<pid> access for the target app (for example, when trying to # Allow /proc/<pid> access for the target app (for example, when trying to
# discover it by cmdline). # discover it by cmdline).
r_dir_file(simpleperf, untrusted_app_all) r_dir_file(simpleperf, simpleperf_profileable_apps)
# Allow apps signalling simpleperf domain, which is the domain that the simpleperf
# profiler runs as when executed by the app. The signals are used to control
# the profiler (which would be profiling the app that is sending the signal).
allow simpleperf_profileable_apps simpleperf:process signal;
# Suppress denial logspam when simpleperf is trying to find a matching process # Suppress denial logspam when simpleperf is trying to find a matching process
# by scanning /proc/<pid>/cmdline files. The /proc/<pid> directories are within # by scanning /proc/<pid>/cmdline files. The /proc/<pid> directories are within

2
private/simpleperf_app_runner.te
View file

@ -21,7 +21,7 @@ allow simpleperf_app_runner self:global_capability_class_set { setuid setgid };
# simpleperf_app_runner switches to the app security context. # simpleperf_app_runner switches to the app security context.
selinux_check_context(simpleperf_app_runner) # validate context selinux_check_context(simpleperf_app_runner) # validate context
allow simpleperf_app_runner self:process setcurrent; allow simpleperf_app_runner self:process setcurrent;
allow simpleperf_app_runner untrusted_app_all:process dyntransition; # setcon allow simpleperf_app_runner { ephemeral_app isolated_app platform_app priv_app untrusted_app_all }:process dyntransition; # setcon
# simpleperf_app_runner/libselinux needs access to seapp_contexts_file to # simpleperf_app_runner/libselinux needs access to seapp_contexts_file to
# determine which domain to transition to. # determine which domain to transition to.

5
private/untrusted_app_all.te
View file

@ -171,11 +171,6 @@ userdebug_or_eng(`
allow untrusted_app_all self:lockdown integrity; allow untrusted_app_all self:lockdown integrity;
') ')
# Allow signalling simpleperf domain, which is the domain that the simpleperf
# profiler runs as when executed by the app. The signals are used to control
# the profiler (which would be profiling the app that is sending the signal).
allow untrusted_app_all simpleperf:process signal;
# Allow running a VM for test/demo purposes. Note that access the service is # Allow running a VM for test/demo purposes. Note that access the service is
# still guarded with the `android.permission.MANAGE_VIRTUAL_MACHINE` # still guarded with the `android.permission.MANAGE_VIRTUAL_MACHINE`
# permission. The protection level of the permission is `signature|development` # permission. The protection level of the permission is `signature|development`