From 71d8467b75c8bf46352f59c88c3e1c2db32197ae Mon Sep 17 00:00:00 2001
From: Calin Juravle <calin@google.com>
Date: Thu, 3 May 2018 20:14:51 -0700
Subject: [PATCH] Allow system server to record its own profile
On userdebug builds we can now profile system server without disabling
selinux. This is the final piece, and allows the system server to save its
own profile.
Test: manual, on a device with system server profiling enabled
Bug: 73313191
Change-Id: Iaf9575d3cda19ae4c38f1e20a8e1b9288b7abc83
---
private/system_server.te | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/private/system_server.te b/private/system_server.te
index c5b83ecc8..045acc6d1 100644
--- a/private/system_server.te
+++ b/private/system_server.te
@@ -773,9 +773,14 @@ allow system_server netd:bpf map_read;
# Allow system_server to open profile snapshots for read.
# System server never reads the actual content. It passes the descriptor to
# to privileged apps which acquire the permissions to inspect the profiles.
-allow system_server user_profile_data_file:dir { search };
+allow system_server user_profile_data_file:dir { getattr search };
allow system_server user_profile_data_file:file { getattr open read };
+# On userdebug build we may profile system server. Allow it to write and create its own profile.
+userdebug_or_eng(`
+ allow system_server user_profile_data_file:file create_file_perms;
+')
+
userdebug_or_eng(`
# Allow system server to notify mediaextractor of the plugin update.
allow system_server mediaextractor_update_service:service_manager find;
--
GitLab