Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 1 | ### |
| 2 | ### neverallow rules for untrusted app domains |
| 3 | ### |
| 4 | |
Jeff Vander Stoep | bacb6d7 | 2017-02-13 13:33:27 -0800 | [diff] [blame] | 5 | # Only allow domains in AOSP to use the untrusted_app_all attribute. |
| 6 | neverallow { untrusted_app_all -untrusted_app -untrusted_app_25 } domain:process fork; |
| 7 | |
Nick Kralevich | db5962c | 2017-04-26 16:14:40 -0700 | [diff] [blame^] | 8 | define(`all_untrusted_apps',`{ |
| 9 | ephemeral_app |
| 10 | isolated_app |
| 11 | mediaprovider |
| 12 | untrusted_app |
| 13 | untrusted_app_25 |
| 14 | untrusted_app_all |
| 15 | untrusted_v2_app |
| 16 | }') |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 17 | # Receive or send uevent messages. |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 18 | neverallow all_untrusted_apps domain:netlink_kobject_uevent_socket *; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 19 | |
| 20 | # Receive or send generic netlink messages |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 21 | neverallow all_untrusted_apps domain:netlink_socket *; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 22 | |
| 23 | # Too much leaky information in debugfs. It's a security |
| 24 | # best practice to ensure these files aren't readable. |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 25 | neverallow all_untrusted_apps debugfs_type:file read; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 26 | |
| 27 | # Do not allow untrusted apps to register services. |
| 28 | # Only trusted components of Android should be registering |
| 29 | # services. |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 30 | neverallow all_untrusted_apps service_manager_type:service_manager add; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 31 | |
| 32 | # Do not allow untrusted apps to connect to the property service |
| 33 | # or set properties. b/10243159 |
Jerry Zhang | 9f152d9 | 2017-04-10 16:57:48 -0700 | [diff] [blame] | 34 | neverallow { all_untrusted_apps -mediaprovider } property_socket:sock_file write; |
| 35 | neverallow { all_untrusted_apps -mediaprovider } init:unix_stream_socket connectto; |
| 36 | neverallow { all_untrusted_apps -mediaprovider } property_type:property_service set; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 37 | |
| 38 | # Do not allow untrusted apps to be assigned mlstrustedsubject. |
| 39 | # This would undermine the per-user isolation model being |
| 40 | # enforced via levelFrom=user in seapp_contexts and the mls |
| 41 | # constraints. As there is no direct way to specify a neverallow |
| 42 | # on attribute assignment, this relies on the fact that fork |
| 43 | # permission only makes sense within a domain (hence should |
| 44 | # never be granted to any other domain within mlstrustedsubject) |
| 45 | # and an untrusted app is allowed fork permission to itself. |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 46 | neverallow all_untrusted_apps mlstrustedsubject:process fork; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 47 | |
| 48 | # Do not allow untrusted apps to hard link to any files. |
| 49 | # In particular, if an untrusted app links to other app data |
| 50 | # files, installd will not be able to guarantee the deletion |
| 51 | # of the linked to file. Hard links also contribute to security |
| 52 | # bugs, so we want to ensure untrusted apps never have this |
| 53 | # capability. |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 54 | neverallow all_untrusted_apps file_type:file link; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 55 | |
| 56 | # Do not allow untrusted apps to access network MAC address file |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 57 | neverallow all_untrusted_apps sysfs_mac_address:file no_rw_file_perms; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 58 | |
| 59 | # Restrict socket ioctls. Either 1. disallow privileged ioctls, 2. disallow the |
| 60 | # ioctl permission, or 3. disallow the socket class. |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 61 | neverallowxperm all_untrusted_apps domain:{ rawip_socket tcp_socket udp_socket } ioctl priv_sock_ioctls; |
| 62 | neverallow all_untrusted_apps *:{ netlink_route_socket netlink_selinux_socket } ioctl; |
| 63 | neverallow all_untrusted_apps *:{ |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 64 | socket netlink_socket packet_socket key_socket appletalk_socket |
Stephen Smalley | 4921085 | 2017-02-06 14:14:58 -0500 | [diff] [blame] | 65 | netlink_tcpdiag_socket netlink_nflog_socket |
| 66 | netlink_xfrm_socket netlink_audit_socket |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 67 | netlink_dnrt_socket netlink_kobject_uevent_socket tun_socket |
| 68 | netlink_iscsi_socket netlink_fib_lookup_socket netlink_connector_socket |
| 69 | netlink_netfilter_socket netlink_generic_socket netlink_scsitransport_socket |
| 70 | netlink_rdma_socket netlink_crypto_socket |
| 71 | } *; |
| 72 | |
| 73 | # Do not allow untrusted apps access to /cache |
Jerry Zhang | 9f152d9 | 2017-04-10 16:57:48 -0700 | [diff] [blame] | 74 | neverallow { all_untrusted_apps -mediaprovider } { cache_file cache_recovery_file }:dir ~{ r_dir_perms }; |
| 75 | neverallow { all_untrusted_apps -mediaprovider } { cache_file cache_recovery_file }:file ~{ read getattr }; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 76 | |
| 77 | # Do not allow untrusted apps to create/unlink files outside of its sandbox, |
| 78 | # internal storage or sdcard. |
| 79 | # World accessible data locations allow application to fill the device |
| 80 | # with unaccounted for data. This data will not get removed during |
| 81 | # application un-installation. |
Jerry Zhang | 9f152d9 | 2017-04-10 16:57:48 -0700 | [diff] [blame] | 82 | neverallow { all_untrusted_apps -mediaprovider } { |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 83 | fs_type |
| 84 | -fuse # sdcard |
| 85 | -sdcardfs # sdcard |
| 86 | -vfat |
| 87 | file_type |
| 88 | -app_data_file # The apps sandbox itself |
| 89 | -media_rw_data_file # Internal storage. Known that apps can |
| 90 | # leave artfacts here after uninstall. |
| 91 | -user_profile_data_file # Access to profile files |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 92 | userdebug_or_eng(` |
| 93 | -method_trace_data_file # only on ro.debuggable=1 |
| 94 | -coredump_file # userdebug/eng only |
| 95 | ') |
| 96 | }:dir_file_class_set { create unlink }; |
| 97 | |
| 98 | # Do not allow untrusted apps to directly open tun_device |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 99 | neverallow all_untrusted_apps tun_device:chr_file open; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 100 | |
| 101 | # Only allow appending to /data/anr/traces.txt (b/27853304, b/18340553) |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 102 | neverallow all_untrusted_apps anr_data_file:file ~{ open append }; |
| 103 | neverallow all_untrusted_apps anr_data_file:dir ~search; |
Chad Brubaker | 46e5a06 | 2017-01-27 15:53:38 -0800 | [diff] [blame] | 104 | |
| 105 | # Avoid reads from generically labeled /proc files |
| 106 | # Create a more specific label if needed |
Chad Brubaker | a782a81 | 2017-02-06 10:31:45 -0800 | [diff] [blame] | 107 | neverallow all_untrusted_apps proc:file { no_rw_file_perms no_x_file_perms }; |
Fyodor Kupolov | b238fe6 | 2017-03-14 11:42:03 -0700 | [diff] [blame] | 108 | |
| 109 | # Do not allow untrusted apps access to preloads data files |
| 110 | neverallow all_untrusted_apps preloads_data_file:file no_rw_file_perms; |
Nick Kralevich | 92c44a5 | 2017-03-22 10:35:24 -0700 | [diff] [blame] | 111 | |
| 112 | # Locking of files on /system could lead to denial of service attacks |
| 113 | # against privileged system components |
| 114 | neverallow all_untrusted_apps system_file:file lock; |