diff --git a/private/adbd.te b/private/adbd.te index d28b836f14ad69343f208b5191577ebbe4696874..47a6cbd8cf701a04032f8236fa3dc8f7364e43d6 100644 --- a/private/adbd.te +++ b/private/adbd.te @@ -65,9 +65,14 @@ get_prop(adbd, serialno_prop) # Run /system/bin/bu allow adbd system_file:file rx_file_perms; -# Use screencap -domain_auto_trans(adbd, screencap_exec, screencap) -allow adbd screencap:process signal; +# Perform binder IPC to surfaceflinger (screencap) +# XXX Run screencap in a separate domain? +binder_use(adbd) +binder_call(adbd, surfaceflinger) +# b/13188914 +allow adbd gpu_device:chr_file rw_file_perms; +allow adbd ion_device:chr_file rw_file_perms; +r_dir_file(adbd, system_file) # Needed for various screenshots hal_client_domain(adbd, hal_graphics_allocator) @@ -134,5 +139,5 @@ allow adbd rootfs:dir r_dir_perms; # No transitions from adbd to non-shell, non-crash_dump domains. adbd only ever # transitions to the shell domain (except when it crashes). In particular, we # never want to see a transition from adbd to su (aka "adb root") -neverallow adbd { domain -crash_dump -shell -screencap }:process transition; +neverallow adbd { domain -crash_dump -shell }:process transition; neverallow adbd { domain userdebug_or_eng(`-su') }:process dyntransition; diff --git a/private/app.te b/private/app.te index c4147234a6a15f34e7b5837ad57fe02cafc0b9ab..70b42b9bdf991a1bc4c0a816cdd7c425a338ad3b 100644 --- a/private/app.te +++ b/private/app.te @@ -411,9 +411,7 @@ neverallow { appdomain -shell } { domain -appdomain }:file no_rw_file_perms; # sigchld allowed for parent death notification. # signull allowed for kill(pid, 0) existence test. # All others prohibited. -neverallow { appdomain -shell } { domain -appdomain }:process - { sigkill sigstop signal }; -neverallow shell { domain -appdomain -screencap }:process +neverallow appdomain { domain -appdomain }:process { sigkill sigstop signal }; # Transition to a non-app domain. diff --git a/private/compat/26.0/26.0.ignore.cil b/private/compat/26.0/26.0.ignore.cil index 09f216d387c90c7935b46431547db5839de566e2..9a418de9f3138c970294134f925338a1cb678fcb 100644 --- a/private/compat/26.0/26.0.ignore.cil +++ b/private/compat/26.0/26.0.ignore.cil @@ -34,6 +34,4 @@ ;; Thus, these types are also not mapped, but recorded for checkapi tests (typeattribute priv_objects) (typeattributeset priv_objects - ( adbd_tmpfs - screencap - screencap_exec )) + ( adbd_tmpfs )) diff --git a/private/dumpstate.te b/private/dumpstate.te index 8f003aab2b7695d8d56600665a288a961d4b1353..b8f81526cddc1f713b42a4f2e83c97882c1d82dd 100644 --- a/private/dumpstate.te +++ b/private/dumpstate.te @@ -23,7 +23,3 @@ binder_call(dumpstate, storaged) # Collect metrics on boot time created by init get_prop(dumpstate, boottime_prop) - -# Use screencap -domain_auto_trans(dumpstate, screencap_exec, screencap) -allow dumpstate screencap:process signal; diff --git a/private/file_contexts b/private/file_contexts index 6447bc786a8843b7934b7a1bdfd7e0659451972c..ed51482c65bd6505bd0e5ce7c3e682e853cfd8f7 100644 --- a/private/file_contexts +++ b/private/file_contexts @@ -212,7 +212,6 @@ /system/bin/mediametrics u:object_r:mediametrics_exec:s0 /system/bin/cameraserver u:object_r:cameraserver_exec:s0 /system/bin/mediaextractor u:object_r:mediaextractor_exec:s0 -/system/bin/screencap u:object_r:screencap_exec:s0 /system/bin/mdnsd u:object_r:mdnsd_exec:s0 /system/bin/installd u:object_r:installd_exec:s0 /system/bin/otapreopt_chroot u:object_r:otapreopt_chroot_exec:s0 diff --git a/private/screencap.te b/private/screencap.te deleted file mode 100644 index 579373aa6a3b41a20edfa24d477c3723bbcf9c9d..0000000000000000000000000000000000000000 --- a/private/screencap.te +++ /dev/null @@ -1,26 +0,0 @@ -type screencap, domain; -type screencap_exec, exec_type, file_type; - -typeattribute screencap coredomain; - -allow screencap gpu_device:chr_file rw_file_perms; -allow screencap ion_device:chr_file rw_file_perms; - -allow screencap adbd:fifo_file write; -allow screencap adbd:fd use; -allow screencap adbd:unix_stream_socket { read write }; - -allow screencap shell_data_file:file write; -allow screencap shell:fd use; -allow screencap shell:unix_stream_socket { read write }; - -allow screencap dumpstate:fd use; -allow screencap dumpstate:unix_stream_socket { read write }; - -binder_use(screencap) -binder_call(screencap, surfaceflinger) -allow screencap surfaceflinger_service:service_manager find; -allow screencap surfaceflinger:fd use; - -hwbinder_use(screencap) -hal_client_domain(screencap, hal_graphics_allocator) diff --git a/private/shell.te b/private/shell.te index 095dc43910c91bf5553f5a311c435a18b20ec76c..5299532ac64e15c88232b8a37da11609c585d578 100644 --- a/private/shell.te +++ b/private/shell.te @@ -26,7 +26,3 @@ binder_call(shell, storaged) # Perform SELinux access checks, needed for CTS selinux_check_access(shell) selinux_check_context(shell) - -# Use screencap -domain_auto_trans(shell, screencap_exec, screencap) -allow shell screencap:process signal; diff --git a/private/surfaceflinger.te b/private/surfaceflinger.te index 7184fbcc729a00002f5113a3f7cc85cdf60c951d..b33035e8ea9d51ec1422b6105c9acba6e1330e7c 100644 --- a/private/surfaceflinger.te +++ b/private/surfaceflinger.te @@ -21,7 +21,6 @@ binder_use(surfaceflinger) binder_call(surfaceflinger, binderservicedomain) binder_call(surfaceflinger, appdomain) binder_call(surfaceflinger, bootanim) -binder_call(surfaceflinger, screencap) binder_service(surfaceflinger) # Binder IPC to bu, presently runs in adbd domain. diff --git a/public/dumpstate.te b/public/dumpstate.te index bcbc6f52481f04e3a0077f8079be9be8591be33f..3d80495189f54f25d29b2e93400932ca91b40d04 100644 --- a/public/dumpstate.te +++ b/public/dumpstate.te @@ -142,6 +142,9 @@ allow dumpstate bluetooth_data_file:dir search; allow dumpstate bluetooth_logs_data_file:dir r_dir_perms; allow dumpstate bluetooth_logs_data_file:file r_file_perms; +# Dumpstate calls screencap, which grabs a screenshot. Needs gpu access +allow dumpstate gpu_device:chr_file rw_file_perms; + # logd access read_logd(dumpstate) control_logd(dumpstate)