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

Fix m4 warnings

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
parent 0fd3ed3b
No related branches found
No related tags found
No related merge requests found
......@@ -408,7 +408,7 @@ allow $1 sysfs_wake_lock:file rw_file_perms;
allow $1 self:global_capability2_class_set block_suspend;
# TODO(b/36375663): wake lock clients should be tagged with
# hal_system_suspend_client and halclientdomain attributes. However,
# typeattribute statements don't allow applying attributes to other attributes,
# typeattribute statements do not allow applying attributes to other attributes,
# so instead we grant appropriate permissions directly within this macro.
# hal_system_suspend_client permissions
binder_call($1, hal_system_suspend_server)
......
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