tequilaOS/platform_system_sepolicy
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

sepolicy: Allow apps to read ashmem fds from system_server

Browse source 
Kernel commit 8a2af06415ef0fc922162503dd18da0d9be7771f (ashmem: switch
to ->read_iter) switched ashmem from using __vfs_read to vfs_iter_read
to read the backing shmem file.  Prior to this, reading from an ashmem
fd that was passed between processes didn't hit any permission checks;
now SELinux checks that the receiver can read from the creator's file
context.

Some apps receive buffers through ashmem from system_server, e.g., the
settings app reads battery stats from system_server through ashmem when
an app details page is opened.  Restore this ability by giving apps read
access to system_server_tmpfs.  system_server is still responsible for
creating and passing across the ashmem buffers, so this doesn't give
apps the ability to read anything system_server isn't willing to give
them.

Bug: 112987536
Bug: 111381531
Test: atest android.appsecurity.cts.PermissionsHostTest on kernel 4.14
Change-Id: Ice5e25f55bc409e91ad7e8c7ea8b28ae213191a3
This commit is contained in:
Benjamin Gordon 2018-09-06 14:41:34 -06:00 committed by Nick Kralevich
parent 8d7d5b42b5
commit 360559e7bb
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
private/app.te
View file

@ -2,6 +2,10 @@
# Read system properties managed by zygote. # Read system properties managed by zygote.
allow appdomain zygote_tmpfs:file read; allow appdomain zygote_tmpfs:file read;
# Read from (but not create) system_server buffers transferred through
# ashmem, e.g. battery stats.
allow appdomain system_server_tmpfs:file read;
neverallow appdomain system_server:udp_socket { neverallow appdomain system_server:udp_socket {
accept append bind create ioctl listen lock name_bind accept append bind create ioctl listen lock name_bind
relabelfrom relabelto setattr shutdown }; relabelfrom relabelto setattr shutdown };