Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
| 2 | |
| 3 | # Copyright (C) 2019 The Android Open Source Project |
| 4 | # |
| 5 | # Licensed under the Apache License, Version 2.0 (the "License"); |
| 6 | # you may not use this file except in compliance with the License. |
| 7 | # You may obtain a copy of the License at |
| 8 | # |
| 9 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 10 | # |
| 11 | # Unless required by applicable law or agreed to in writing, software |
| 12 | # distributed under the License is distributed on an "AS IS" BASIS, |
| 13 | # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 14 | # See the License for the specific language governing permissions and |
| 15 | # limitations under the License. |
| 16 | # |
| 17 | |
| 18 | import argparse |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 19 | import fnmatch |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 20 | import logging |
| 21 | import os |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 22 | import os.path |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 23 | import subprocess |
| 24 | import sys |
| 25 | import zipfile |
| 26 | |
| 27 | logging.basicConfig(format='%(message)s') |
| 28 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 29 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 30 | class FSObject: |
| 31 | def __init__(self, name, is_dir, is_exec, is_symlink): |
| 32 | self.name = name |
| 33 | self.is_dir = is_dir |
| 34 | self.is_exec = is_exec |
| 35 | self.is_symlink = is_symlink |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 36 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 37 | def __str__(self): |
| 38 | return '%s(dir=%r,exec=%r,symlink=%r)' % (self.name, self.is_dir, self.is_exec, self.is_symlink) |
| 39 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 40 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 41 | class TargetApexProvider: |
| 42 | def __init__(self, apex, tmpdir, debugfs): |
| 43 | self._tmpdir = tmpdir |
| 44 | self._debugfs = debugfs |
| 45 | self._folder_cache = {} |
| 46 | self._payload = os.path.join(self._tmpdir, 'apex_payload.img') |
| 47 | # Extract payload to tmpdir. |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 48 | apex_zip = zipfile.ZipFile(apex) |
| 49 | apex_zip.extract('apex_payload.img', tmpdir) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 50 | |
| 51 | def __del__(self): |
| 52 | # Delete temps. |
| 53 | if os.path.exists(self._payload): |
| 54 | os.remove(self._payload) |
| 55 | |
| 56 | def get(self, path): |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 57 | apex_dir, name = os.path.split(path) |
| 58 | if not apex_dir: |
| 59 | apex_dir = '.' |
| 60 | apex_map = self.read_dir(apex_dir) |
| 61 | return apex_map[name] if name in apex_map else None |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 62 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 63 | def read_dir(self, apex_dir): |
| 64 | if apex_dir in self._folder_cache: |
| 65 | return self._folder_cache[apex_dir] |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 66 | # Cannot use check_output as it will annoy with stderr. |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 67 | process = subprocess.Popen([self._debugfs, '-R', 'ls -l -p %s' % apex_dir, self._payload], |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 68 | stdout=subprocess.PIPE, stderr=subprocess.PIPE, |
| 69 | universal_newlines=True) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 70 | stdout, _ = process.communicate() |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 71 | res = str(stdout) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 72 | apex_map = {} |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 73 | # Debugfs output looks like this: |
| 74 | # debugfs 1.44.4 (18-Aug-2018) |
| 75 | # /12/040755/0/2000/.// |
| 76 | # /2/040755/1000/1000/..// |
| 77 | # /13/100755/0/2000/dalvikvm32/28456/ |
| 78 | # /14/100755/0/2000/dexoptanalyzer/20396/ |
| 79 | # /15/100755/0/2000/linker/1152724/ |
| 80 | # /16/100755/0/2000/dex2oat/563508/ |
| 81 | # /17/100755/0/2000/linker64/1605424/ |
| 82 | # /18/100755/0/2000/profman/85304/ |
| 83 | # /19/100755/0/2000/dalvikvm64/28576/ |
| 84 | # | | | | | | |
| 85 | # | | | #- gid #- name #- size |
| 86 | # | | #- uid |
| 87 | # | #- type and permission bits |
| 88 | # #- inode nr (?) |
| 89 | # |
| 90 | # Note: could break just on '/' to avoid names with newlines. |
| 91 | for line in res.split("\n"): |
| 92 | if not line: |
| 93 | continue |
| 94 | comps = line.split('/') |
| 95 | if len(comps) != 8: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 96 | logging.warning('Could not break and parse line \'%s\'', line) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 97 | continue |
| 98 | bits = comps[2] |
| 99 | name = comps[5] |
| 100 | if len(bits) != 6: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 101 | logging.warning('Dont understand bits \'%s\'', bits) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 102 | continue |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 103 | is_dir = bits[1] == '4' |
| 104 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 105 | def is_exec_bit(ch): |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 106 | return int(ch) & 1 == 1 |
| 107 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 108 | is_exec = is_exec_bit(bits[3]) and is_exec_bit(bits[4]) and is_exec_bit(bits[5]) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 109 | is_symlink = bits[1] == '2' |
| 110 | apex_map[name] = FSObject(name, is_dir, is_exec, is_symlink) |
| 111 | self._folder_cache[apex_dir] = apex_map |
| 112 | return apex_map |
| 113 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 114 | |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 115 | class HostApexProvider: |
| 116 | def __init__(self, apex, tmpdir): |
| 117 | self._tmpdir = tmpdir |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 118 | self.folder_cache = {} |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 119 | self._payload = os.path.join(self._tmpdir, 'apex_payload.zip') |
| 120 | # Extract payload to tmpdir. |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 121 | apex_zip = zipfile.ZipFile(apex) |
| 122 | apex_zip.extract('apex_payload.zip', tmpdir) |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 123 | |
| 124 | def __del__(self): |
| 125 | # Delete temps. |
| 126 | if os.path.exists(self._payload): |
| 127 | os.remove(self._payload) |
| 128 | |
| 129 | def get(self, path): |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 130 | apex_dir, name = os.path.split(path) |
| 131 | if not apex_dir: |
| 132 | apex_dir = '' |
| 133 | apex_map = self.read_dir(apex_dir) |
| 134 | return apex_map[name] if name in apex_map else None |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 135 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 136 | def read_dir(self, apex_dir): |
| 137 | if apex_dir in self.folder_cache: |
| 138 | return self.folder_cache[apex_dir] |
| 139 | if not self.folder_cache: |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 140 | self.parse_zip() |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 141 | if apex_dir in self.folder_cache: |
| 142 | return self.folder_cache[apex_dir] |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 143 | return {} |
| 144 | |
| 145 | def parse_zip(self): |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 146 | apex_zip = zipfile.ZipFile(self._payload) |
| 147 | infos = apex_zip.infolist() |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 148 | for zipinfo in infos: |
| 149 | path = zipinfo.filename |
| 150 | |
| 151 | # Assume no empty file is stored. |
| 152 | assert path |
| 153 | |
| 154 | def get_octal(val, index): |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 155 | return (val >> (index * 3)) & 0x7 |
| 156 | |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 157 | def bits_is_exec(val): |
| 158 | # TODO: Enforce group/other, too? |
| 159 | return get_octal(val, 2) & 1 == 1 |
| 160 | |
| 161 | is_zipinfo = True |
| 162 | while path: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 163 | apex_dir, base = os.path.split(path) |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 164 | # TODO: If directories are stored, base will be empty. |
| 165 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 166 | if apex_dir not in self.folder_cache: |
| 167 | self.folder_cache[apex_dir] = {} |
| 168 | dir_map = self.folder_cache[apex_dir] |
| 169 | if base not in dir_map: |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 170 | if is_zipinfo: |
| 171 | bits = (zipinfo.external_attr >> 16) & 0xFFFF |
| 172 | is_dir = get_octal(bits, 4) == 4 |
| 173 | is_symlink = get_octal(bits, 4) == 2 |
| 174 | is_exec = bits_is_exec(bits) |
| 175 | else: |
| 176 | is_exec = False # Seems we can't get this easily? |
| 177 | is_symlink = False |
| 178 | is_dir = True |
| 179 | dir_map[base] = FSObject(base, is_dir, is_exec, is_symlink) |
| 180 | is_zipinfo = False |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 181 | path = apex_dir |
| 182 | |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 183 | |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 184 | # DO NOT USE DIRECTLY! This is an "abstract" base class. |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 185 | class Checker: |
| 186 | def __init__(self, provider): |
| 187 | self._provider = provider |
| 188 | self._errors = 0 |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 189 | self._expected_file_globs = set() |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 190 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 191 | def fail(self, msg, *fail_args): |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 192 | self._errors += 1 |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 193 | logging.error(msg, *fail_args) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 194 | |
| 195 | def error_count(self): |
| 196 | return self._errors |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 197 | |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 198 | def reset_errors(self): |
| 199 | self._errors = 0 |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 200 | |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 201 | def is_file(self, path): |
| 202 | fs_object = self._provider.get(path) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 203 | if fs_object is None: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 204 | return False, 'Could not find %s' |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 205 | if fs_object.is_dir: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 206 | return False, '%s is a directory' |
| 207 | return True, '' |
Andreas Gampe | a0242cf | 2019-01-29 13:01:23 -0800 | [diff] [blame] | 208 | |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 209 | def check_file(self, path): |
| 210 | ok, msg = self.is_file(path) |
| 211 | if not ok: |
| 212 | self.fail(msg, path) |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 213 | self._expected_file_globs.add(path) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 214 | return ok |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 215 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 216 | def check_executable(self, filename): |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 217 | path = 'bin/%s' % filename |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 218 | if not self.check_file(path): |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 219 | return |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 220 | if not self._provider.get(path).is_exec: |
| 221 | self.fail('%s is not executable', path) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 222 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 223 | def check_executable_symlink(self, filename): |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 224 | path = 'bin/%s' % filename |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 225 | fs_object = self._provider.get(path) |
| 226 | if fs_object is None: |
| 227 | self.fail('Could not find %s', path) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 228 | return |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 229 | if fs_object.is_dir: |
| 230 | self.fail('%s is a directory', path) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 231 | return |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 232 | if not fs_object.is_symlink: |
| 233 | self.fail('%s is not a symlink', path) |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 234 | self._expected_file_globs.add(path) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 235 | |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 236 | def check_single_library(self, filename): |
| 237 | lib_path = 'lib/%s' % filename |
| 238 | lib64_path = 'lib64/%s' % filename |
| 239 | lib_is_file, _ = self.is_file(lib_path) |
| 240 | if lib_is_file: |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 241 | self._expected_file_globs.add(lib_path) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 242 | lib64_is_file, _ = self.is_file(lib64_path) |
| 243 | if lib64_is_file: |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 244 | self._expected_file_globs.add(lib64_path) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 245 | if not lib_is_file and not lib64_is_file: |
| 246 | self.fail('Library missing: %s', filename) |
Andreas Gampe | a0242cf | 2019-01-29 13:01:23 -0800 | [diff] [blame] | 247 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 248 | def check_java_library(self, basename): |
| 249 | return self.check_file('javalib/%s.jar' % basename) |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 250 | |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 251 | def ignore_path(self, path_glob): |
| 252 | self._expected_file_globs.add(path_glob) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 253 | |
| 254 | def check_no_superfluous_files(self, dir_path): |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 255 | paths = [] |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 256 | for name in sorted(self._provider.read_dir(dir_path).keys()): |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 257 | if name not in ('.', '..'): |
| 258 | paths.append(os.path.join(dir_path, name)) |
| 259 | expected_paths = set() |
| 260 | dir_prefix = dir_path + '/' |
| 261 | for path_glob in self._expected_file_globs: |
| 262 | expected_paths |= set(fnmatch.filter(paths, path_glob)) |
| 263 | # If there are globs in subdirectories of dir_path we want to match their |
| 264 | # path segments at this directory level. |
| 265 | if path_glob.startswith(dir_prefix): |
| 266 | subpath = path_glob[len(dir_prefix):] |
| 267 | subpath_first_segment, _, _ = subpath.partition('/') |
| 268 | expected_paths |= set(fnmatch.filter(paths, dir_prefix + subpath_first_segment)) |
| 269 | for unexpected_path in set(paths) - expected_paths: |
| 270 | self.fail('Unexpected file \'%s\'', unexpected_path) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 271 | |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 272 | # Just here for docs purposes, even if it isn't good Python style. |
| 273 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 274 | def check_symlinked_multilib_executable(self, filename): |
| 275 | """Check bin/filename32, and/or bin/filename64, with symlink bin/filename.""" |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 276 | raise NotImplementedError |
| 277 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 278 | def check_symlinked_prefer32_executable(self, filename): |
| 279 | """Check bin/filename32, or bin/filename64 on 64 bit only, with symlink bin/filename.""" |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 280 | raise NotImplementedError |
| 281 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 282 | def check_multilib_executable(self, filename): |
| 283 | """Check bin/filename for 32 bit, and/or bin/filename64.""" |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 284 | raise NotImplementedError |
| 285 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 286 | def check_native_library(self, basename): |
| 287 | """Check lib/basename.so, and/or lib64/basename.so.""" |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 288 | raise NotImplementedError |
| 289 | |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 290 | def check_optional_native_library(self, basename_glob): |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 291 | """Allow lib/basename.so and/or lib64/basename.so to exist.""" |
Martin Stjernholm | b81fe23 | 2019-03-25 17:38:04 +0000 | [diff] [blame] | 292 | raise NotImplementedError |
| 293 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 294 | def check_prefer64_library(self, basename): |
| 295 | """Check lib64/basename.so, or lib/basename.so on 32 bit only.""" |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 296 | raise NotImplementedError |
| 297 | |
| 298 | |
| 299 | class Arch32Checker(Checker): |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 300 | def check_symlinked_multilib_executable(self, filename): |
| 301 | self.check_executable('%s32' % filename) |
| 302 | self.check_executable_symlink(filename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 303 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 304 | def check_symlinked_prefer32_executable(self, filename): |
| 305 | self.check_executable('%s32' % filename) |
| 306 | self.check_executable_symlink(filename) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 307 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 308 | def check_multilib_executable(self, filename): |
| 309 | self.check_executable(filename) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 310 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 311 | def check_native_library(self, basename): |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 312 | # TODO: Use $TARGET_ARCH (e.g. check whether it is "arm" or "arm64") to improve |
| 313 | # the precision of this test? |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 314 | self.check_file('lib/%s.so' % basename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 315 | |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 316 | def check_optional_native_library(self, basename_glob): |
| 317 | self.ignore_path('lib/%s.so' % basename_glob) |
Martin Stjernholm | b81fe23 | 2019-03-25 17:38:04 +0000 | [diff] [blame] | 318 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 319 | def check_prefer64_library(self, basename): |
| 320 | self.check_native_library(basename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 321 | |
| 322 | |
| 323 | class Arch64Checker(Checker): |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 324 | def check_symlinked_multilib_executable(self, filename): |
| 325 | self.check_executable('%s64' % filename) |
| 326 | self.check_executable_symlink(filename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 327 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 328 | def check_symlinked_prefer32_executable(self, filename): |
| 329 | self.check_executable('%s64' % filename) |
| 330 | self.check_executable_symlink(filename) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 331 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 332 | def check_multilib_executable(self, filename): |
| 333 | self.check_executable('%s64' % filename) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 334 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 335 | def check_native_library(self, basename): |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 336 | # TODO: Use $TARGET_ARCH (e.g. check whether it is "arm" or "arm64") to improve |
| 337 | # the precision of this test? |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 338 | self.check_file('lib64/%s.so' % basename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 339 | |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 340 | def check_optional_native_library(self, basename_glob): |
| 341 | self.ignore_path('lib64/%s.so' % basename_glob) |
Martin Stjernholm | b81fe23 | 2019-03-25 17:38:04 +0000 | [diff] [blame] | 342 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 343 | def check_prefer64_library(self, basename): |
| 344 | self.check_native_library(basename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 345 | |
| 346 | |
| 347 | class MultilibChecker(Checker): |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 348 | def check_symlinked_multilib_executable(self, filename): |
| 349 | self.check_executable('%s32' % filename) |
| 350 | self.check_executable('%s64' % filename) |
| 351 | self.check_executable_symlink(filename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 352 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 353 | def check_symlinked_prefer32_executable(self, filename): |
| 354 | self.check_executable('%s32' % filename) |
| 355 | self.check_executable_symlink(filename) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 356 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 357 | def check_multilib_executable(self, filename): |
| 358 | self.check_executable('%s64' % filename) |
| 359 | self.check_executable(filename) |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 360 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 361 | def check_native_library(self, basename): |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 362 | # TODO: Use $TARGET_ARCH (e.g. check whether it is "arm" or "arm64") to improve |
| 363 | # the precision of this test? |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 364 | self.check_file('lib/%s.so' % basename) |
| 365 | self.check_file('lib64/%s.so' % basename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 366 | |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 367 | def check_optional_native_library(self, basename_glob): |
| 368 | self.ignore_path('lib/%s.so' % basename_glob) |
| 369 | self.ignore_path('lib64/%s.so' % basename_glob) |
Martin Stjernholm | b81fe23 | 2019-03-25 17:38:04 +0000 | [diff] [blame] | 370 | |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 371 | def check_prefer64_library(self, basename): |
| 372 | self.check_file('lib64/%s.so' % basename) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 373 | |
| 374 | |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 375 | class ReleaseChecker: |
| 376 | def __init__(self, checker): |
| 377 | self._checker = checker |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 378 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 379 | def __str__(self): |
| 380 | return 'Release Checker' |
| 381 | |
| 382 | def run(self): |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 383 | # Check the APEX manifest. |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 384 | self._checker.check_file('apex_manifest.json') |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 385 | |
Martin Stjernholm | 77f1766 | 2019-04-03 17:08:21 +0100 | [diff] [blame] | 386 | # Check binaries for ART. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 387 | self._checker.check_executable('dex2oat') |
Martin Stjernholm | 77f1766 | 2019-04-03 17:08:21 +0100 | [diff] [blame] | 388 | self._checker.check_executable('dexdump') |
| 389 | self._checker.check_executable('dexlist') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 390 | self._checker.check_executable('dexoptanalyzer') |
| 391 | self._checker.check_executable('profman') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 392 | self._checker.check_symlinked_multilib_executable('dalvikvm') |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 393 | |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 394 | # Check exported libraries for ART. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 395 | self._checker.check_native_library('libdexfile_external') |
| 396 | self._checker.check_native_library('libnativebridge') |
| 397 | self._checker.check_native_library('libnativehelper') |
| 398 | self._checker.check_native_library('libnativeloader') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 399 | |
| 400 | # Check internal libraries for ART. |
| 401 | self._checker.check_native_library('libadbconnection') |
| 402 | self._checker.check_native_library('libart') |
| 403 | self._checker.check_native_library('libart-compiler') |
| 404 | self._checker.check_native_library('libart-dexlayout') |
| 405 | self._checker.check_native_library('libartbase') |
| 406 | self._checker.check_native_library('libartpalette') |
| 407 | self._checker.check_native_library('libdexfile') |
| 408 | self._checker.check_native_library('libdexfile_support') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 409 | self._checker.check_native_library('libopenjdkjvm') |
| 410 | self._checker.check_native_library('libopenjdkjvmti') |
| 411 | self._checker.check_native_library('libprofile') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 412 | self._checker.check_native_library('libsigchain') |
| 413 | |
| 414 | # Check java libraries for Managed Core Library. |
| 415 | self._checker.check_java_library('apache-xml') |
| 416 | self._checker.check_java_library('bouncycastle') |
| 417 | self._checker.check_java_library('core-libart') |
| 418 | self._checker.check_java_library('core-oj') |
| 419 | self._checker.check_java_library('okhttp') |
| 420 | |
| 421 | # Check internal native libraries for Managed Core Library. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 422 | self._checker.check_native_library('libjavacore') |
| 423 | self._checker.check_native_library('libopenjdk') |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 424 | |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 425 | # Check internal native library dependencies. |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 426 | # |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 427 | # Any internal dependency not listed here will cause a failure in |
| 428 | # NoSuperfluousLibrariesChecker. Internal dependencies are generally just |
| 429 | # implementation details, but in the release package we want to track them |
| 430 | # because a) they add to the package size and the RAM usage (in particular |
| 431 | # if the library is also present in /system or another APEX and hence might |
| 432 | # get loaded twice through linker namespace separation), and b) we need to |
| 433 | # catch invalid dependencies on /system or other APEXes that should go |
| 434 | # through an exported library with stubs (b/128708192 tracks implementing a |
| 435 | # better approach for that). |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 436 | self._checker.check_native_library('libbacktrace') |
| 437 | self._checker.check_native_library('libbase') |
| 438 | self._checker.check_native_library('libc++') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 439 | self._checker.check_native_library('libdt_fd_forward') |
| 440 | self._checker.check_native_library('libdt_socket') |
| 441 | self._checker.check_native_library('libjdwp') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 442 | self._checker.check_native_library('liblzma') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 443 | self._checker.check_native_library('libnpt') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 444 | self._checker.check_native_library('libunwindstack') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 445 | self._checker.check_native_library('libziparchive') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 446 | self._checker.check_optional_native_library('libvixl') # Only on ARM/ARM64 |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 447 | |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 448 | # Allow extra dependencies that appear in ASAN builds. |
| 449 | self._checker.check_optional_native_library('libclang_rt.asan*') |
| 450 | self._checker.check_optional_native_library('libclang_rt.hwasan*') |
| 451 | self._checker.check_optional_native_library('libclang_rt.ubsan*') |
| 452 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 453 | |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 454 | class ReleaseTargetChecker: |
| 455 | def __init__(self, checker): |
| 456 | self._checker = checker |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 457 | |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 458 | def __str__(self): |
| 459 | return 'Release (Target) Checker' |
| 460 | |
| 461 | def run(self): |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 462 | # Check the APEX package scripts. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 463 | self._checker.check_executable('art_postinstall_hook') |
| 464 | self._checker.check_executable('art_preinstall_hook') |
| 465 | self._checker.check_executable('art_preinstall_hook_boot') |
| 466 | self._checker.check_executable('art_preinstall_hook_system_server') |
| 467 | self._checker.check_executable('art_prepostinstall_utils') |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 468 | |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 469 | # Check binaries for ART. |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 470 | self._checker.check_executable('oatdump') |
| 471 | |
| 472 | # Check internal libraries for ART. |
| 473 | self._checker.check_prefer64_library('libart-disassembler') |
| 474 | |
| 475 | # Check binaries for Bionic. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 476 | self._checker.check_multilib_executable('linker') |
| 477 | self._checker.check_multilib_executable('linker_asan') |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 478 | |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 479 | # Check libraries for Bionic. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 480 | self._checker.check_native_library('bionic/libc') |
| 481 | self._checker.check_native_library('bionic/libdl') |
| 482 | self._checker.check_native_library('bionic/libm') |
Jiyong Park | 408f7d0 | 2019-04-02 23:04:42 +0900 | [diff] [blame] | 483 | # ... and its internal dependencies |
| 484 | self._checker.check_native_library('libc_malloc_hooks') |
| 485 | self._checker.check_native_library('libc_malloc_debug') |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 486 | |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 487 | # Check exported native libraries for Managed Core Library. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 488 | self._checker.check_native_library('libandroidicu') |
| 489 | self._checker.check_native_library('libandroidio') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 490 | |
| 491 | # Check internal native library dependencies. |
| 492 | self._checker.check_native_library('libcrypto') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 493 | self._checker.check_native_library('libexpat') |
| 494 | self._checker.check_native_library('libicui18n') |
| 495 | self._checker.check_native_library('libicuuc') |
| 496 | self._checker.check_native_library('libpac') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 497 | self._checker.check_native_library('libz') |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 498 | |
Martin Stjernholm | 4160c12 | 2019-04-23 17:05:39 +0100 | [diff] [blame] | 499 | # Guest architecture proxy libraries currently end up in these |
| 500 | # subdirectories in x86 builds with native bridge. |
| 501 | # TODO(b/131155689): These are unused - fix the build rules to avoid |
| 502 | # creating them. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 503 | self._checker.ignore_path('lib/arm') |
| 504 | self._checker.ignore_path('lib/arm64') |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 505 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 506 | |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 507 | class ReleaseHostChecker: |
| 508 | def __init__(self, checker): |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 509 | self._checker = checker |
| 510 | |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 511 | def __str__(self): |
| 512 | return 'Release (Host) Checker' |
| 513 | |
| 514 | def run(self): |
Martin Stjernholm | 84a32cd | 2019-03-29 16:39:46 +0000 | [diff] [blame] | 515 | # Check binaries for ART. |
| 516 | self._checker.check_executable('hprof-conv') |
| 517 | self._checker.check_symlinked_multilib_executable('dex2oatd') |
| 518 | |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 519 | # Check exported native libraries for Managed Core Library. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 520 | self._checker.check_native_library('libandroidicu-host') |
| 521 | self._checker.check_native_library('libandroidio') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 522 | |
| 523 | # Check internal libraries for Managed Core Library. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 524 | self._checker.check_native_library('libexpat-host') |
| 525 | self._checker.check_native_library('libicui18n-host') |
| 526 | self._checker.check_native_library('libicuuc-host') |
| 527 | self._checker.check_native_library('libz-host') |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 528 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 529 | |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 530 | class DebugChecker: |
| 531 | def __init__(self, checker): |
| 532 | self._checker = checker |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 533 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 534 | def __str__(self): |
| 535 | return 'Debug Checker' |
| 536 | |
| 537 | def run(self): |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 538 | # Check binaries for ART. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 539 | self._checker.check_executable('dexdiag') |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 540 | |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 541 | # Check debug binaries for ART. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 542 | self._checker.check_executable('dexoptanalyzerd') |
| 543 | self._checker.check_executable('profmand') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 544 | self._checker.check_symlinked_prefer32_executable('dex2oatd') |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 545 | |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 546 | # Check internal libraries for ART. |
| 547 | self._checker.check_native_library('libadbconnectiond') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 548 | self._checker.check_native_library('libartbased') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 549 | self._checker.check_native_library('libartd') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 550 | self._checker.check_native_library('libartd-compiler') |
| 551 | self._checker.check_native_library('libartd-dexlayout') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 552 | self._checker.check_native_library('libdexfiled') |
| 553 | self._checker.check_native_library('libopenjdkjvmd') |
| 554 | self._checker.check_native_library('libopenjdkjvmtid') |
| 555 | self._checker.check_native_library('libprofiled') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 556 | |
| 557 | # Check internal libraries for Managed Core Library. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 558 | self._checker.check_native_library('libopenjdkd') |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 559 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 560 | |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 561 | class DebugTargetChecker: |
| 562 | def __init__(self, checker): |
| 563 | self._checker = checker |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 564 | |
Andreas Gampe | a0242cf | 2019-01-29 13:01:23 -0800 | [diff] [blame] | 565 | def __str__(self): |
| 566 | return 'Debug (Target) Checker' |
| 567 | |
| 568 | def run(self): |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 569 | # Check ART debug binaries. |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 570 | self._checker.check_executable('oatdumpd') |
Martin Stjernholm | 023c218 | 2019-03-28 15:52:32 +0000 | [diff] [blame] | 571 | |
| 572 | # Check ART internal libraries. |
Martin Stjernholm | 356864a | 2019-04-30 16:22:17 +0100 | [diff] [blame^] | 573 | self._checker.check_native_library('libdexfiled_external') |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 574 | self._checker.check_prefer64_library('libartd-disassembler') |
Andreas Gampe | a0242cf | 2019-01-29 13:01:23 -0800 | [diff] [blame] | 575 | |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 576 | # Check internal native library dependencies. |
| 577 | # |
| 578 | # Like in the release package, we check that we don't get other dependencies |
| 579 | # besides those listed here. In this case the concern is not bloat, but |
| 580 | # rather that we don't get behavioural differences between user (release) |
| 581 | # and userdebug/eng builds, which could happen if the debug package has |
| 582 | # duplicate library instances where releases don't. In other words, it's |
| 583 | # uncontroversial to add debug-only dependencies, as long as they don't make |
| 584 | # assumptions on having a single global state (ideally they should have |
| 585 | # double_loadable:true, cf. go/double_loadable). Also, like in the release |
| 586 | # package we need to look out for dependencies that should go through |
| 587 | # exported library stubs (until b/128708192 is fixed). |
Martin Stjernholm | c17ace2 | 2019-03-18 21:47:31 +0000 | [diff] [blame] | 588 | self._checker.check_optional_native_library('libvixld') # Only on ARM/ARM64 |
Martin Stjernholm | 77f1766 | 2019-04-03 17:08:21 +0100 | [diff] [blame] | 589 | self._checker.check_prefer64_library('libmeminfo') |
| 590 | self._checker.check_prefer64_library('libprocinfo') |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 591 | |
| 592 | |
| 593 | class NoSuperfluousBinariesChecker: |
| 594 | def __init__(self, checker): |
| 595 | self._checker = checker |
| 596 | |
| 597 | def __str__(self): |
| 598 | return 'No superfluous binaries checker' |
| 599 | |
| 600 | def run(self): |
| 601 | self._checker.check_no_superfluous_files('bin') |
| 602 | |
| 603 | |
| 604 | class NoSuperfluousLibrariesChecker: |
| 605 | def __init__(self, checker): |
| 606 | self._checker = checker |
| 607 | |
| 608 | def __str__(self): |
| 609 | return 'No superfluous libraries checker' |
| 610 | |
| 611 | def run(self): |
| 612 | self._checker.check_no_superfluous_files('javalib') |
| 613 | self._checker.check_no_superfluous_files('lib') |
| 614 | self._checker.check_no_superfluous_files('lib/bionic') |
| 615 | self._checker.check_no_superfluous_files('lib64') |
| 616 | self._checker.check_no_superfluous_files('lib64/bionic') |
| 617 | |
Andreas Gampe | b1d5567 | 2019-01-29 22:17:02 +0000 | [diff] [blame] | 618 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 619 | class List: |
| 620 | def __init__(self, provider): |
| 621 | self._provider = provider |
| 622 | self._path = '' |
| 623 | |
| 624 | def print_list(self): |
| 625 | apex_map = self._provider.read_dir(self._path) |
| 626 | if apex_map is None: |
| 627 | return |
| 628 | apex_map = dict(apex_map) |
| 629 | if '.' in apex_map: |
| 630 | del apex_map['.'] |
| 631 | if '..' in apex_map: |
| 632 | del apex_map['..'] |
| 633 | for (_, val) in sorted(apex_map.items()): |
| 634 | self._path = os.path.join(self._path, val.name) |
| 635 | print(self._path) |
| 636 | if val.is_dir: |
| 637 | self.print_list() |
| 638 | |
| 639 | |
| 640 | class Tree: |
| 641 | def __init__(self, provider, title): |
| 642 | print('%s' % title) |
| 643 | self._provider = provider |
| 644 | self._path = '' |
| 645 | self._has_next_list = [] |
| 646 | |
| 647 | @staticmethod |
| 648 | def get_vertical(has_next_list): |
| 649 | string = '' |
| 650 | for v in has_next_list: |
| 651 | string += '%s ' % ('│' if v else ' ') |
| 652 | return string |
| 653 | |
| 654 | @staticmethod |
| 655 | def get_last_vertical(last): |
| 656 | return '└── ' if last else '├── ' |
| 657 | |
| 658 | def print_tree(self): |
| 659 | apex_map = self._provider.read_dir(self._path) |
| 660 | if apex_map is None: |
| 661 | return |
| 662 | apex_map = dict(apex_map) |
| 663 | if '.' in apex_map: |
| 664 | del apex_map['.'] |
| 665 | if '..' in apex_map: |
| 666 | del apex_map['..'] |
| 667 | key_list = list(sorted(apex_map.keys())) |
Andreas Gampe | 7dd0f0f | 2019-03-14 16:55:25 -0700 | [diff] [blame] | 668 | for i, key in enumerate(key_list): |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 669 | prev = self.get_vertical(self._has_next_list) |
| 670 | last = self.get_last_vertical(i == len(key_list) - 1) |
Andreas Gampe | 7dd0f0f | 2019-03-14 16:55:25 -0700 | [diff] [blame] | 671 | val = apex_map[key] |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 672 | print('%s%s%s' % (prev, last, val.name)) |
| 673 | if val.is_dir: |
| 674 | self._has_next_list.append(i < len(key_list) - 1) |
Andreas Gampe | 7dd0f0f | 2019-03-14 16:55:25 -0700 | [diff] [blame] | 675 | saved_dir = self._path |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 676 | self._path = os.path.join(self._path, val.name) |
| 677 | self.print_tree() |
Andreas Gampe | 7dd0f0f | 2019-03-14 16:55:25 -0700 | [diff] [blame] | 678 | self._path = saved_dir |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 679 | self._has_next_list.pop() |
| 680 | |
Andreas Gampe | b1d5567 | 2019-01-29 22:17:02 +0000 | [diff] [blame] | 681 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 682 | # Note: do not sys.exit early, for __del__ cleanup. |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 683 | def art_apex_test_main(test_args): |
| 684 | if test_args.tree and test_args.debug: |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 685 | logging.error("Both of --tree and --debug set") |
Andreas Gampe | b1d5567 | 2019-01-29 22:17:02 +0000 | [diff] [blame] | 686 | return 1 |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 687 | if test_args.list and test_args.debug: |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 688 | logging.error("Both of --list and --debug set") |
Andreas Gampe | b1d5567 | 2019-01-29 22:17:02 +0000 | [diff] [blame] | 689 | return 1 |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 690 | if test_args.list and test_args.tree: |
Andreas Gampe | b1d5567 | 2019-01-29 22:17:02 +0000 | [diff] [blame] | 691 | logging.error("Both of --list and --tree set") |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 692 | return 1 |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 693 | if not test_args.tmpdir: |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 694 | logging.error("Need a tmpdir.") |
| 695 | return 1 |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 696 | if not test_args.host and not test_args.debugfs: |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 697 | logging.error("Need debugfs.") |
| 698 | return 1 |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 699 | if test_args.bitness not in ['32', '64', 'multilib', 'auto']: |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 700 | logging.error('--bitness needs to be one of 32|64|multilib|auto') |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 701 | |
| 702 | try: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 703 | if test_args.host: |
| 704 | apex_provider = HostApexProvider(test_args.apex, test_args.tmpdir) |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 705 | else: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 706 | apex_provider = TargetApexProvider(test_args.apex, test_args.tmpdir, test_args.debugfs) |
| 707 | except (zipfile.BadZipFile, zipfile.LargeZipFile) as e: |
Andreas Gampe | a0242cf | 2019-01-29 13:01:23 -0800 | [diff] [blame] | 708 | logging.error('Failed to create provider: %s', e) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 709 | return 1 |
| 710 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 711 | if test_args.tree: |
| 712 | Tree(apex_provider, test_args.apex).print_tree() |
Andreas Gampe | b1d5567 | 2019-01-29 22:17:02 +0000 | [diff] [blame] | 713 | return 0 |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 714 | if test_args.list: |
| 715 | List(apex_provider).print_list() |
Andreas Gampe | b1d5567 | 2019-01-29 22:17:02 +0000 | [diff] [blame] | 716 | return 0 |
| 717 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 718 | checkers = [] |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 719 | if test_args.bitness == 'auto': |
| 720 | logging.warning('--bitness=auto, trying to autodetect. This may be incorrect!') |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 721 | has_32 = apex_provider.get('lib') is not None |
| 722 | has_64 = apex_provider.get('lib64') is not None |
| 723 | if has_32 and has_64: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 724 | logging.warning(' Detected multilib') |
| 725 | test_args.bitness = 'multilib' |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 726 | elif has_32: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 727 | logging.warning(' Detected 32-only') |
| 728 | test_args.bitness = '32' |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 729 | elif has_64: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 730 | logging.warning(' Detected 64-only') |
| 731 | test_args.bitness = '64' |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 732 | else: |
| 733 | logging.error(' Could not detect bitness, neither lib nor lib64 contained.') |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 734 | print('%s' % apex_provider.folder_cache) |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 735 | return 1 |
| 736 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 737 | if test_args.bitness == '32': |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 738 | base_checker = Arch32Checker(apex_provider) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 739 | elif test_args.bitness == '64': |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 740 | base_checker = Arch64Checker(apex_provider) |
| 741 | else: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 742 | assert test_args.bitness == 'multilib' |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 743 | base_checker = MultilibChecker(apex_provider) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 744 | |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 745 | checkers.append(ReleaseChecker(base_checker)) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 746 | if test_args.host: |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 747 | checkers.append(ReleaseHostChecker(base_checker)) |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 748 | else: |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 749 | checkers.append(ReleaseTargetChecker(base_checker)) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 750 | if test_args.debug: |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 751 | checkers.append(DebugChecker(base_checker)) |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 752 | if test_args.debug and not test_args.host: |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 753 | checkers.append(DebugTargetChecker(base_checker)) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 754 | |
Martin Stjernholm | 2babede | 2019-03-18 21:04:49 +0000 | [diff] [blame] | 755 | # These checkers must be last. |
| 756 | checkers.append(NoSuperfluousBinariesChecker(base_checker)) |
| 757 | if not test_args.host: |
| 758 | # We only care about superfluous libraries on target, where their absence |
| 759 | # can be vital to ensure they get picked up from the right package. |
| 760 | checkers.append(NoSuperfluousLibrariesChecker(base_checker)) |
| 761 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 762 | failed = False |
| 763 | for checker in checkers: |
| 764 | logging.info('%s...', checker) |
| 765 | checker.run() |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 766 | if base_checker.error_count() > 0: |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 767 | logging.error('%s FAILED', checker) |
| 768 | failed = True |
| 769 | else: |
| 770 | logging.info('%s SUCCEEDED', checker) |
Andreas Gampe | 9dc4b05 | 2019-01-30 13:47:25 -0800 | [diff] [blame] | 771 | base_checker.reset_errors() |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 772 | |
| 773 | return 1 if failed else 0 |
| 774 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 775 | |
| 776 | def art_apex_test_default(test_parser): |
| 777 | if 'ANDROID_PRODUCT_OUT' not in os.environ: |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 778 | logging.error('No-argument use requires ANDROID_PRODUCT_OUT') |
| 779 | sys.exit(1) |
| 780 | product_out = os.environ['ANDROID_PRODUCT_OUT'] |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 781 | if 'ANDROID_HOST_OUT' not in os.environ: |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 782 | logging.error('No-argument use requires ANDROID_HOST_OUT') |
| 783 | sys.exit(1) |
| 784 | host_out = os.environ['ANDROID_HOST_OUT'] |
| 785 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 786 | test_args = test_parser.parse_args(['dummy']) # For consistency. |
| 787 | test_args.debugfs = '%s/bin/debugfs' % host_out |
| 788 | test_args.tmpdir = '.' |
| 789 | test_args.tree = False |
| 790 | test_args.list = False |
| 791 | test_args.bitness = 'auto' |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 792 | failed = False |
| 793 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 794 | if not os.path.exists(test_args.debugfs): |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 795 | logging.error("Cannot find debugfs (default path %s). Please build it, e.g., m debugfs", |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 796 | test_args.debugfs) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 797 | sys.exit(1) |
| 798 | |
| 799 | # TODO: Add host support |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 800 | configs = [ |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 801 | {'name': 'com.android.runtime.release', 'debug': False, 'host': False}, |
| 802 | {'name': 'com.android.runtime.debug', 'debug': True, 'host': False}, |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 803 | ] |
| 804 | |
| 805 | for config in configs: |
| 806 | logging.info(config['name']) |
| 807 | # TODO: Host will need different path. |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 808 | test_args.apex = '%s/system/apex/%s.apex' % (product_out, config['name']) |
| 809 | if not os.path.exists(test_args.apex): |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 810 | failed = True |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 811 | logging.error("Cannot find APEX %s. Please build it first.", test_args.apex) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 812 | continue |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 813 | test_args.debug = config['debug'] |
| 814 | test_args.host = config['host'] |
| 815 | failed = art_apex_test_main(test_args) != 0 |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 816 | |
| 817 | if failed: |
| 818 | sys.exit(1) |
| 819 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 820 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 821 | if __name__ == "__main__": |
| 822 | parser = argparse.ArgumentParser(description='Check integrity of a Runtime APEX.') |
| 823 | |
| 824 | parser.add_argument('apex', help='apex file input') |
| 825 | |
| 826 | parser.add_argument('--host', help='Check as host apex', action='store_true') |
Andreas Gampe | 0912395 | 2019-01-30 13:17:02 -0800 | [diff] [blame] | 827 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 828 | parser.add_argument('--debug', help='Check as debug apex', action='store_true') |
| 829 | |
Andreas Gampe | b1d5567 | 2019-01-29 22:17:02 +0000 | [diff] [blame] | 830 | parser.add_argument('--list', help='List all files', action='store_true') |
| 831 | parser.add_argument('--tree', help='Print directory tree', action='store_true') |
| 832 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 833 | parser.add_argument('--tmpdir', help='Directory for temp files') |
| 834 | parser.add_argument('--debugfs', help='Path to debugfs') |
| 835 | |
Andreas Gampe | eb555b8 | 2019-01-30 14:47:49 -0800 | [diff] [blame] | 836 | parser.add_argument('--bitness', help='Bitness to check, 32|64|multilib|auto', default='auto') |
| 837 | |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 838 | if len(sys.argv) == 1: |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 839 | art_apex_test_default(parser) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 840 | else: |
| 841 | args = parser.parse_args() |
| 842 | |
| 843 | if args is None: |
| 844 | sys.exit(1) |
| 845 | |
Luca Stefani | 4e91ee9 | 2019-03-06 15:08:16 +0100 | [diff] [blame] | 846 | exit_code = art_apex_test_main(args) |
Andreas Gampe | 2c846cd | 2019-01-29 22:06:46 +0000 | [diff] [blame] | 847 | sys.exit(exit_code) |