Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 1 | ### |
| 2 | ### Services with isolatedProcess=true in their manifest. |
| 3 | ### |
| 4 | ### This file defines the rules for isolated apps. An "isolated |
| 5 | ### app" is an APP with UID between AID_ISOLATED_START (99000) |
| 6 | ### and AID_ISOLATED_END (99999). |
| 7 | ### |
| 8 | |
Alex Klyubin | f5446eb | 2017-03-23 14:27:32 -0700 | [diff] [blame] | 9 | typeattribute isolated_app coredomain; |
| 10 | |
dcashman | 3e8dbf0 | 2016-12-08 11:23:34 -0800 | [diff] [blame] | 11 | app_domain(isolated_app) |
dcashman | 2e00e63 | 2016-10-12 14:58:09 -0700 | [diff] [blame] | 12 | |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 13 | # Access already open app data files received over Binder or local socket IPC. |
Nick Kralevich | 0025220 | 2018-10-31 12:47:27 -0700 | [diff] [blame] | 14 | allow isolated_app { app_data_file privapp_data_file }:file { append read write getattr lock map }; |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 15 | |
| 16 | allow isolated_app activity_service:service_manager find; |
| 17 | allow isolated_app display_service:service_manager find; |
| 18 | allow isolated_app webviewupdate_service:service_manager find; |
| 19 | |
| 20 | # Google Breakpad (crash reporter for Chrome) relies on ptrace |
| 21 | # functionality. Without the ability to ptrace, the crash reporter |
| 22 | # tool is broken. |
| 23 | # b/20150694 |
| 24 | # https://code.google.com/p/chromium/issues/detail?id=475270 |
| 25 | allow isolated_app self:process ptrace; |
| 26 | |
| 27 | # b/32896414: Allow accessing sdcard file descriptors passed to isolated_apps |
| 28 | # by other processes. Open should never be allowed, and is blocked by |
| 29 | # neverallow rules below. |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 30 | # media_rw_data_file is included for sdcardfs, and can be removed if sdcardfs |
| 31 | # is modified to change the secontext when accessing the lower filesystem. |
Nick Kralevich | 0025220 | 2018-10-31 12:47:27 -0700 | [diff] [blame] | 32 | allow isolated_app { sdcard_type media_rw_data_file }:file { read write append getattr lock map }; |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 33 | |
| 34 | # For webviews, isolated_app processes can be forked from the webview_zygote |
| 35 | # in addition to the zygote. Allow access to resources inherited from the |
| 36 | # webview_zygote process. These rules are specialized copies of the ones in app.te. |
| 37 | # Inherit FDs from the webview_zygote. |
| 38 | allow isolated_app webview_zygote:fd use; |
| 39 | # Notify webview_zygote of child death. |
| 40 | allow isolated_app webview_zygote:process sigchld; |
| 41 | # Inherit logd write socket. |
| 42 | allow isolated_app webview_zygote:unix_dgram_socket write; |
dcashman | 2e00e63 | 2016-10-12 14:58:09 -0700 | [diff] [blame] | 43 | # Read system properties managed by webview_zygote. |
| 44 | allow isolated_app webview_zygote_tmpfs:file read; |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 45 | |
Martijn Coenen | 1bbda7e | 2018-11-05 11:39:15 +0100 | [diff] [blame] | 46 | # Inherit FDs from the app_zygote. |
| 47 | allow isolated_app app_zygote:fd use; |
| 48 | # Notify app_zygote of child death. |
| 49 | allow isolated_app app_zygote:process sigchld; |
| 50 | # Inherit logd write socket. |
| 51 | allow isolated_app app_zygote:unix_dgram_socket write; |
| 52 | |
Dan Cashman | 91d398d | 2017-09-26 12:58:29 -0700 | [diff] [blame] | 53 | # TODO (b/63631799) fix this access |
| 54 | # suppress denials to /data/local/tmp |
| 55 | dontaudit isolated_app shell_data_file:dir search; |
| 56 | |
Primiano Tucci | c80f9e0 | 2017-12-21 03:51:15 +0100 | [diff] [blame] | 57 | # Write app-specific trace data to the Perfetto traced damon. This requires |
| 58 | # connecting to its producer socket and obtaining a (per-process) tmpfs fd. |
| 59 | allow isolated_app traced:fd use; |
| 60 | allow isolated_app traced_tmpfs:file { read write getattr map }; |
| 61 | unix_socket_connect(isolated_app, traced_producer, traced) |
| 62 | |
Ryan Savitski | ca0690e | 2019-01-16 16:29:43 +0000 | [diff] [blame] | 63 | # Allow heap profiling if the main app has been marked as profileable or |
| 64 | # debuggable. |
| 65 | can_profile_heap(isolated_app) |
| 66 | |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 67 | ##### |
| 68 | ##### Neverallow |
| 69 | ##### |
| 70 | |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 71 | # Isolated apps should not directly open app data files themselves. |
Nick Kralevich | 23c9d91 | 2018-08-02 15:54:23 -0700 | [diff] [blame] | 72 | neverallow isolated_app { app_data_file privapp_data_file }:file open; |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 73 | |
| 74 | # Only allow appending to /data/anr/traces.txt (b/27853304, b/18340553) |
| 75 | # TODO: are there situations where isolated_apps write to this file? |
| 76 | # TODO: should we tighten these restrictions further? |
| 77 | neverallow isolated_app anr_data_file:file ~{ open append }; |
| 78 | neverallow isolated_app anr_data_file:dir ~search; |
| 79 | |
Dan Cashman | 91d398d | 2017-09-26 12:58:29 -0700 | [diff] [blame] | 80 | # Isolated apps must not be permitted to use HwBinder |
| 81 | neverallow isolated_app hwbinder_device:chr_file *; |
| 82 | neverallow isolated_app *:hwservice_manager *; |
| 83 | |
| 84 | # Isolated apps must not be permitted to use VndBinder |
| 85 | neverallow isolated_app vndbinder_device:chr_file *; |
| 86 | |
| 87 | # Isolated apps must not be permitted to perform actions on Binder and VndBinder service_manager |
| 88 | # except the find actions for services whitelisted below. |
| 89 | neverallow isolated_app *:service_manager ~find; |
| 90 | |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 91 | # b/17487348 |
| 92 | # Isolated apps can only access three services, |
Tri Vo | 73d0a67 | 2019-01-27 13:39:19 -0800 | [diff] [blame] | 93 | # activity_service, display_service, webviewupdate_service, and |
| 94 | # ashmem_device_service. |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 95 | neverallow isolated_app { |
| 96 | service_manager_type |
| 97 | -activity_service |
Tri Vo | 73d0a67 | 2019-01-27 13:39:19 -0800 | [diff] [blame] | 98 | -ashmem_device_service |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 99 | -display_service |
| 100 | -webviewupdate_service |
| 101 | }:service_manager find; |
| 102 | |
| 103 | # Isolated apps shouldn't be able to access the driver directly. |
| 104 | neverallow isolated_app gpu_device:chr_file { rw_file_perms execute }; |
| 105 | |
| 106 | # Do not allow isolated_app access to /cache |
| 107 | neverallow isolated_app cache_file:dir ~{ r_dir_perms }; |
| 108 | neverallow isolated_app cache_file:file ~{ read getattr }; |
| 109 | |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 110 | # Do not allow isolated_app to access external storage, except for files passed |
| 111 | # via file descriptors (b/32896414). |
| 112 | neverallow isolated_app { storage_file mnt_user_file sdcard_type }:dir ~getattr; |
| 113 | neverallow isolated_app { storage_file mnt_user_file }:file_class_set *; |
| 114 | neverallow isolated_app sdcard_type:{ devfile_class_set lnk_file sock_file fifo_file } *; |
Nick Kralevich | 0025220 | 2018-10-31 12:47:27 -0700 | [diff] [blame] | 115 | neverallow isolated_app sdcard_type:file ~{ read write append getattr lock map }; |
Alex Klyubin | a390397 | 2017-01-05 16:06:54 -0800 | [diff] [blame] | 116 | |
| 117 | # Do not allow USB access |
| 118 | neverallow isolated_app { usb_device usbaccessory_device }:chr_file *; |
| 119 | |
| 120 | # Restrict the webview_zygote control socket. |
Robert Sesek | 869562e | 2018-01-30 10:54:33 -0500 | [diff] [blame] | 121 | neverallow isolated_app webview_zygote:sock_file write; |
Nick Kralevich | 62d3b4f | 2017-10-09 15:10:30 -0700 | [diff] [blame] | 122 | |
| 123 | # Limit the /sys files which isolated_app can access. This is important |
| 124 | # for controlling isolated_app attack surface. |
| 125 | neverallow isolated_app { |
| 126 | sysfs_type |
| 127 | -sysfs_devices_system_cpu |
| 128 | -sysfs_usb # TODO: check with audio team if needed for isolated_app (b/28417852) |
| 129 | }:file no_rw_file_perms; |
Nick Kralevich | 8f08456 | 2018-10-02 21:43:20 -0700 | [diff] [blame] | 130 | |
| 131 | # No creation of sockets families other than AF_UNIX sockets. |
| 132 | # List taken from system/sepolicy/public/global_macros - socket_class_set |
| 133 | # excluding unix_stream_socket and unix_dgram_socket. |
| 134 | # Many of these are socket families which have never and will never |
| 135 | # be compiled into the Android kernel. |
| 136 | neverallow isolated_app self:{ |
| 137 | socket tcp_socket udp_socket rawip_socket netlink_socket packet_socket |
| 138 | key_socket appletalk_socket netlink_route_socket |
| 139 | netlink_tcpdiag_socket netlink_nflog_socket netlink_xfrm_socket |
| 140 | netlink_selinux_socket netlink_audit_socket netlink_dnrt_socket |
| 141 | netlink_kobject_uevent_socket tun_socket netlink_iscsi_socket |
| 142 | netlink_fib_lookup_socket netlink_connector_socket netlink_netfilter_socket |
| 143 | netlink_generic_socket netlink_scsitransport_socket netlink_rdma_socket |
| 144 | netlink_crypto_socket sctp_socket icmp_socket ax25_socket ipx_socket |
| 145 | netrom_socket atmpvc_socket x25_socket rose_socket decnet_socket atmsvc_socket |
| 146 | rds_socket irda_socket pppox_socket llc_socket can_socket tipc_socket |
| 147 | bluetooth_socket iucv_socket rxrpc_socket isdn_socket phonet_socket |
| 148 | ieee802154_socket caif_socket alg_socket nfc_socket vsock_socket kcm_socket |
Nick Kralevich | a194d37 | 2018-11-16 02:48:03 -0800 | [diff] [blame] | 149 | qipcrtr_socket smc_socket xdp_socket |
Nick Kralevich | 8f08456 | 2018-10-02 21:43:20 -0700 | [diff] [blame] | 150 | } create; |