Skip to content
Snippets Groups Projects
Commit 03a3d246 authored by Stephen Smalley's avatar Stephen Smalley Committed by Android Git Automerger
Browse files

am a07db539: am 0e8f0230: am 7d715164: Do not allow isolated_app to directly open app data files.

* commit 'a07db539':
  Do not allow isolated_app to directly open app data files.
parents 1ae6e949 a07db539
No related branches found
No related tags found
No related merge requests found
...@@ -46,8 +46,8 @@ allow appdomain appdomain:fifo_file rw_file_perms; ...@@ -46,8 +46,8 @@ allow appdomain appdomain:fifo_file rw_file_perms;
allow appdomain surfaceflinger:unix_stream_socket { read write setopt getattr getopt shutdown }; allow appdomain surfaceflinger:unix_stream_socket { read write setopt getattr getopt shutdown };
# App sandbox file accesses. # App sandbox file accesses.
allow appdomain app_data_file:dir create_dir_perms; allow { appdomain -isolated_app } app_data_file:dir create_dir_perms;
allow appdomain app_data_file:notdevfile_class_set create_file_perms; allow { appdomain -isolated_app } app_data_file:notdevfile_class_set create_file_perms;
# lib subdirectory of /data/data dir is system-owned. # lib subdirectory of /data/data dir is system-owned.
allow appdomain system_data_file:dir r_dir_perms; allow appdomain system_data_file:dir r_dir_perms;
......
...@@ -12,6 +12,12 @@ ...@@ -12,6 +12,12 @@
type isolated_app, domain; type isolated_app, domain;
app_domain(isolated_app) app_domain(isolated_app)
# Access already open app data files received over Binder or local socket IPC.
allow isolated_app app_data_file:file { read write getattr };
# Isolated apps should not directly open app data files themselves.
neverallow isolated_app app_data_file:file open;
# Isolated apps shouldn't be able to access the driver directly. # Isolated apps shouldn't be able to access the driver directly.
neverallow isolated_app gpu_device:file { rw_file_perms execute }; neverallow isolated_app gpu_device:file { rw_file_perms execute };
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment