summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Iván Budnik <ivanbuper@google.com> 2023-09-13 16:35:08 +0000
committer Iván Budnik <ivanbuper@google.com> 2023-09-25 18:48:23 +0100
commit2a8c60a917de3d48b1e912cd40a03b64ebed18f6 (patch)
tree1a7927f27b2e529f391511f15218712dba140a3f
parent03021851c25b5651330ac256cf200656c156c173 (diff)
Fix inconsistent system routing permission check
MR2ServiceImpl was not checking for BT permissions when granting access to system routing to routing clients on some cases. Bug: 298338671 Test: MediaRouter2Test MediaRouter2HostSideTest Change-Id: Iac9986bc37c78c49b6feb3b31e47cff2702a9c2c
-rw-r--r--services/core/java/com/android/server/media/MediaRouter2ServiceImpl.java4
1 files changed, 3 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/media/MediaRouter2ServiceImpl.java b/services/core/java/com/android/server/media/MediaRouter2ServiceImpl.java
index 4892c22449cb..a3f63545448b 100644
--- a/services/core/java/com/android/server/media/MediaRouter2ServiceImpl.java
+++ b/services/core/java/com/android/server/media/MediaRouter2ServiceImpl.java
@@ -22,6 +22,7 @@ import static android.content.Intent.ACTION_SCREEN_ON;
import static android.media.MediaRoute2ProviderService.REASON_UNKNOWN_ERROR;
import static android.media.MediaRouter2Utils.getOriginalId;
import static android.media.MediaRouter2Utils.getProviderId;
+
import static com.android.internal.util.function.pooled.PooledLambda.obtainMessage;
import static com.android.server.media.MediaFeatureFlagManager.FEATURE_SCANNING_MINIMUM_PACKAGE_IMPORTANCE;
@@ -705,7 +706,8 @@ class MediaRouter2ServiceImpl {
}
private boolean checkCallerHasSystemRoutingPermissions(int pid, int uid) {
- return checkCallerHasModifyAudioRoutingPermission(pid, uid);
+ return checkCallerHasModifyAudioRoutingPermission(pid, uid)
+ || checkCallerHasBluetoothPermissions(pid, uid);
}
private boolean checkCallerHasModifyAudioRoutingPermission(int pid, int uid) {