drgreen/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) {
if ( (deletedPs != null) && (deletedPs.sharedUser != null)) {
// remove permissions associated with package
mSettings.updateSharedUserPerms (deletedPs);
mSettings.updateSharedUserPermsLP(deletedPs, mGlobalGids);
}
// Save settings now
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)) {
Log.i(TAG, "Trying to update info for null package. Just ignoring");
return;
@ -6037,9 +6045,10 @@ class PackageManagerService extends IPackageManager.Stub {
}
}
// Update gids
int newGids[] = null;
for (PackageSetting pkg:sus.packages) {
newGids = appendInts(newGids, pkg.gids);
int newGids[] = globalGids;
for (String eachPerm : sus.grantedPermissions) {
BasePermission bp = mPermissions.get(eachPerm);
newGids = appendInts(newGids, bp.gids);
}
sus.gids = newGids;
}