Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 1 | ### |
| 2 | ### A domain for further sandboxing privileged apps. |
| 3 | ### |
| 4 | |
Alex Klyubin | f5446eb | 2017-03-23 14:27:32 -0700 | [diff] [blame] | 5 | typeattribute priv_app coredomain; |
dcashman | 3e8dbf0 | 2016-12-08 11:23:34 -0800 | [diff] [blame] | 6 | app_domain(priv_app) |
dcashman | 2e00e63 | 2016-10-12 14:58:09 -0700 | [diff] [blame] | 7 | |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 8 | # Access the network. |
| 9 | net_domain(priv_app) |
| 10 | # Access bluetooth. |
| 11 | bluetooth_domain(priv_app) |
| 12 | |
dcashman | 2e00e63 | 2016-10-12 14:58:09 -0700 | [diff] [blame] | 13 | # Allow the allocation and use of ptys |
| 14 | # Used by: https://play.privileged.com/store/apps/details?id=jackpal.androidterm |
| 15 | create_pty(priv_app) |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 16 | |
| 17 | # webview crash handling depends on self ptrace (b/27697529, b/20150694, b/19277529#comment7) |
| 18 | allow priv_app self:process ptrace; |
| 19 | |
Nick Kralevich | e1ddd74 | 2018-10-27 15:20:38 -0700 | [diff] [blame] | 20 | # Allow loading executable code from writable priv-app home |
| 21 | # directories. This is a W^X violation, however, it needs |
| 22 | # to be supported for now for the following reasons. |
| 23 | # * /data/user_*/0/*/code_cache/* POSSIBLE uses (b/117841367) |
| 24 | # 1) com.android.opengl.shaders_cache |
| 25 | # 2) com.android.skia.shaders_cache |
| 26 | # 3) com.android.renderscript.cache |
| 27 | # * /data/user_de/0/com.google.android.gms/app_chimera |
| 28 | # TODO: Tighten (b/112357170) |
| 29 | allow priv_app privapp_data_file:file execute; |
Ashwini Oruganti | e6ed127 | 2019-11-26 11:02:31 -0800 | [diff] [blame^] | 30 | # b/142672293: No other priv-app should need this allow rule now that GMS core runs in its own domain. |
| 31 | userdebug_or_eng(` |
| 32 | auditallow priv_app privapp_data_file:file execute; |
| 33 | ') |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 34 | |
Nick Kralevich | 87e9123 | 2019-01-24 13:05:03 -0800 | [diff] [blame] | 35 | allow priv_app privapp_data_file:lnk_file create_file_perms; |
| 36 | |
Alan Stokes | 5c378a5 | 2019-03-21 23:52:30 +0000 | [diff] [blame] | 37 | # Priv apps can find services that expose both @SystemAPI and normal APIs. |
yro | 31b11d8 | 2018-01-09 11:27:36 -0800 | [diff] [blame] | 38 | allow priv_app app_api_service:service_manager find; |
Alan Stokes | 5c378a5 | 2019-03-21 23:52:30 +0000 | [diff] [blame] | 39 | allow priv_app system_api_service:service_manager find; |
| 40 | |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 41 | allow priv_app audioserver_service:service_manager find; |
| 42 | allow priv_app cameraserver_service:service_manager find; |
| 43 | allow priv_app drmserver_service:service_manager find; |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 44 | allow priv_app mediadrmserver_service:service_manager find; |
| 45 | allow priv_app mediaextractor_service:service_manager find; |
yro | 31b11d8 | 2018-01-09 11:27:36 -0800 | [diff] [blame] | 46 | allow priv_app mediametrics_service:service_manager find; |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 47 | allow priv_app mediaserver_service:service_manager find; |
Ricky Wai | c635297 | 2017-11-13 17:52:05 +0000 | [diff] [blame] | 48 | allow priv_app network_watchlist_service:service_manager find; |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 49 | allow priv_app nfc_service:service_manager find; |
Andrew Scull | 3717424 | 2017-02-17 13:51:32 +0000 | [diff] [blame] | 50 | allow priv_app oem_lock_service:service_manager find; |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 51 | allow priv_app persistent_data_block_service:service_manager find; |
yro | 31b11d8 | 2018-01-09 11:27:36 -0800 | [diff] [blame] | 52 | allow priv_app radio_service:service_manager find; |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 53 | allow priv_app recovery_service:service_manager find; |
yro | 31b11d8 | 2018-01-09 11:27:36 -0800 | [diff] [blame] | 54 | allow priv_app stats_service:service_manager find; |
Yiwei Zhang | 544d6b3 | 2019-02-07 15:00:55 -0800 | [diff] [blame] | 55 | |
| 56 | # Allow privileged apps to interact with gpuservice |
| 57 | binder_call(priv_app, gpuservice) |
Alan Stokes | 5c378a5 | 2019-03-21 23:52:30 +0000 | [diff] [blame] | 58 | allow priv_app gpu_service:service_manager find; |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 59 | |
| 60 | # Write to /cache. |
| 61 | allow priv_app { cache_file cache_recovery_file }:dir create_dir_perms; |
| 62 | allow priv_app { cache_file cache_recovery_file }:file create_file_perms; |
Nick Kralevich | 21cb045 | 2017-01-23 22:19:06 -0800 | [diff] [blame] | 63 | # /cache is a symlink to /data/cache on some devices. Allow reading the link. |
| 64 | allow priv_app cache_file:lnk_file r_file_perms; |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 65 | |
| 66 | # Write to /data/ota_package for OTA packages. |
| 67 | allow priv_app ota_package_file:dir rw_dir_perms; |
| 68 | allow priv_app ota_package_file:file create_file_perms; |
| 69 | |
| 70 | # Access to /data/media. |
| 71 | allow priv_app media_rw_data_file:dir create_dir_perms; |
| 72 | allow priv_app media_rw_data_file:file create_file_perms; |
| 73 | |
| 74 | # Used by Finsky / Android "Verify Apps" functionality when |
| 75 | # running "adb install foo.apk". |
| 76 | allow priv_app shell_data_file:file r_file_perms; |
| 77 | allow priv_app shell_data_file:dir r_dir_perms; |
| 78 | |
Max Bires | 715e2ae | 2018-03-13 09:56:27 -0700 | [diff] [blame] | 79 | # Allow traceur to pass file descriptors through a content provider to betterbug |
| 80 | allow priv_app trace_data_file:file { getattr read }; |
| 81 | |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 82 | # Allow verifier to access staged apks. |
| 83 | allow priv_app { apk_tmp_file apk_private_tmp_file }:dir r_dir_perms; |
| 84 | allow priv_app { apk_tmp_file apk_private_tmp_file }:file r_file_perms; |
| 85 | |
| 86 | # b/18504118: Allow reads from /data/anr/traces.txt |
| 87 | allow priv_app anr_data_file:file r_file_perms; |
| 88 | |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 89 | # For AppFuse. |
| 90 | allow priv_app vold:fd use; |
| 91 | allow priv_app fuse_device:chr_file { read write }; |
| 92 | |
Tri Vo | f92cfb9 | 2017-12-08 15:37:01 -0800 | [diff] [blame] | 93 | # /proc access |
| 94 | allow priv_app { |
| 95 | proc_vmstat |
| 96 | }:file r_file_perms; |
| 97 | |
| 98 | allow priv_app sysfs_type:dir search; |
| 99 | # Read access to /sys/class/net/wlan*/address |
| 100 | r_dir_file(priv_app, sysfs_net) |
| 101 | # Read access to /sys/block/zram*/mm_stat |
| 102 | r_dir_file(priv_app, sysfs_zram) |
| 103 | |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 104 | r_dir_file(priv_app, rootfs) |
| 105 | |
Sandeep Patil | 0465442 | 2017-04-19 11:35:15 -0700 | [diff] [blame] | 106 | # Allow GMS core to open kernel config for OTA matching through libvintf |
| 107 | allow priv_app config_gz:file { open read getattr }; |
Ashwini Oruganti | e6ed127 | 2019-11-26 11:02:31 -0800 | [diff] [blame^] | 108 | # b/142672293: No other priv-app should need this allow rule now that GMS core runs in its own domain. |
| 109 | userdebug_or_eng(` |
| 110 | auditallow priv_app config_gz:file { open read getattr }; |
| 111 | ') |
Sandeep Patil | 0465442 | 2017-04-19 11:35:15 -0700 | [diff] [blame] | 112 | |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 113 | # access the mac address |
| 114 | allowxperm priv_app self:udp_socket ioctl SIOCGIFHWADDR; |
| 115 | |
| 116 | # Allow GMS core to communicate with update_engine for A/B update. |
| 117 | binder_call(priv_app, update_engine) |
| 118 | allow priv_app update_engine_service:service_manager find; |
Ashwini Oruganti | e6ed127 | 2019-11-26 11:02:31 -0800 | [diff] [blame^] | 119 | # b/142672293: No other priv-app should need this allow rule now that GMS core runs in its own domain. |
| 120 | userdebug_or_eng(` |
| 121 | auditallow priv_app update_engine:binder { call transfer }; |
| 122 | auditallow update_engine priv_app:binder transfer; |
| 123 | auditallow priv_app update_engine:fd use; |
| 124 | auditallow priv_app update_engine_service:service_manager find; |
| 125 | ') |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 126 | |
Jin Qian | 00a1789 | 2017-04-12 17:38:11 -0700 | [diff] [blame] | 127 | # Allow GMS core to communicate with dumpsys storaged. |
| 128 | binder_call(priv_app, storaged) |
| 129 | allow priv_app storaged_service:service_manager find; |
Ashwini Oruganti | e6ed127 | 2019-11-26 11:02:31 -0800 | [diff] [blame^] | 130 | # b/142672293: No other priv-app should need this allow rule now that GMS core runs in its own domain. |
| 131 | userdebug_or_eng(` |
| 132 | auditallow priv_app storaged:binder { call transfer }; |
| 133 | auditallow storaged priv_app:binder transfer; |
| 134 | auditallow priv_app storaged:fd use; |
| 135 | auditallow priv_app storaged_service:service_manager find; |
| 136 | ') |
| 137 | |
Jin Qian | 00a1789 | 2017-04-12 17:38:11 -0700 | [diff] [blame] | 138 | |
Tao Bao | d7d9cfc | 2017-10-16 21:57:12 -0700 | [diff] [blame] | 139 | # Allow GMS core to access system_update_service (e.g. to publish pending |
| 140 | # system update info). |
| 141 | allow priv_app system_update_service:service_manager find; |
Ashwini Oruganti | e6ed127 | 2019-11-26 11:02:31 -0800 | [diff] [blame^] | 142 | # b/142672293: No other priv-app should need this allow rule now that GMS core runs in its own domain. |
| 143 | userdebug_or_eng(` |
| 144 | auditallow priv_app system_update_service:service_manager find; |
| 145 | ') |
Tao Bao | d7d9cfc | 2017-10-16 21:57:12 -0700 | [diff] [blame] | 146 | |
yro | 31b11d8 | 2018-01-09 11:27:36 -0800 | [diff] [blame] | 147 | # Allow GMS core to communicate with statsd. |
| 148 | binder_call(priv_app, statsd) |
Ashwini Oruganti | e6ed127 | 2019-11-26 11:02:31 -0800 | [diff] [blame^] | 149 | # b/142672293: No other priv-app should need this allow rule now that GMS core runs in its own domain. |
| 150 | userdebug_or_eng(` |
| 151 | auditallow priv_app statsd:binder { call transfer }; |
| 152 | auditallow statsd priv_app:binder transfer; |
| 153 | auditallow priv_app statsd:fd use; |
| 154 | ') |
yro | 31b11d8 | 2018-01-09 11:27:36 -0800 | [diff] [blame] | 155 | |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 156 | # Allow Phone to read/write cached ringtones (opened by system). |
| 157 | allow priv_app ringtone_file:file { getattr read write }; |
| 158 | |
| 159 | # Access to /data/preloads |
| 160 | allow priv_app preloads_data_file:file r_file_perms; |
| 161 | allow priv_app preloads_data_file:dir r_dir_perms; |
Fyodor Kupolov | b238fe6 | 2017-03-14 11:42:03 -0700 | [diff] [blame] | 162 | allow priv_app preloads_media_file:file r_file_perms; |
| 163 | allow priv_app preloads_media_file:dir r_dir_perms; |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 164 | |
Shawn Willden | a0c7f01 | 2017-04-11 09:41:25 -0600 | [diff] [blame] | 165 | # Allow privileged apps (e.g. GMS core) to generate unique hardware IDs |
| 166 | allow priv_app keystore:keystore_key gen_unique_id; |
Ashwini Oruganti | e6ed127 | 2019-11-26 11:02:31 -0800 | [diff] [blame^] | 167 | # b/142672293: No other priv-app should need this allow rule now that GMS core runs in its own domain. |
| 168 | userdebug_or_eng(` |
| 169 | auditallow priv_app keystore:keystore_key gen_unique_id; |
| 170 | ') |
Shawn Willden | a0c7f01 | 2017-04-11 09:41:25 -0600 | [diff] [blame] | 171 | |
Dan Cashman | 91d398d | 2017-09-26 12:58:29 -0700 | [diff] [blame] | 172 | # Allow GMS core to access /sys/fs/selinux/policyvers for compatibility check |
| 173 | allow priv_app selinuxfs:file r_file_perms; |
Ashwini Oruganti | e6ed127 | 2019-11-26 11:02:31 -0800 | [diff] [blame^] | 174 | # b/142672293: No other priv-app should need this allow rule now that GMS core runs in its own domain. |
| 175 | userdebug_or_eng(` |
| 176 | auditallow priv_app selinuxfs:file r_file_perms; |
| 177 | ') |
Dan Cashman | 91d398d | 2017-09-26 12:58:29 -0700 | [diff] [blame] | 178 | |
Mark Salyzyn | d33a9a1 | 2016-11-07 15:11:39 -0800 | [diff] [blame] | 179 | read_runtime_log_tags(priv_app) |
| 180 | |
Primiano Tucci | c80f9e0 | 2017-12-21 03:51:15 +0100 | [diff] [blame] | 181 | # Write app-specific trace data to the Perfetto traced damon. This requires |
| 182 | # connecting to its producer socket and obtaining a (per-process) tmpfs fd. |
Florian Mayer | 5e52281 | 2019-10-08 16:15:14 +0100 | [diff] [blame] | 183 | perfetto_producer(priv_app) |
Primiano Tucci | c80f9e0 | 2017-12-21 03:51:15 +0100 | [diff] [blame] | 184 | |
Joe Onorato | 9cc5c09 | 2019-03-16 15:45:45 -0700 | [diff] [blame] | 185 | # Allow priv_apps to request and collect incident reports. |
| 186 | # (Also requires DUMP and PACKAGE_USAGE_STATS permissions) |
| 187 | allow priv_app incident_service:service_manager find; |
| 188 | binder_call(priv_app, incidentd) |
| 189 | allow priv_app incidentd:fifo_file { read write }; |
| 190 | |
Ryan Savitski | ca0690e | 2019-01-16 16:29:43 +0000 | [diff] [blame] | 191 | # Allow heap profiling if the app opts in by being marked |
| 192 | # profileable/debuggable. |
| 193 | can_profile_heap(priv_app) |
| 194 | |
Hung-ying Tyan | 565384d | 2019-04-26 16:14:52 +0800 | [diff] [blame] | 195 | # Allow priv_apps to check whether Dynamic System Update is enabled |
| 196 | get_prop(priv_app, dynamic_system_prop) |
| 197 | |
Jeff Vander Stoep | 6d8a876 | 2018-01-18 08:55:02 -0800 | [diff] [blame] | 198 | # suppress denials for non-API accesses. |
Dan Cashman | 91d398d | 2017-09-26 12:58:29 -0700 | [diff] [blame] | 199 | dontaudit priv_app exec_type:file getattr; |
Jeff Vander Stoep | 6233848 | 2017-10-20 13:35:42 -0700 | [diff] [blame] | 200 | dontaudit priv_app device:dir read; |
Joel Galenson | 9ec59f6 | 2018-04-20 15:27:21 -0700 | [diff] [blame] | 201 | dontaudit priv_app fs_bpf:dir search; |
Jeff Vander Stoep | 9dc1d53 | 2018-04-04 14:36:13 -0700 | [diff] [blame] | 202 | dontaudit priv_app net_dns_prop:file read; |
Tri Vo | f92cfb9 | 2017-12-08 15:37:01 -0800 | [diff] [blame] | 203 | dontaudit priv_app proc:file read; |
Jeff Vander Stoep | 6233848 | 2017-10-20 13:35:42 -0700 | [diff] [blame] | 204 | dontaudit priv_app proc_interrupts:file read; |
| 205 | dontaudit priv_app proc_modules:file read; |
Tri Vo | e319c03 | 2019-05-09 16:14:04 -0700 | [diff] [blame] | 206 | dontaudit priv_app proc_net:file read; |
Jeff Vander Stoep | e88d649 | 2018-01-29 20:50:59 -0800 | [diff] [blame] | 207 | dontaudit priv_app proc_stat:file read; |
Jeff Vander Stoep | 6d8a876 | 2018-01-18 08:55:02 -0800 | [diff] [blame] | 208 | dontaudit priv_app proc_version:file read; |
Jeff Vander Stoep | 9dc1d53 | 2018-04-04 14:36:13 -0700 | [diff] [blame] | 209 | dontaudit priv_app sysfs:dir read; |
Jeff Vander Stoep | 4894d9f | 2018-06-29 11:04:24 -0700 | [diff] [blame] | 210 | dontaudit priv_app sysfs:file read; |
Jeff Vander Stoep | 9dc1d53 | 2018-04-04 14:36:13 -0700 | [diff] [blame] | 211 | dontaudit priv_app sysfs_android_usb:file read; |
Jeff Vander Stoep | 90bd1de | 2019-10-25 11:01:40 +0200 | [diff] [blame] | 212 | dontaudit priv_app sysfs_dm:file r_file_perms; |
Jeff Vander Stoep | 6d8a876 | 2018-01-18 08:55:02 -0800 | [diff] [blame] | 213 | dontaudit priv_app wifi_prop:file read; |
Jaekyun Seok | 224921d | 2018-04-09 12:07:32 +0900 | [diff] [blame] | 214 | dontaudit priv_app { wifi_prop exported_wifi_prop }:file read; |
Dan Cashman | 91d398d | 2017-09-26 12:58:29 -0700 | [diff] [blame] | 215 | |
Nathan Harold | ee26864 | 2017-12-14 18:20:30 -0800 | [diff] [blame] | 216 | # allow privileged apps to use UDP sockets provided by the system server but not |
| 217 | # modify them other than to connect |
Nathan Harold | 252b015 | 2018-03-27 06:34:54 -0700 | [diff] [blame] | 218 | allow priv_app system_server:udp_socket { |
| 219 | connect getattr read recvfrom sendto write getopt setopt }; |
Nathan Harold | ee26864 | 2017-12-14 18:20:30 -0800 | [diff] [blame] | 220 | |
Jeff Vander Stoep | 9c7396d | 2018-06-01 12:12:11 -0700 | [diff] [blame] | 221 | # Attempts to write to system_data_file is generally a sign |
| 222 | # that apps are attempting to access encrypted storage before |
| 223 | # the ACTION_USER_UNLOCKED intent is delivered. Suppress this |
| 224 | # denial to prevent apps from spamming the logs. |
| 225 | dontaudit priv_app system_data_file:dir write; |
| 226 | |
Alex Klyubin | 92295ef | 2017-01-05 15:44:32 -0800 | [diff] [blame] | 227 | ### |
| 228 | ### neverallow rules |
| 229 | ### |
| 230 | |
| 231 | # Receive or send uevent messages. |
| 232 | neverallow priv_app domain:netlink_kobject_uevent_socket *; |
| 233 | |
| 234 | # Receive or send generic netlink messages |
| 235 | neverallow priv_app domain:netlink_socket *; |
| 236 | |
| 237 | # Too much leaky information in debugfs. It's a security |
| 238 | # best practice to ensure these files aren't readable. |
| 239 | neverallow priv_app debugfs:file read; |
| 240 | |
| 241 | # Do not allow privileged apps to register services. |
| 242 | # Only trusted components of Android should be registering |
| 243 | # services. |
| 244 | neverallow priv_app service_manager_type:service_manager add; |
| 245 | |
| 246 | # Do not allow privileged apps to connect to the property service |
| 247 | # or set properties. b/10243159 |
| 248 | neverallow priv_app property_socket:sock_file write; |
| 249 | neverallow priv_app init:unix_stream_socket connectto; |
| 250 | neverallow priv_app property_type:property_service set; |
| 251 | |
| 252 | # Do not allow priv_app to be assigned mlstrustedsubject. |
| 253 | # This would undermine the per-user isolation model being |
| 254 | # enforced via levelFrom=user in seapp_contexts and the mls |
| 255 | # constraints. As there is no direct way to specify a neverallow |
| 256 | # on attribute assignment, this relies on the fact that fork |
| 257 | # permission only makes sense within a domain (hence should |
| 258 | # never be granted to any other domain within mlstrustedsubject) |
| 259 | # and priv_app is allowed fork permission to itself. |
| 260 | neverallow priv_app mlstrustedsubject:process fork; |
| 261 | |
| 262 | # Do not allow priv_app to hard link to any files. |
| 263 | # In particular, if priv_app links to other app data |
| 264 | # files, installd will not be able to guarantee the deletion |
| 265 | # of the linked to file. Hard links also contribute to security |
| 266 | # bugs, so we want to ensure priv_app never has this |
| 267 | # capability. |
| 268 | neverallow priv_app file_type:file link; |
Max Bires | 715e2ae | 2018-03-13 09:56:27 -0700 | [diff] [blame] | 269 | |
| 270 | # priv apps should not be able to open trace data files, they should depend |
| 271 | # upon traceur to pass a file descriptor which they can then read |
| 272 | neverallow priv_app trace_data_file:dir *; |
| 273 | neverallow priv_app trace_data_file:file { no_w_file_perms open }; |
Tri Vo | f55c989 | 2018-10-10 22:48:15 +0000 | [diff] [blame] | 274 | |
| 275 | # Do not allow priv_app access to cgroups. |
| 276 | neverallow priv_app cgroup:file *; |
Nick Kralevich | e1ddd74 | 2018-10-27 15:20:38 -0700 | [diff] [blame] | 277 | |
| 278 | # Do not allow loading executable code from non-privileged |
| 279 | # application home directories. Code loading across a security boundary |
| 280 | # is dangerous and allows a full compromise of a privileged process |
| 281 | # by an unprivileged process. b/112357170 |
| 282 | neverallow priv_app app_data_file:file no_x_file_perms; |
Nick Kralevich | 87e9123 | 2019-01-24 13:05:03 -0800 | [diff] [blame] | 283 | |
| 284 | # Do not follow untrusted app provided symlinks |
| 285 | neverallow priv_app app_data_file:lnk_file { open read getattr }; |