diff --git a/private/app_neverallows.te b/private/app_neverallows.te index 5b8d93594e63aba149f0e64511ab5985fb0e0722..db14f1b585486f85aa97a47940da6b71c34c3a5b 100644 --- a/private/app_neverallows.te +++ b/private/app_neverallows.te @@ -5,12 +5,12 @@ define(`all_untrusted_apps',`{ ephemeral_app isolated_app + mediaprovider untrusted_app untrusted_app_25 untrusted_app_all untrusted_v2_app }') - # Receive or send uevent messages. neverallow all_untrusted_apps domain:netlink_kobject_uevent_socket *; @@ -32,9 +32,9 @@ neverallow all_untrusted_apps vndservice_manager_type:service_manager *; # Do not allow untrusted apps to connect to the property service # or set properties. b/10243159 -neverallow all_untrusted_apps property_socket:sock_file write; -neverallow all_untrusted_apps init:unix_stream_socket connectto; -neverallow all_untrusted_apps property_type:property_service set; +neverallow { all_untrusted_apps -mediaprovider } property_socket:sock_file write; +neverallow { all_untrusted_apps -mediaprovider } init:unix_stream_socket connectto; +neverallow { all_untrusted_apps -mediaprovider } property_type:property_service set; # Do not allow untrusted apps to be assigned mlstrustedsubject. # This would undermine the per-user isolation model being @@ -72,15 +72,15 @@ neverallow all_untrusted_apps *:{ } *; # Do not allow untrusted apps access to /cache -neverallow all_untrusted_apps { cache_file cache_recovery_file }:dir ~{ r_dir_perms }; -neverallow all_untrusted_apps { cache_file cache_recovery_file }:file ~{ read getattr }; +neverallow { all_untrusted_apps -mediaprovider } { cache_file cache_recovery_file }:dir ~{ r_dir_perms }; +neverallow { all_untrusted_apps -mediaprovider } { cache_file cache_recovery_file }:file ~{ read getattr }; # Do not allow untrusted apps to create/unlink files outside of its sandbox, # internal storage or sdcard. # World accessible data locations allow application to fill the device # with unaccounted for data. This data will not get removed during # application un-installation. -neverallow all_untrusted_apps { +neverallow { all_untrusted_apps -mediaprovider } { fs_type -fuse # sdcard -sdcardfs # sdcard diff --git a/private/mediaprovider.te b/private/mediaprovider.te new file mode 100644 index 0000000000000000000000000000000000000000..63f56c876ba516623db5277849d4e9f2565c656e --- /dev/null +++ b/private/mediaprovider.te @@ -0,0 +1,35 @@ +### +### A domain for android.process.media, which contains both +### MediaProvider and DownloadProvider and associated services. +### + +typeattribute mediaprovider coredomain; +app_domain(mediaprovider) + +# DownloadProvider accesses the network. +net_domain(mediaprovider) + +# DownloadProvider uses /cache. +allow mediaprovider cache_file:dir create_dir_perms; +allow mediaprovider cache_file:file create_file_perms; +# /cache is a symlink to /data/cache on some devices. Allow reading the link. +allow mediaprovider cache_file:lnk_file r_file_perms; + +allow mediaprovider app_api_service:service_manager find; +allow mediaprovider audioserver_service:service_manager find; +allow mediaprovider drmserver_service:service_manager find; +allow mediaprovider mediaserver_service:service_manager find; +allow mediaprovider surfaceflinger_service:service_manager find; + +# Allow MediaProvider to read/write cached ringtones (opened by system). +allow mediaprovider ringtone_file:file { getattr read write }; + +# MtpServer uses /dev/mtp_usb +allow mediaprovider mtp_device:chr_file rw_file_perms; + +# MtpServer uses /dev/usb-ffs/mtp +allow mediaprovider functionfs:dir search; +allow mediaprovider functionfs:file rw_file_perms; + +# MtpServer sets sys.usb.ffs.mtp.ready +set_prop(mediaprovider, ffs_prop) diff --git a/private/priv_app.te b/private/priv_app.te index 291832e5d62ef6540f0f8ff42760e78e5937cdd8..60fb411096bc661d8767297cca9dc7614b07146d 100644 --- a/private/priv_app.te +++ b/private/priv_app.te @@ -104,20 +104,6 @@ allow priv_app preloads_data_file:dir r_dir_perms; allow priv_app preloads_media_file:file r_file_perms; allow priv_app preloads_media_file:dir r_dir_perms; -# TODO: revert this as part of fixing 33574909 -# android.process.media uses /dev/mtp_usb -allow priv_app mtp_device:chr_file rw_file_perms; - -# TODO: revert this as part of fixing 33574909 -# MtpServer uses /dev/usb-ffs/mtp -allow priv_app functionfs:dir search; -allow priv_app functionfs:file rw_file_perms; - -# TODO: revert this as part of fixing 33574909 -# Traverse into /mnt/media_rw for bypassing FUSE daemon -# TODO: narrow this to just MediaProvider -allow priv_app mnt_media_rw_file:dir search; - # Allow privileged apps (e.g. GMS core) to generate unique hardware IDs allow priv_app keystore:keystore_key gen_unique_id; diff --git a/private/seapp_contexts b/private/seapp_contexts index 4356889b5b79d9798525c39c10785027a029a5ec..dc7e3893bf4b57438a74f6b3b7cb35e8c22ac207 100644 --- a/private/seapp_contexts +++ b/private/seapp_contexts @@ -102,6 +102,7 @@ user=radio seinfo=platform domain=radio type=radio_data_file user=shared_relro domain=shared_relro user=shell seinfo=platform domain=shell type=shell_data_file user=_isolated domain=isolated_app levelFrom=user +user=_app seinfo=media domain=mediaprovider name=android.process.media type=app_data_file levelFrom=user user=_app seinfo=platform domain=platform_app type=app_data_file levelFrom=user user=_app isV2App=true isEphemeralApp=true domain=ephemeral_app type=app_data_file levelFrom=user user=_app isV2App=true domain=untrusted_v2_app type=app_data_file levelFrom=user diff --git a/public/kernel.te b/public/kernel.te index 9a66559b96246a7b816a621ec99b52f96e5e53dc..64111b0fa889db57ce67de3f75bdf5fc89c2f915 100644 --- a/public/kernel.te +++ b/public/kernel.te @@ -51,11 +51,12 @@ allow kernel tmpfs:chr_file write; allow kernel selinuxfs:file write; allow kernel self:security setcheckreqprot; -# MTP sync (b/15835289) # kernel thread "loop0", used by the loop block device, for ASECs (b/17158723) -allow kernel priv_app:fd use; allow kernel sdcard_type:file { read write }; +# f_mtp driver accesses files from kernel context. +allow kernel mediaprovider:fd use; + # Allow the kernel to read OBB files from app directories. (b/17428116) # Kernel thread "loop0" reads a vold supplied file descriptor. # Fixes CTS tests: diff --git a/public/mediaprovider.te b/public/mediaprovider.te new file mode 100644 index 0000000000000000000000000000000000000000..24170a5cf177593ec3a6f398ba19a340210c0390 --- /dev/null +++ b/public/mediaprovider.te @@ -0,0 +1,6 @@ +### +### A domain for android.process.media, which contains both +### MediaProvider and DownloadProvider and associated services. +### + +type mediaprovider, domain;