diff options
| author | 2023-09-13 16:35:08 +0000 | |
|---|---|---|
| committer | 2023-09-25 18:48:23 +0100 | |
| commit | 2a8c60a917de3d48b1e912cd40a03b64ebed18f6 (patch) | |
| tree | 1a7927f27b2e529f391511f15218712dba140a3f | |
| parent | 03021851c25b5651330ac256cf200656c156c173 (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.java | 4 |
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) { |