summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Nicholas Sauer <nicksauer@google.com> 2018-12-16 11:23:46 -0800
committer Nicholas Sauer <nicksauer@google.com> 2018-12-16 19:34:36 +0000
commit8d7120f68a5f62cf0e173c7a1062cf50f6eb5822 (patch)
tree1d1583202fdf9c7162d30e487e434b0bc8aafa0c
parent90dcdb2177a1edae7b37ae72a54bedb8eb19507b (diff)
Retrieve packages for the specified user.
DefaultDialerManager#setDefaultDialerApplication is meant to execute on the current user. However it retreives packages for the user specified in the context -- which can be different than the user specified in the method call. As a result, calls to setDefaultDialerApplication can be successful even if the component is disabled in the current (secondary) user. bug: 121094764 Test: via CTS adb shell am switch-user 10 cts-tradefed run cts-dev -a arm64-v8a -m CtsTelecomTestCases -t android.telecom.cts.DefaultDialerOpest#testSetDefaultDialerNoDialIntent_notSupported adb shell am switch-user 0 cts-tradefed run cts-dev -a arm64-v8a -m CtsTelecomTestCases -t android.telecom.cts.DefaultDialerOpest#testSetDefaultDialerNoDialIntent_notSupported Change-Id: I0992594f8791791eacba43e03ca1c293832eddcf
-rw-r--r--telecomm/java/android/telecom/DefaultDialerManager.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/telecomm/java/android/telecom/DefaultDialerManager.java b/telecomm/java/android/telecom/DefaultDialerManager.java
index 1806aee27e31..2680af76d803 100644
--- a/telecomm/java/android/telecom/DefaultDialerManager.java
+++ b/telecomm/java/android/telecom/DefaultDialerManager.java
@@ -74,7 +74,7 @@ public class DefaultDialerManager {
}
// Only make the change if the new package belongs to a valid phone application
- List<String> packageNames = getInstalledDialerApplications(context);
+ List<String> packageNames = getInstalledDialerApplications(context, user);
if (packageNames.contains(packageName)) {
// Update the secure setting.