Merge "Add caller check to com.android.credentials.RESET" into qt-dev am: 50b65a168a
am: 1ef5f42b54
am: f43f129dc0
am: d902f8c7d6
am: ea3d0361c8
am: 3a4ebbfcec
am: 45bfdf58b9
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/16577205 Change-Id: Iaced4c96addd8b30fc9eb63f65d496ec122530bd
This commit is contained in:
@@ -86,7 +86,7 @@ public final class CredentialStorage extends FragmentActivity {
|
|||||||
final String action = intent.getAction();
|
final String action = intent.getAction();
|
||||||
final UserManager userManager = (UserManager) getSystemService(Context.USER_SERVICE);
|
final UserManager userManager = (UserManager) getSystemService(Context.USER_SERVICE);
|
||||||
if (!userManager.hasUserRestriction(UserManager.DISALLOW_CONFIG_CREDENTIALS)) {
|
if (!userManager.hasUserRestriction(UserManager.DISALLOW_CONFIG_CREDENTIALS)) {
|
||||||
if (ACTION_RESET.equals(action)) {
|
if (ACTION_RESET.equals(action) && checkCallerIsSelf()) {
|
||||||
new ResetDialog();
|
new ResetDialog();
|
||||||
} else {
|
} else {
|
||||||
if (ACTION_INSTALL.equals(action) && checkCallerIsCertInstallerOrSelfInProfile()) {
|
if (ACTION_INSTALL.equals(action) && checkCallerIsCertInstallerOrSelfInProfile()) {
|
||||||
@@ -318,6 +318,19 @@ public final class CredentialStorage extends FragmentActivity {
|
|||||||
finish();
|
finish();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Check that the caller is Settings.
|
||||||
|
*/
|
||||||
|
private boolean checkCallerIsSelf() {
|
||||||
|
try {
|
||||||
|
return Process.myUid() == android.app.ActivityManager.getService()
|
||||||
|
.getLaunchedFromUid(getActivityToken());
|
||||||
|
} catch (RemoteException re) {
|
||||||
|
// Error talking to ActivityManager, just give up
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Check that the caller is either certinstaller or Settings running in a profile of this user.
|
* Check that the caller is either certinstaller or Settings running in a profile of this user.
|
||||||
*/
|
*/
|
||||||
|
Reference in New Issue
Block a user