Skip to content
Snippets Groups Projects
Commit cfe1baea authored by Nick Kralevich's avatar Nick Kralevich
Browse files

place dex2oat auditallow statements in userdebug_or_eng blocks

By convention, auditallow statements are always placed in
userdebug_or_eng() blocks. This ensures that we don't inadvertently ship
audit rules on production devices, which could result in device logspam,
and in pathological situations, impact device performance (generating
audit messages is much more expensive than a standard SELinux check).

Bug: 117606664
Test: policy compiles.
Change-Id: I681ed73c83683e8fdbef9cf662488115f6e7a490
parent c7be91d5
No related branches found
No related tags found
No related merge requests found
...@@ -53,5 +53,4 @@ allow untrusted_app_25 app_data_file:file execute_no_trans; ...@@ -53,5 +53,4 @@ allow untrusted_app_25 app_data_file:file execute_no_trans;
# The ability to invoke dex2oat. Historically required by ART, now only # The ability to invoke dex2oat. Historically required by ART, now only
# allowed for targetApi<=28 for compat reasons. # allowed for targetApi<=28 for compat reasons.
allow untrusted_app_25 dex2oat_exec:file rx_file_perms; allow untrusted_app_25 dex2oat_exec:file rx_file_perms;
auditallow untrusted_app_25 dex2oat_exec:file rx_file_perms; userdebug_or_eng(`auditallow untrusted_app_25 dex2oat_exec:file rx_file_perms;')
...@@ -34,5 +34,4 @@ allow untrusted_app_27 app_data_file:file execute_no_trans; ...@@ -34,5 +34,4 @@ allow untrusted_app_27 app_data_file:file execute_no_trans;
# The ability to invoke dex2oat. Historically required by ART, now only # The ability to invoke dex2oat. Historically required by ART, now only
# allowed for targetApi<=28 for compat reasons. # allowed for targetApi<=28 for compat reasons.
allow untrusted_app_27 dex2oat_exec:file rx_file_perms; allow untrusted_app_27 dex2oat_exec:file rx_file_perms;
auditallow untrusted_app_27 dex2oat_exec:file rx_file_perms; userdebug_or_eng(`auditallow untrusted_app_27 dex2oat_exec:file rx_file_perms;')
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