Merge "Skip apk verification when collecting certificates." into rvc-dev

This commit is contained in:
Song Pan
2020-04-27 10:53:21 +00:00
committed by Android (Google) Code Review

View File

@@ -568,7 +568,10 @@ public class AppIntegrityManagerServiceImpl extends IAppIntegrityManager.Stub {
try (PackageParser2 parser = mParserSupplier.get()) { try (PackageParser2 parser = mParserSupplier.get()) {
ParsedPackage pkg = parser.parsePackage(installationPath, 0, false); ParsedPackage pkg = parser.parsePackage(installationPath, 0, false);
int flags = PackageManager.GET_SIGNING_CERTIFICATES | PackageManager.GET_META_DATA; int flags = PackageManager.GET_SIGNING_CERTIFICATES | PackageManager.GET_META_DATA;
pkg.setSigningDetails(ParsingPackageUtils.collectCertificates(pkg, false)); // APK signatures is already verified elsewhere in PackageManager. We do not need to
// verify it again since it could cause a timeout for large APKs.
pkg.setSigningDetails(
ParsingPackageUtils.collectCertificates(pkg, /* skipVerify= */ true));
return PackageInfoUtils.generate( return PackageInfoUtils.generate(
pkg, pkg,
null, null,
@@ -709,7 +712,7 @@ public class AppIntegrityManagerServiceImpl extends IAppIntegrityManager.Stub {
// Filter out the rule provider packages that are not system apps. // Filter out the rule provider packages that are not system apps.
List<String> systemAppRuleProviders = new ArrayList<>(); List<String> systemAppRuleProviders = new ArrayList<>();
for (String ruleProvider: integrityRuleProviders) { for (String ruleProvider : integrityRuleProviders) {
if (isSystemApp(ruleProvider)) { if (isSystemApp(ruleProvider)) {
systemAppRuleProviders.add(ruleProvider); systemAppRuleProviders.add(ruleProvider);
} }