diff options
| author | Tom Cherry <tomcherry@google.com> | 2018-01-18 16:14:25 -0800 |
|---|---|---|
| committer | Tom Cherry <tomcherry@google.com> | 2018-01-22 18:20:56 +0000 |
| commit | 32228485ffac6ff0b674210be448b121bbd6427c (patch) | |
| tree | 5091f252b64d206d32747aa66aacbd15a763f6ce /init/builtins.cpp | |
| parent | ad939afaf437f910dd3c4869c2683ca797a44f7c (diff) | |
Make vendor_init check SELinux before setting properties
Finishing a TODO from vendor_init, check SELinux permissions before
setting properties in vendor_init.
Bug: 62875318
Test: N/A
Change-Id: I3cb6abadd2613ae083705cc6b9c970587b6c6b19
Diffstat (limited to 'init/builtins.cpp')
| -rw-r--r-- | init/builtins.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/init/builtins.cpp b/init/builtins.cpp index f58402166..413d11eb0 100644 --- a/init/builtins.cpp +++ b/init/builtins.cpp @@ -1039,9 +1039,7 @@ const BuiltinFunctionMap::Map& BuiltinFunctionMap::map() const { {"restorecon_recursive", {1, kMax, {true, do_restorecon_recursive}}}, {"rm", {1, 1, {true, do_rm}}}, {"rmdir", {1, 1, {true, do_rmdir}}}, - // TODO: setprop should be run in the subcontext, but property service needs to be split - // out from init before that is possible. - {"setprop", {2, 2, {false, do_setprop}}}, + {"setprop", {2, 2, {true, do_setprop}}}, {"setrlimit", {3, 3, {false, do_setrlimit}}}, {"start", {1, 1, {false, do_start}}}, {"stop", {1, 1, {false, do_stop}}}, |