From d3a11613c35520146c28fc391c3b10af5a598dc8 Mon Sep 17 00:00:00 2001
From: Jin Qian <jinqian@google.com>
Date: Wed, 22 Feb 2017 17:27:57 -0800
Subject: [PATCH] storaged: remove rules no longer necessary
Test: adb shell dumpsys storaged --force
Bug: 35323867
Change-Id: I6944ca357875a24465054d3891a00dbcd67495cf
---
private/storaged.te | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/private/storaged.te b/private/storaged.te
index 134bec468..4796675ac 100644
--- a/private/storaged.te
+++ b/private/storaged.te
@@ -15,8 +15,6 @@ allow storaged proc_uid_io_stats:file r_file_perms;
# Read /data/system/packages.list
allow storaged system_data_file:file r_file_perms;
-allow storaged self:capability { setgid setuid sys_nice sys_ptrace };
-
userdebug_or_eng(`
# Read access to debugfs
allow storaged debugfs_mmc:dir search;
@@ -33,7 +31,12 @@ binder_call(storaged, system_server)
allow storaged batteryproperties_service:service_manager find;
binder_call(storaged, healthd)
+# Kernel does extra check on CAP_DAC_OVERRIDE for libbinder when storaged is
+# running as root. See b/35323867 #3.
+dontaudit storaged self:capability dac_override;
+
###
### neverallow
###
neverallow storaged domain:process ptrace;
+neverallow storaged self:capability_class_set *;
--
GitLab