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

Commit cc9e2c71 authored Nov 08, 2019 by Nate Myren

Add update flag notify to reset callback

The resetRuntimePermissionsInternal method defines its own version of
the PermissionCallback. However, it did not override the -NotifyListener
versions of the onPermissionUpdated method, which is used by the
updatePermissionFlagsInternal method. This meant that the updated
onPermissionsUpdated was not being called, in favor of the empty,
default onPermissionsUpdatedNotifyListener.

Test: manual
Change-Id: Id1e1b23f9407016c526c10c6a527a44d7ef1b67b

parent e7ca7543

services/core/java/com/android/server/pm/permission/PermissionManagerService.java

+10 −0

Original line number	Diff line number	Diff line
		@@ -1539,6 +1539,16 @@ public class PermissionManagerService extends IPermissionManager.Stub {
		public void onInstallPermissionUpdated() {
		mDefaultPermissionCallback.onInstallPermissionUpdated();
		}

		public void onPermissionUpdatedNotifyListener(@UserIdInt int[] updatedUserIds,
		boolean sync, int uid) {
		onPermissionUpdated(updatedUserIds, sync);
		mOnPermissionChangeListeners.onPermissionsChanged(uid);
		}

		public void onInstallPermissionUpdatedNotifyListener(int uid) {
		mDefaultPermissionCallback.onInstallPermissionUpdatedNotifyListener(uid);
		}
		};

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

services/core/java/com/android/server/pm/permission/PermissionManagerServiceInternal.java

+2 −0

Original line number	Diff line number	Diff line
		@@ -162,12 +162,14 @@ public abstract class PermissionManagerServiceInternal extends PermissionManager
		}
		public void onPermissionUpdatedNotifyListener(@UserIdInt int[] updatedUserIds, boolean sync,
		int uid) {
		onPermissionUpdated(updatedUserIds, sync);
		}
		public void onPermissionRemoved() {
		}
		public void onInstallPermissionUpdated() {
		}
		public void onInstallPermissionUpdatedNotifyListener(int uid) {
		onInstallPermissionUpdated();
		}
		}