commit f4c8c3590a82a4f4b56c6ecb7ceb407abdbb83d8
author Treehugger Robot <treehugger-gerrit@google.com> Thu Sep 22 20:00:01 2016 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Thu Sep 22 20:00:01 2016 +0000
tree 6f2e584b801773654eea623b3cfa1acc066d88cc
parent d333f2839736e2d459255c3a061b452b5658bfe8
parent 2527fa3067ce5ccc707203a06f00eea595d05308

Merge "Skip uninstalled package for getPackagesForUid()"

services/core/java/com/android/server/pm/PackageManagerService.java

diff --git a/services/core/java/com/android/server/pm/PackageManagerService.java b/services/core/java/com/android/server/pm/PackageManagerService.java
index 76d987b..85cc124 100644
--- a/services/core/java/com/android/server/pm/PackageManagerService.java
+++ b/services/core/java/com/android/server/pm/PackageManagerService.java
@@ -4578,6 +4578,7 @@
 
     @Override
     public String[] getPackagesForUid(int uid) {
+        final int userId = UserHandle.getUserId(uid);
         uid = UserHandle.getAppId(uid);
         // reader
         synchronized (mPackages) {
@@ -4585,9 +4586,16 @@
             if (obj instanceof SharedUserSetting) {
                 final SharedUserSetting sus = (SharedUserSetting) obj;
                 final int N = sus.packages.size();
-                final String[] res = new String[N];
-                for (int i = 0; i < N; i++) {
-                    res[i] = sus.packages.valueAt(i).name;
+                String[] res = new String[N];
+                final Iterator<PackageSetting> it = sus.packages.iterator();
+                int i = 0;
+                while (it.hasNext()) {
+                    PackageSetting ps = it.next();
+                    if (ps.getInstalled(userId)) {
+                        res[i++] = ps.name;
+                    } else {
+                        res = ArrayUtils.removeElement(String.class, res, res[i]);
+                    }
                 }
                 return res;
             } else if (obj instanceof PackageSetting) {