Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b32c0210 authored by Rhed Jao's avatar Rhed Jao Committed by Automerger Merge Worker
Browse files

Merge "Enforce package visibility filter on the SharedLibraryInfo" into sc-dev... 

Merge "Enforce package visibility filter on the SharedLibraryInfo" into sc-dev am: dc61ccec am: 4d7db289 am: c481b3c6

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/14661878

Change-Id: If6d012f34f4adee96364ecc2e529598424182557
parents db6a6465 c481b3c6

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

+16 −8
Original line number Diff line number Diff line
@@ -8470,7 +8470,7 @@ public class PackageManagerService extends IPackageManager.Stub 
                            libInfo.getPackageName(), libInfo.getAllCodePaths(),

                            libInfo.getName(), libInfo.getLongVersion(),

                            libInfo.getType(), libInfo.getDeclaringPackage(),

                            getPackagesUsingSharedLibraryLPr(libInfo, flags, userId),

                            getPackagesUsingSharedLibraryLPr(libInfo, flags, callingUid, userId),

                            (libInfo.getDependencies() == null

                                    ? null

                                    : new ArrayList<>(libInfo.getDependencies())),
@@ -8542,8 +8542,10 @@ public class PackageManagerService extends IPackageManager.Stub 
                            libraryInfo.getPath(), libraryInfo.getPackageName(),

                            libraryInfo.getAllCodePaths(), libraryInfo.getName(),

                            libraryInfo.getLongVersion(), libraryInfo.getType(),

                            libraryInfo.getDeclaringPackage(), getPackagesUsingSharedLibraryLPr(

                            libraryInfo, flags, userId), libraryInfo.getDependencies() == null

                            libraryInfo.getDeclaringPackage(),

                            getPackagesUsingSharedLibraryLPr(

                                    libraryInfo, flags, callingUid, userId),

                            libraryInfo.getDependencies() == null

                                    ? null : new ArrayList<>(libraryInfo.getDependencies()),

                            libraryInfo.isNative());









@@ -8560,7 +8562,7 @@ public class PackageManagerService extends IPackageManager.Stub



























    @GuardedBy("mLock")















    private List<VersionedPackage> getPackagesUsingSharedLibraryLPr(













            SharedLibraryInfo libInfo, int flags, int userId) {













            SharedLibraryInfo libInfo, int flags, int callingUid, int userId) {















        List<VersionedPackage> versionedPackages = null;















        final int packageCount = mSettings.getPackagesLocked().size();















        for (int i = 0; i < packageCount; i++) {









@@ -8583,6 +8585,9 @@ public class PackageManagerService extends IPackageManager.Stub













                if (ps.usesStaticLibrariesVersions[libIdx] != libInfo.getLongVersion()) {















                    continue;















                }













                if (shouldFilterApplicationLocked(ps, callingUid, userId)) {













                    continue;













                }















                if (versionedPackages == null) {















                    versionedPackages = new ArrayList<>();















                }









@@ -8595,6 +8600,9 @@ public class PackageManagerService extends IPackageManager.Stub













            } else if (ps.pkg != null) {















                if (ArrayUtils.contains(ps.pkg.getUsesLibraries(), libName)















                        || ArrayUtils.contains(ps.pkg.getUsesOptionalLibraries(), libName)) {













                    if (shouldFilterApplicationLocked(ps, callingUid, userId)) {













                        continue;













                    }















                    if (versionedPackages == null) {















                        versionedPackages = new ArrayList<>();















                    }










@@ -14322,7 +14330,7 @@ public class PackageManagerService extends IPackageManager.Stub













        // Remove the shared library overlays from its dependent packages.















        for (int currentUserId : UserManagerService.getInstance().getUserIds()) {















            final List<VersionedPackage> dependents = getPackagesUsingSharedLibraryLPr(













                    libraryInfo, 0, currentUserId);













                    libraryInfo, 0, Process.SYSTEM_UID, currentUserId);















            if (dependents == null) {















                continue;















            }










@@ -20682,7 +20690,7 @@ public class PackageManagerService extends IPackageManager.Stub













                            continue;















                        }















                        List<VersionedPackage> libClientPackages = getPackagesUsingSharedLibraryLPr(













                                libraryInfo, MATCH_KNOWN_PACKAGES, currUserId);













                                libraryInfo, MATCH_KNOWN_PACKAGES, Process.SYSTEM_UID, currUserId);















                        if (!ArrayUtils.isEmpty(libClientPackages)) {















                            Slog.w(TAG, "Not removing package " + pkg.getManifestPackageName()















                                    + " hosting lib " + libraryInfo.getName() + " version "










@@ -26811,7 +26819,7 @@ public class PackageManagerService extends IPackageManager.Stub













                            continue;















                        }















                        final List<VersionedPackage> dependents = getPackagesUsingSharedLibraryLPr(













                                info, 0, userId);













                                info, 0, Process.SYSTEM_UID, userId);















                        if (dependents == null) {















                            continue;















                        }