Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge change 20037 into donut

Browse Source 
* changes:
  fix issue with gids not being added for packages with shared user id
This commit is contained in:
Android (Google) Code Review
2009-08-05 15:42:54 -07:00
parent 89888b2d9d 2ed287b6ba
commit 8b2cdcd184
1 changed files with 17 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
services/java/com/android/server/PackageManagerService.java
View File

@@ -4220,7 +4220,7 @@ class PackageManagerService extends IPackageManager.Stub {
synchronized (mPackages) { synchronized (mPackages) {
if ( (deletedPs != null) && (deletedPs.sharedUser != null)) { if ( (deletedPs != null) && (deletedPs.sharedUser != null)) {
// remove permissions associated with package // remove permissions associated with package
mSettings.updateSharedUserPerms (deletedPs); mSettings.updateSharedUserPermsLP(deletedPs, mGlobalGids);
} }
// Save settings now // Save settings now
mSettings.writeLP (); mSettings.writeLP ();
@@ -6008,7 +6008,15 @@ class PackageManagerService extends IPackageManager.Stub {
} }
} }
private void updateSharedUserPerms (PackageSetting deletedPs) { /*
* Update the shared user setting when a package using
* specifying the shared user id is removed. The gids
* associated with each permission of the deleted package
* are removed from the shared user's gid list only if its
* not in use by other permissions of packages in the
* shared user setting.
*/
private void updateSharedUserPermsLP(PackageSetting deletedPs, int[] globalGids) {
if ( (deletedPs == null) || (deletedPs.pkg == null)) { if ( (deletedPs == null) || (deletedPs.pkg == null)) {
Log.i(TAG, "Trying to update info for null package. Just ignoring"); Log.i(TAG, "Trying to update info for null package. Just ignoring");
return; return;
@@ -6037,9 +6045,10 @@ class PackageManagerService extends IPackageManager.Stub {
} }
} }
// Update gids // Update gids
int newGids[] = null; int newGids[] = globalGids;
for (PackageSetting pkg:sus.packages) { for (String eachPerm : sus.grantedPermissions) {
newGids = appendInts(newGids, pkg.gids); BasePermission bp = mPermissions.get(eachPerm);
newGids = appendInts(newGids, bp.gids);
} }
sus.gids = newGids; sus.gids = newGids;
} }