- Sep 04, 2018
-
-
Jeff Vander Stoep authored
DropboxManager may pass FDs to any app with the READ_LOGS permission which is available to all apps as a development permission. Test: atest CtsIncidentHostTestCases Fixes: 111856304 Change-Id: I329e3125dab83de948b860061df9d232e31cb23e
-
Mark Salyzyn authored
llkd needs the ptrace capabilities and dac override to monitor for live lock conditions on the stack dumps. Test: compile Bug: 33808187 Change-Id: Ibc1e4cc10395fa9685c4ef0ca214daf212a5e126
-
- Sep 03, 2018
-
-
Alan Stokes authored
This is not needed and could conceivably be abused. Test: Builds. Bug: 110107376 Change-Id: I73f301439af435fe40b3902409964cdf6e2c7dd5
-
- Aug 31, 2018
-
-
Chih-Hung Hsieh authored
Test: make with WITH_TIDY=1 and clang-analyzer-* checks. Change-Id: Ide1eaf8880132c566545710e6287f66a5a2b393c
-
- Aug 30, 2018
-
-
Kevin Chyn authored
Bug: 72825012 Test: manual Change-Id: I850c869cdc0ad8735800130bb4a8d67822197ff9
-
- Aug 29, 2018
-
-
Treehugger Robot authored
-
- Aug 28, 2018
-
-
Treehugger Robot authored
-
Mark Salyzyn authored
Test: compile Bug: 64114943 Change-Id: I1d20cc027dbd1a94e2a79b6aebdd265cefe8a6a5
-
Nick Kralevich authored
Shell access to existing input devices is an abuse vector. The shell user can inject events that look like they originate from the touchscreen etc. Everyone should have already moved to UiAutomation#injectInputEvent if they are running instrumentation tests (i.e. CTS), Monkey for their stress tests, and the input command (adb shell input ...) for injecting swipes and things. Remove the write ability for shell users, and add a neverallow assertion (which is also a CTS test) to prevent regressions. Bug: 30861057 Test: auditallow statement added in f617a404 hasn't triggered. Test: ran getevent, saw correct output, played with device Change-Id: Ia78eeec05f6015478dd32bd59505b51fef200a99
-
Jeff Vander Stoep authored
Remove permissions and add neverallow assertion. (cherry picked from commit f1554f15) Bug: 110107376 Test: kill -6 <components excluded from ptrace> Change-Id: I2dc872f5c02749fbaf8ca6bc7e3e38404151442c
-
- Aug 27, 2018
-
-
Treehugger Robot authored
-
- Aug 25, 2018
-
-
Howard Ro authored
-
- Aug 24, 2018
-
-
Nick Kralevich authored
Test to see if anyone is writing to /dev/input from the shell. Bug: 30861057 Test: device boots and no avc granted messages. Change-Id: Ia3499ef9436f83cf13c633525348b63edd95990f
-
Treehugger Robot authored
-
- Aug 23, 2018
- Aug 22, 2018
-
-
Christine Franks authored
Bug: 111215474 Test: boots Change-Id: I98955bcd02f643400c3eb97232467c09a2c5c1e5
-
- Aug 21, 2018
-
-
Tri Vo authored
Bug: 110887137 Test: Flash new system policy onto a device with vendor policy that uses untrusted_app_visible_* attributes, and check that old and new attributes are applied to exactly same types. Change-Id: Ibee0ec645878fcc8c93cd0fbd169a8d45129d79e Merged-In: Ibee0ec645878fcc8c93cd0fbd169a8d45129d79e (cherry picked from commit 7abca51d)
-
Benjamin Gordon authored
commit 9b2e0cbe added a new self:global_capability_class_set macro that covers both self:capability and self:cap_userns. Apply the new macro to various self:capability references that have cropped up since then. Bug: 112307595 Test: policy diff shows new rules are all cap_userns Change-Id: I3eb38ef07532a8e693fd549dfdbc4a6df5329609
-
Yifan Hong authored
-
- Aug 20, 2018
-
-
Treehugger Robot authored
-
Hridya Valsaraju authored
Bug: 78793464 Test: fastboot getvar partition-size:super 'super_block_device' corresponds to the super partition required for flashing dynamic partitions. Change-Id: I323634b6797ead7c5face117a7028bf9ab947aea
-
Dario Freni authored
Attempting to reduce the number of different spellings we have for "product services" partition in the codebase. Bug: 112431447 Test: m Change-Id: I1499c60e3d6c6c9fbe2e3f30f097f83b1e837c1c Merged-In: I1499c60e3d6c6c9fbe2e3f30f097f83b1e837c1c
-
- Aug 17, 2018
-
-
Zheng Zhang authored
mediaserver is receiving a file passed as a file descriptor. Just read and map is enough, and open should not be allowed for mediaserver. Bug: 78436043
-
- Aug 15, 2018
-
-
Treehugger Robot authored
-
Treehugger Robot authored
-
Joel Galenson authored
avc: denied { getattr } for comm="Binder:1231_1" path="/storage/emulated" dev="tmpfs" ino=72787 scontext=u:r:system_server:s0 tcontext=u:object_r:storage_stub_file:s0 tclass=dir Bug: 112609936 Test: Built policy. Change-Id: Ib772aca11bad4ba267de259be4ad065f228ef1df
-
Nick Kralevich authored
Error out if m4 generates a warning. This will help detect and prevent malformed macros. See 85508496 for motivation. Test: policy compiles Test: Policy doesn't compile if 85508496 is reverted. Change-Id: Iee6b6273bc2a24b1220861fd662573e76001defc
-
Jerry Zhang authored
Also allow adb and fastboot to talk to recovery through recovery_socket. This enables changing between modes with usb commands. Test: No selinux denials Bug: 78793464 Change-Id: I80c54d4eaf3b94a1fe26d2280af4e57cb1593790
-
Florian Mayer authored
This reverts commit 0fd3ed3b. Reason for revert: Broke user builds. Change-Id: If95f1a25d22425a5a2b68a02d1561352fb5a52f0
-
- Aug 14, 2018
-
-
Steven Moreland authored
Forgotten cleanup item. Bug: 35870313 Test: making sepolicy (neverallows resolved at compile time) Change-Id: If9a583c4508db63356869502ec374727afa84b0b
-
Nick Kralevich authored
Quotes and backticks are sensitive characters and should never show up in a comment. Fix comment to avoid the use of a single quote. Also fixes a bug where certain rules were not getting included in the compiled policy. Fixes the following build warnings: [ 3% 3564/114975] build out/target/product/taimen/obj/ETC/sepolicy_neverallows_intermediates/plat_pub_policy.conf m4:system/sepolicy/public/te_macros:404: Warning: excess arguments to builtin `define' ignored [ 3% 3578/114975] build out/target/product/taimen/obj/ETC/plat_sepolicy.cil_intermediates/plat_policy.conf m4:system/sepolicy/public/te_macros:404: Warning: excess arguments to builtin `define' ignored [ 3% 3579/114975] build out/target/product/taimen/obj/ETC/vendor_sepolicy.cil_intermediates/vendor_policy.conf m4:system/sepolicy/public/te_macros:404: Warning: excess arguments to builtin `define' ignored [ 3% 3607/114975] build out/target/product/taimen/obj/ETC/sepolicy_neverallows_intermediates/policy.conf m4:system/sepolicy/public/te_macros:404: Warning: excess arguments to builtin `define' ignored [ 3% 3677/114975] build out/target/product/taimen/obj/ETC/built_plat_sepolicy_intermediates/base_plat_policy.conf m4:system/sepolicy/public/te_macros:404: Warning: excess arguments to builtin `define' ignored Test: policy compiles and no warnings. Change-Id: Ie32d8b536955b40888b79e3a93851d2ae297f8ee
-
Jerry Zhang authored
Also allow adb and fastboot to talk to recovery through recovery_socket. This enables changing between modes with usb commands. Test: No selinux denials Bug: 78793464 Change-Id: I1f97659736429fe961319c642f458c80f199ffb4
-
Mark Salyzyn authored
Replace more complicated logic that determines that persistent properties are now valid with a simple check of ro.persistent_properties.ready. Test: manual Bug: 109821005 Change-Id: I8c63beb294377ea9ce6eb6336b83f529deedd830
-
Tri Vo authored
Bug: 78888165 Test: device can boot with HAL running. Change-Id: I3bf7c8203e038b892176c97ec006152a2904c7be
-
Tri Vo authored
-
- Aug 13, 2018
-
-
Yifan Hong authored
-
Treehugger Robot authored
-
Treehugger Robot authored
-
Treehugger Robot authored
-