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

Merge "Show a warning about mismatching test / APK certs" into sc-v2-dev

Browse Source
This commit is contained in:
Makoto Onuki
2022-03-08 16:32:56 +00:00
committed by Android (Google) Code Review
parent 7a40de6568 983caeddb4
commit 34c8baf080
1 changed files with 17 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
services/core/java/com/android/server/am/ActivityManagerService.java
View File

@@ -13804,14 +13804,26 @@ public class ActivityManagerService extends IActivityManager.Stub
return false; return false;
} }
if (!Build.IS_DEBUGGABLE) {
int match = mContext.getPackageManager().checkSignatures( int match = mContext.getPackageManager().checkSignatures(
ii.targetPackage, ii.packageName); ii.targetPackage, ii.packageName);
if (match < 0 && match != PackageManager.SIGNATURE_FIRST_NOT_SIGNED) { if (match < 0 && match != PackageManager.SIGNATURE_FIRST_NOT_SIGNED) {
if (Build.IS_DEBUGGABLE) {
String message = "Instrumentation test " + ii.packageName
+ " doesn't have a signature matching the target "
+ ii.targetPackage
+ ", which would not be allowed on the production Android builds";
if (callingUid != Process.ROOT_UID) {
Slog.e(TAG, message
+ ". THIS WILL BE DISALLOWED ON FUTURE ANDROID VERSIONS"
+ " unless from a rooted ADB shell.");
} else {
Slog.w(TAG, message);
}
} else {
String msg = "Permission Denial: starting instrumentation " String msg = "Permission Denial: starting instrumentation "
+ className + " from pid=" + className + " from pid="
+ Binder.getCallingPid() + Binder.getCallingPid()
+ ", uid=" + Binder.getCallingPid() + ", uid=" + Binder.getCallingUid()
+ " not allowed because package " + ii.packageName + " not allowed because package " + ii.packageName
+ " does not have a signature matching the target " + " does not have a signature matching the target "
+ ii.targetPackage; + ii.targetPackage;