diff options
| author | Alex Buynytskyy <alexbuy@google.com> | 2021-01-12 11:10:03 -0800 |
|---|---|---|
| committer | Alex Buynytskyy <alexbuy@google.com> | 2021-01-12 12:35:53 -0800 |
| commit | 42d4ba418e97c6e34fba7b1053d69e92ec32b846 (patch) | |
| tree | 51329c4aa638a50db0adaa57690c49310041b931 /services/incremental/IncrementalServiceValidation.cpp | |
| parent | daea39f8a21c1900f45e652df2fa10ffe3e3d5bf (diff) | |
Require INTERACT_ACROSS_USERS to enable readlogs.
As requested by PWG.
Bug: 177330562
Fixes: 177330562
Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest
Change-Id: I1b03bc1c5b4c3cffaa0f97d864cc088782290bc2
Diffstat (limited to 'services/incremental/IncrementalServiceValidation.cpp')
| -rw-r--r-- | services/incremental/IncrementalServiceValidation.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/services/incremental/IncrementalServiceValidation.cpp b/services/incremental/IncrementalServiceValidation.cpp index abadbbf10742..9f2639a81666 100644 --- a/services/incremental/IncrementalServiceValidation.cpp +++ b/services/incremental/IncrementalServiceValidation.cpp @@ -56,13 +56,18 @@ binder::Status CheckPermissionForDataDelivery(const char* permission, const char String16 packageName{package}; - // Caller must also have op granted. PermissionController pc; if (auto packageUid = pc.getPackageUid(packageName, 0); packageUid != uid) { return Exception(binder::Status::EX_SECURITY, StringPrintf("UID %d / PID %d does not own package %s", uid, pid, package)); } + + if (!operation) { + return binder::Status::ok(); + } + + // Caller must also have op granted. switch (auto result = pc.noteOp(String16(operation), uid, packageName); result) { case PermissionController::MODE_ALLOWED: case PermissionController::MODE_DEFAULT: |