Fix conference log history for work profiles.
The calling user in ConnectionServiceWrapper#addExistingConnection is
always user 0, even in the context of work calls. This is problematic as
we leverage the calling user to retrieve all the phone accounts that are
visible for that user. In order to support the notion of multi-user
awareness, we can validate that the calling user has granted the
INTERACT_ACROSS_USERS permission in order to allow access to all phone
accounts.
Fixes: 279974597
Test: Manual (Can see conference logs in telecom dumpsys)
Change-Id: I53686c7e3e6f73050474c28e86e86c34b3e91ff6
diff --git a/src/com/android/server/telecom/ConnectionServiceWrapper.java b/src/com/android/server/telecom/ConnectionServiceWrapper.java
index cc5932c..5b727ab 100644
--- a/src/com/android/server/telecom/ConnectionServiceWrapper.java
+++ b/src/com/android/server/telecom/ConnectionServiceWrapper.java
@@ -919,6 +919,9 @@
callingPhoneAccountHandle.getComponentName().getPackageName());
}
+ boolean hasCrossUserAccess = mContext.checkCallingOrSelfPermission(
+ android.Manifest.permission.INTERACT_ACROSS_USERS)
+ == PackageManager.PERMISSION_GRANTED;
long token = Binder.clearCallingIdentity();
try {
synchronized (mLock) {
@@ -930,7 +933,7 @@
// an emergency call.
mPhoneAccountRegistrar.getCallCapablePhoneAccounts(null /*uriScheme*/,
false /*includeDisabledAccounts*/, userHandle, 0 /*capabilities*/,
- 0 /*excludedCapabilities*/, false);
+ 0 /*excludedCapabilities*/, hasCrossUserAccess);
PhoneAccountHandle phoneAccountHandle = null;
for (PhoneAccountHandle accountHandle : accountHandles) {
if(accountHandle.equals(callingPhoneAccountHandle)) {