diff options
| author | 2024-02-28 13:06:39 -0800 | |
|---|---|---|
| committer | 2024-02-29 05:08:48 +0000 | |
| commit | 2fbe49ca5cd77f21c5cd60d40115f3f1e50440fe (patch) | |
| tree | b3c8011c71721d70537527698aa9a901138dfd60 | |
| parent | 332b2ea40e19083bbefffdabe915656ecf041632 (diff) | |
[ECM] Fix boolean typo
This expression should be inverted.
The intention was "if the app's installer is allowlisted then trust the
app", but this method returns a value indicating whether to restrict the
app, so it needs to be inverted.
LOW_COVERAGE_REASON=327533775
Bug: 327469699
Test: atest CtsPermissionUiTestCases:android.permissionui.cts.EnhancedConfirmationManagerTest
Change-Id: Ia783d7ce99700fd228fd224186a903ab2a072572
| -rw-r--r-- | service/java/com/android/ecm/EnhancedConfirmationService.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/service/java/com/android/ecm/EnhancedConfirmationService.java b/service/java/com/android/ecm/EnhancedConfirmationService.java index d91e04dfa..b245aa481 100644 --- a/service/java/com/android/ecm/EnhancedConfirmationService.java +++ b/service/java/com/android/ecm/EnhancedConfirmationService.java @@ -260,7 +260,7 @@ public class EnhancedConfirmationService extends SystemService { // ECM doesn't consider a transitive chain of trust for install sources. // If this package hasn't been explicitly handled by this point // then it is exempt from ECM if the immediate parent is a trusted installer - return isAllowlistedInstaller(installSource.getInstallingPackageName()); + return !isAllowlistedInstaller(installSource.getInstallingPackageName()); } private boolean isAllowlistedPackage(String packageName) { |