diff options
| author | Hai Zhang <zhanghai@google.com> | 2020-11-12 15:37:00 -0800 |
|---|---|---|
| committer | Hai Zhang <zhanghai@google.com> | 2020-11-17 12:04:59 -0800 |
| commit | 888c718befdb7f2c1a4f9b235161059d643e8e99 (patch) | |
| tree | 96f1694c9d556735c4d1854c557dddfae3e9bbd8 /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | 15b8613a4a033033aae716da742942850e6e3f68 (diff) | |
Move enforceCrossUser*Permission() back into PackageManagerService.
The three methods aren't part of permission management, but instead
they are only called by PackageManagerService as a utility method
containing a specific series of checks it needs for its own business
logic. So these shouldn't be permission APIs and thus moving them
back.
This leaves a simplified copy of enforceCrossUserPermission() inside
PermissionManagerService for its own usage, and it seems fine because
different system services have different needs for cross user
permission checking (See cs/enforceCrossUserPermission). If we ever
want to have a unified method for cross user permission checking, it
should be on a higher level component, e.g. Context.
Bug: 158736025
Test: presubmit
Change-Id: Ic1ebc5d83c8c9e854bd6fbff7dbbb7f2e07d613d
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions