Alex Klyubin | f5446eb | 2017-03-23 14:27:32 -0700 | [diff] [blame] | 1 | typeattribute incidentd coredomain; |
Yi Jin | bc24ba7 | 2018-01-22 14:00:46 -0800 | [diff] [blame] | 2 | typeattribute incidentd mlstrustedsubject; |
Alex Klyubin | f5446eb | 2017-03-23 14:27:32 -0700 | [diff] [blame] | 3 | |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 4 | init_daemon_domain(incidentd) |
Nick Kralevich | 5e37271 | 2018-09-27 10:21:37 -0700 | [diff] [blame] | 5 | type incidentd_exec, system_file_type, exec_type, file_type; |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 6 | binder_use(incidentd) |
| 7 | wakelock_use(incidentd) |
| 8 | |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 9 | # Allow incidentd to scan through /proc/pid for all processes |
| 10 | r_dir_file(incidentd, domain) |
| 11 | |
Yi Jin | bc24ba7 | 2018-01-22 14:00:46 -0800 | [diff] [blame] | 12 | # Allow incidentd to kill incident_helper when timeout |
| 13 | allow incidentd incident_helper:process sigkill; |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 14 | |
| 15 | # Allow executing files on system, such as: |
| 16 | # /system/bin/toolbox |
| 17 | # /system/bin/logcat |
| 18 | # /system/bin/dumpsys |
| 19 | allow incidentd system_file:file execute_no_trans; |
| 20 | allow incidentd toolbox_exec:file rx_file_perms; |
| 21 | |
Yi Jin | 880de35 | 2018-06-26 14:17:04 -0700 | [diff] [blame] | 22 | # section id 1002, allow reading kernel version /proc/version |
| 23 | allow incidentd proc_version:file r_file_perms; |
| 24 | |
Yi Jin | bc24ba7 | 2018-01-22 14:00:46 -0800 | [diff] [blame] | 25 | # section id 2001, allow reading /proc/pagetypeinfo |
| 26 | allow incidentd proc_pagetypeinfo:file r_file_perms; |
| 27 | |
| 28 | # section id 2002, allow reading /d/wakeup_sources |
| 29 | allow incidentd debugfs_wakeup_sources:file r_file_perms; |
| 30 | |
| 31 | # section id 2003, allow executing top |
| 32 | allow incidentd proc_meminfo:file { open read }; |
| 33 | |
| 34 | # section id 2004, allow reading /sys/devices/system/cpu/cpufreq/all_time_in_state |
| 35 | allow incidentd sysfs_devices_system_cpu:file r_file_perms; |
| 36 | |
Kweku Adams | 06ac7db | 2018-03-06 17:42:35 -0800 | [diff] [blame] | 37 | # section id 2005, allow reading ps dump in full |
| 38 | allow incidentd domain:process getattr; |
| 39 | |
Yi Jin | bc24ba7 | 2018-01-22 14:00:46 -0800 | [diff] [blame] | 40 | # section id 2006, allow reading /sys/class/power_supply/bms/battery_type |
| 41 | allow incidentd sysfs_batteryinfo:dir { search }; |
| 42 | allow incidentd sysfs_batteryinfo:file r_file_perms; |
| 43 | |
Yi Jin | 76238cd | 2018-03-30 10:14:08 -0700 | [diff] [blame] | 44 | # section id 2007, allow reading LAST_KMSG /sys/fs/pstore/console-ramoops |
| 45 | userdebug_or_eng(`allow incidentd pstorefs:dir search'); |
| 46 | userdebug_or_eng(`allow incidentd pstorefs:file r_file_perms'); |
| 47 | |
Bookatz | d2f3d18 | 2018-11-05 15:01:39 -0800 | [diff] [blame] | 48 | # section id 3023, allow obtaining stats report |
| 49 | allow incidentd stats_service:service_manager find; |
| 50 | binder_call(incidentd, statsd) |
| 51 | |
Ryan Savitski | 72f247f | 2019-05-29 17:50:44 +0100 | [diff] [blame] | 52 | # section id 3026, allow reading /data/misc/perfetto-traces. |
Florian Mayer | 487bf1c | 2020-03-26 17:19:21 +0100 | [diff] [blame^] | 53 | allow incidentd perfetto_traces_data_file:dir r_dir_perms; |
| 54 | allow incidentd perfetto_traces_data_file:file r_file_perms; |
Ryan Savitski | 72f247f | 2019-05-29 17:50:44 +0100 | [diff] [blame] | 55 | |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 56 | # Create and write into /data/misc/incidents |
| 57 | allow incidentd incident_data_file:dir rw_dir_perms; |
| 58 | allow incidentd incident_data_file:file create_file_perms; |
| 59 | |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 60 | # Enable incidentd to get stack traces. |
| 61 | binder_use(incidentd) |
| 62 | hwbinder_use(incidentd) |
| 63 | allow incidentd hwservicemanager:hwservice_manager { list }; |
| 64 | get_prop(incidentd, hwservicemanager_prop) |
| 65 | allow incidentd hidl_manager_hwservice:hwservice_manager { find }; |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 66 | |
Kweku Adams | 06ac7db | 2018-03-06 17:42:35 -0800 | [diff] [blame] | 67 | # Read files in /proc |
| 68 | allow incidentd { |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 69 | proc_cmdline |
| 70 | proc_pipe_conf |
Kweku Adams | 06ac7db | 2018-03-06 17:42:35 -0800 | [diff] [blame] | 71 | proc_stat |
| 72 | }:file r_file_perms; |
| 73 | |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 74 | # Signal java processes to dump their stack and get the results |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 75 | allow incidentd { appdomain ephemeral_app system_server }:process signal; |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 76 | |
| 77 | # Signal native processes to dump their stack. |
| 78 | # This list comes from native_processes_to_dump in incidentd/utils.c |
| 79 | allow incidentd { |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 80 | # This list comes from native_processes_to_dump in dumputils/dump_utils.cpp |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 81 | audioserver |
| 82 | cameraserver |
| 83 | drmserver |
| 84 | inputflinger |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 85 | mediadrmserver |
| 86 | mediaextractor |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 87 | mediametrics |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 88 | mediaserver |
| 89 | sdcardd |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 90 | statsd |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 91 | surfaceflinger |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 92 | |
| 93 | # This list comes from hal_interfaces_to_dump in dumputils/dump_utils.cpp |
| 94 | hal_audio_server |
| 95 | hal_bluetooth_server |
| 96 | hal_camera_server |
Pawin Vongmasa | 609c243 | 2019-04-30 05:09:28 -0700 | [diff] [blame] | 97 | hal_codec2_server |
Kevin Chyn | 6d976f4 | 2019-06-19 11:12:11 -0700 | [diff] [blame] | 98 | hal_face_server |
Chia-I Wu | 7c5e18c | 2018-08-09 15:56:09 -0700 | [diff] [blame] | 99 | hal_graphics_allocator_server |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 100 | hal_graphics_composer_server |
Yifan Hong | 0d53ef2 | 2018-11-27 17:35:31 -0800 | [diff] [blame] | 101 | hal_health_server |
Steven Moreland | 7baf725 | 2018-05-25 16:23:37 -0700 | [diff] [blame] | 102 | hal_omx_server |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 103 | hal_sensors_server |
| 104 | hal_vr_server |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 105 | }:process signal; |
| 106 | |
| 107 | # Allow incidentd to make binder calls to any binder service |
Yi Jin | bc24ba7 | 2018-01-22 14:00:46 -0800 | [diff] [blame] | 108 | binder_call(incidentd, system_server) |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 109 | binder_call(incidentd, appdomain) |
| 110 | |
| 111 | # Reading /proc/PID/maps of other processes |
Kweku Adams | 985db6d | 2018-03-12 16:21:40 -0700 | [diff] [blame] | 112 | userdebug_or_eng(`allow incidentd self:global_capability_class_set { sys_ptrace }'); |
| 113 | # incidentd has capability sys_ptrace, but should only use that capability for |
| 114 | # accessing sensitive /proc/PID files, never for using ptrace attach. |
| 115 | neverallow incidentd *:process ptrace; |
| 116 | |
| 117 | allow incidentd self:global_capability_class_set { |
| 118 | # Send signals to processes |
| 119 | kill |
| 120 | }; |
| 121 | |
| 122 | # Connect to tombstoned to intercept dumps. |
| 123 | unix_socket_connect(incidentd, tombstoned_intercept, tombstoned) |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 124 | |
| 125 | # Run a shell. |
| 126 | allow incidentd shell_exec:file rx_file_perms; |
| 127 | |
Mike Ma | 08f494d | 2020-01-17 19:00:16 -0800 | [diff] [blame] | 128 | # For running am, incident-helper-cmd and similar framework commands. |
| 129 | # Run /system/bin/app_process. |
| 130 | allow incidentd zygote_exec:file { rx_file_perms }; |
Mike Ma | ab61935 | 2020-02-11 12:56:36 -0800 | [diff] [blame] | 131 | # Access the runtime feature flag properties. |
| 132 | get_prop(incidentd, device_config_runtime_native_prop) |
| 133 | get_prop(incidentd, device_config_runtime_native_boot_prop) |
| 134 | # ART locks profile files. |
| 135 | allow incidentd system_file:file lock; |
| 136 | # Incidentd should never exec from the memory (e.g. JIT cache). These denials are expected. |
| 137 | dontaudit incidentd dalvikcache_data_file:dir r_dir_perms; |
| 138 | dontaudit incidentd tmpfs:file rwx_file_perms; |
Mike Ma | 08f494d | 2020-01-17 19:00:16 -0800 | [diff] [blame] | 139 | |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 140 | # logd access - work to be done is a PII safe log (possibly an event log?) |
Yi Jin | bc24ba7 | 2018-01-22 14:00:46 -0800 | [diff] [blame] | 141 | userdebug_or_eng(`read_logd(incidentd)') |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 142 | # TODO control_logd(incidentd) |
| 143 | |
Mike Ma | 08f494d | 2020-01-17 19:00:16 -0800 | [diff] [blame] | 144 | # Access /data/misc/logd |
Mike Ma | ab61935 | 2020-02-11 12:56:36 -0800 | [diff] [blame] | 145 | r_dir_file(incidentd, misc_logd_file) |
Mike Ma | 08f494d | 2020-01-17 19:00:16 -0800 | [diff] [blame] | 146 | |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 147 | # Allow incidentd to find these standard groups of services. |
| 148 | # Others can be whitelisted individually. |
| 149 | allow incidentd { |
| 150 | system_server_service |
| 151 | app_api_service |
| 152 | system_api_service |
| 153 | }:service_manager find; |
| 154 | |
| 155 | # Only incidentd can publish the binder service |
| 156 | add_service(incidentd, incident_service) |
| 157 | |
Mike Ma | de3a3e4 | 2018-12-03 10:35:40 -0800 | [diff] [blame] | 158 | # Allow pipes only from dumpstate and incident |
| 159 | allow incidentd { dumpstate incident }:fd use; |
| 160 | allow incidentd { dumpstate incident }:fifo_file write; |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 161 | |
| 162 | # Allow incident to call back to incident with status updates. |
| 163 | binder_call(incidentd, incident) |
| 164 | |
zhouwenjie | c8ae8fa | 2019-10-25 13:45:06 -0700 | [diff] [blame] | 165 | # Read device serial number from system properties |
| 166 | # This is used to track reports from lab testing devices |
| 167 | userdebug_or_eng(` |
| 168 | get_prop(incidentd, serialno_prop) |
| 169 | ') |
| 170 | |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 171 | ### |
| 172 | ### neverallow rules |
| 173 | ### |
| 174 | |
Ryan Savitski | 72f247f | 2019-05-29 17:50:44 +0100 | [diff] [blame] | 175 | # only specific domains can find the incident service |
yro | cf38ca5 | 2018-01-22 13:56:41 -0800 | [diff] [blame] | 176 | neverallow { |
| 177 | domain |
Mike Ma | de3a3e4 | 2018-12-03 10:35:40 -0800 | [diff] [blame] | 178 | -dumpstate |
yro | cf38ca5 | 2018-01-22 13:56:41 -0800 | [diff] [blame] | 179 | -incident |
| 180 | -incidentd |
Florian Mayer | 487bf1c | 2020-03-26 17:19:21 +0100 | [diff] [blame^] | 181 | -perfetto |
Ashwini Oruganti | 73e1229 | 2019-12-09 15:37:05 -0800 | [diff] [blame] | 182 | -permissioncontroller_app |
Joe Onorato | 9cc5c09 | 2019-03-16 15:45:45 -0700 | [diff] [blame] | 183 | -priv_app |
yro | cf38ca5 | 2018-01-22 13:56:41 -0800 | [diff] [blame] | 184 | -statsd |
| 185 | -system_app |
| 186 | -system_server |
| 187 | } incident_service:service_manager find; |
Joe Onorato | 41f93db | 2016-11-20 23:23:04 -0800 | [diff] [blame] | 188 | |
| 189 | # only incidentd and the other root services in limited circumstances |
| 190 | # can get to the files in /data/misc/incidents |
| 191 | # |
| 192 | # write, execute, append are forbidden almost everywhere |
| 193 | neverallow { domain -incidentd -init -vold } incident_data_file:file { |
| 194 | w_file_perms |
| 195 | x_file_perms |
| 196 | create |
| 197 | rename |
| 198 | setattr |
| 199 | unlink |
| 200 | append |
| 201 | }; |
| 202 | # read is also allowed by system_server, for when the file is handed to dropbox |
| 203 | neverallow { domain -incidentd -init -vold -system_server } incident_data_file:file r_file_perms; |
| 204 | # limited access to the directory itself |
| 205 | neverallow { domain -incidentd -init -vold } incident_data_file:dir create_dir_perms; |
| 206 | |