Merge "Add caller check to com.android.credentials.RESET" into qt-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
50b65a168a
@@ -92,7 +92,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()) {
|
||||||
@@ -341,6 +341,19 @@ public final class CredentialStorage extends FragmentActivity {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 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