Cole Faust | 9c5c09f | 2023-09-06 16:11:44 -0700 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
| 2 | |
| 3 | import subprocess |
| 4 | import argparse |
| 5 | import re |
| 6 | import sys |
| 7 | import zipfile |
| 8 | |
| 9 | def check_target_sdk_less_than_30(args): |
| 10 | if not args.aapt2: |
| 11 | sys.exit('--aapt2 is required') |
| 12 | regex = re.compile(r"targetSdkVersion: *'([0-9]+)'") |
| 13 | output = subprocess.check_output([args.aapt2, "dump", "badging", args.apk], text=True) |
| 14 | targetSdkVersion = None |
| 15 | for line in output.splitlines(): |
| 16 | match = regex.fullmatch(line.strip()) |
| 17 | if match: |
| 18 | targetSdkVersion = int(match.group(1)) |
| 19 | break |
| 20 | |
| 21 | if targetSdkVersion is None or targetSdkVersion >= 30: |
| 22 | sys.exit(args.apk + ": Prebuilt, presigned apks with targetSdkVersion >= 30 (or a codename targetSdkVersion) must set preprocessed: true in the Android.bp definition (because they must be signed with signature v2, and the build system would wreck that signature otherwise)") |
| 23 | |
| 24 | def has_preprocessed_issues(args, *, fail=False): |
| 25 | if not args.zipalign: |
| 26 | sys.exit('--zipalign is required') |
| 27 | ret = subprocess.run([args.zipalign, '-c', '-p', '4', args.apk], stdout=subprocess.DEVNULL).returncode |
| 28 | if ret != 0: |
| 29 | if fail: |
| 30 | sys.exit(args.apk + ': Improper zip alignment') |
| 31 | return True |
| 32 | |
| 33 | with zipfile.ZipFile(args.apk) as zf: |
| 34 | for info in zf.infolist(): |
| 35 | if info.filename.startswith('lib/') and info.filename.endswith('.so') and info.compress_type != zipfile.ZIP_STORED: |
| 36 | if fail: |
| 37 | sys.exit(args.apk + ': Contains compressed JNI libraries') |
| 38 | return True |
| 39 | # It's ok for non-privileged apps to have compressed dex files, see go/gms-uncompressed-jni-slides |
| 40 | if args.privileged: |
| 41 | if info.filename.endswith('.dex') and info.compress_type != zipfile.ZIP_STORED: |
| 42 | if fail: |
| 43 | sys.exit(args.apk + ': Contains compressed dex files and is privileged') |
| 44 | return True |
| 45 | return False |
| 46 | |
| 47 | |
| 48 | def main(): |
| 49 | parser = argparse.ArgumentParser() |
| 50 | parser.add_argument('--aapt2', help = "the path to the aapt2 executable") |
| 51 | parser.add_argument('--zipalign', help = "the path to the zipalign executable") |
| 52 | parser.add_argument('--skip-preprocessed-apk-checks', action = 'store_true', help = "the value of the soong property with the same name") |
| 53 | parser.add_argument('--preprocessed', action = 'store_true', help = "the value of the soong property with the same name") |
| 54 | parser.add_argument('--privileged', action = 'store_true', help = "the value of the soong property with the same name") |
| 55 | parser.add_argument('apk', help = "the apk to check") |
| 56 | parser.add_argument('stampfile', help = "a file to touch if successful") |
| 57 | args = parser.parse_args() |
| 58 | |
| 59 | if not args.preprocessed: |
| 60 | check_target_sdk_less_than_30(args) |
| 61 | elif args.skip_preprocessed_apk_checks: |
| 62 | if not has_preprocessed_issues(args): |
| 63 | sys.exit('This module sets `skip_preprocessed_apk_checks: true`, but does not actually have any issues. Please remove `skip_preprocessed_apk_checks`.') |
| 64 | else: |
| 65 | has_preprocessed_issues(args, fail=True) |
| 66 | |
| 67 | subprocess.check_call(["touch", args.stampfile]) |
| 68 | |
| 69 | if __name__ == "__main__": |
| 70 | main() |