Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
| 2 | '''Test the program psa_constant_names. |
| 3 | Gather constant names from header files and test cases. Compile a C program |
| 4 | to print out their numerical values, feed these numerical values to |
| 5 | psa_constant_names, and check that the output is the original name. |
| 6 | Return 0 if all test cases pass, 1 if the output was not always as expected, |
| 7 | or 1 (with a Python backtrace) if there was an operational error.''' |
| 8 | |
| 9 | import argparse |
| 10 | import itertools |
| 11 | import os |
| 12 | import platform |
| 13 | import re |
| 14 | import subprocess |
| 15 | import sys |
| 16 | import tempfile |
| 17 | |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 18 | class ReadFileLineException(Exception): |
| 19 | def __init__(self, filename, line_number): |
| 20 | message = 'in {} at {}'.format(filename, line_number) |
| 21 | super(ReadFileLineException, self).__init__(message) |
| 22 | self.filename = filename |
| 23 | self.line_number = line_number |
| 24 | |
| 25 | class read_file_lines: |
| 26 | '''Context manager to read a text file line by line. |
| 27 | with read_file_lines(filename) as lines: |
| 28 | for line in lines: |
| 29 | process(line) |
| 30 | is equivalent to |
| 31 | with open(filename, 'r') as input_file: |
| 32 | for line in input_file: |
| 33 | process(line) |
| 34 | except that if process(line) raises an exception, then the read_file_lines |
| 35 | snippet annotates the exception with the file name and line number.''' |
| 36 | def __init__(self, filename): |
| 37 | self.filename = filename |
| 38 | self.line_number = 'entry' |
| 39 | def __enter__(self): |
| 40 | self.generator = enumerate(open(self.filename, 'r')) |
| 41 | return self |
| 42 | def __iter__(self): |
| 43 | for line_number, content in self.generator: |
| 44 | self.line_number = line_number |
| 45 | yield content |
| 46 | self.line_number = 'exit' |
| 47 | def __exit__(self, type, value, traceback): |
| 48 | if type is not None: |
| 49 | raise ReadFileLineException(self.filename, self.line_number) \ |
| 50 | from value |
| 51 | |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 52 | class Inputs: |
| 53 | '''Accumulate information about macros to test. |
| 54 | This includes macro names as well as information about their arguments |
| 55 | when applicable.''' |
| 56 | def __init__(self): |
| 57 | # Sets of names per type |
| 58 | self.statuses = set(['PSA_SUCCESS']) |
| 59 | self.algorithms = set(['0xffffffff']) |
| 60 | self.ecc_curves = set(['0xffff']) |
| 61 | self.key_types = set(['0xffffffff']) |
| 62 | self.key_usage_flags = set(['0x80000000']) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 63 | # Hard-coded value for unknown algorithms |
Darryl Green | 61b7f61 | 2019-02-04 16:00:21 +0000 | [diff] [blame] | 64 | self.hash_algorithms = set(['0x010000fe']) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 65 | self.mac_algorithms = set(['0x02ff00ff']) |
Gilles Peskine | 882e57e | 2019-04-12 00:12:07 +0200 | [diff] [blame^] | 66 | self.ka_algorithms = set(['0x30fc0000']) |
| 67 | self.kdf_algorithms = set(['0x200000ff']) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 68 | # For AEAD algorithms, the only variability is over the tag length, |
| 69 | # and this only applies to known algorithms, so don't test an |
| 70 | # unknown algorithm. |
| 71 | self.aead_algorithms = set() |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 72 | # Identifier prefixes |
| 73 | self.table_by_prefix = { |
| 74 | 'ERROR': self.statuses, |
| 75 | 'ALG': self.algorithms, |
| 76 | 'CURVE': self.ecc_curves, |
| 77 | 'KEY_TYPE': self.key_types, |
| 78 | 'KEY_USAGE': self.key_usage_flags, |
| 79 | } |
| 80 | # macro name -> list of argument names |
| 81 | self.argspecs = {} |
| 82 | # argument name -> list of values |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 83 | self.arguments_for = { |
| 84 | 'mac_length': ['1', '63'], |
| 85 | 'tag_length': ['1', '63'], |
| 86 | } |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 87 | |
| 88 | def gather_arguments(self): |
| 89 | '''Populate the list of values for macro arguments. |
| 90 | Call this after parsing all the inputs.''' |
| 91 | self.arguments_for['hash_alg'] = sorted(self.hash_algorithms) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 92 | self.arguments_for['mac_alg'] = sorted(self.mac_algorithms) |
Gilles Peskine | 882e57e | 2019-04-12 00:12:07 +0200 | [diff] [blame^] | 93 | self.arguments_for['ka_alg'] = sorted(self.ka_algorithms) |
Gilles Peskine | 1754208 | 2019-01-04 19:46:31 +0100 | [diff] [blame] | 94 | self.arguments_for['kdf_alg'] = sorted(self.kdf_algorithms) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 95 | self.arguments_for['aead_alg'] = sorted(self.aead_algorithms) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 96 | self.arguments_for['curve'] = sorted(self.ecc_curves) |
| 97 | |
| 98 | def format_arguments(self, name, arguments): |
| 99 | '''Format a macro call with arguments..''' |
| 100 | return name + '(' + ', '.join(arguments) + ')' |
| 101 | |
| 102 | def distribute_arguments(self, name): |
| 103 | '''Generate macro calls with each tested argument set. |
| 104 | If name is a macro without arguments, just yield "name". |
| 105 | If name is a macro with arguments, yield a series of "name(arg1,...,argN)" |
| 106 | where each argument takes each possible value at least once.''' |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 107 | try: |
| 108 | if name not in self.argspecs: |
| 109 | yield name |
| 110 | return |
| 111 | argspec = self.argspecs[name] |
| 112 | if argspec == []: |
| 113 | yield name + '()' |
| 114 | return |
| 115 | argument_lists = [self.arguments_for[arg] for arg in argspec] |
| 116 | arguments = [values[0] for values in argument_lists] |
| 117 | yield self.format_arguments(name, arguments) |
| 118 | for i in range(len(arguments)): |
| 119 | for value in argument_lists[i][1:]: |
| 120 | arguments[i] = value |
| 121 | yield self.format_arguments(name, arguments) |
Gilles Peskine | f96ed66 | 2018-10-19 11:29:56 +0200 | [diff] [blame] | 122 | arguments[i] = argument_lists[0][0] |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 123 | except BaseException as e: |
| 124 | raise Exception('distribute_arguments({})'.format(name)) from e |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 125 | |
| 126 | # Regex for interesting header lines. |
| 127 | # Groups: 1=macro name, 2=type, 3=argument list (optional). |
| 128 | header_line_re = \ |
| 129 | re.compile(r'#define +' + |
| 130 | r'(PSA_((?:KEY_)?[A-Z]+)_\w+)' + |
| 131 | r'(?:\(([^\n()]*)\))?') |
| 132 | # Regex of macro names to exclude. |
| 133 | excluded_name_re = re.compile('_(?:GET|IS|OF)_|_(?:BASE|FLAG|MASK)\Z') |
Gilles Peskine | c68ce96 | 2018-10-19 11:31:52 +0200 | [diff] [blame] | 134 | # Additional excluded macros. |
Darryl Green | b8fe068 | 2019-02-06 13:21:31 +0000 | [diff] [blame] | 135 | # PSA_ALG_ECDH and PSA_ALG_FFDH are excluded for now as the script |
Jaeden Amero | 5e6d24c | 2019-02-21 10:41:29 +0000 | [diff] [blame] | 136 | # currently doesn't support them. Deprecated errors are also excluded. |
Gilles Peskine | c68ce96 | 2018-10-19 11:31:52 +0200 | [diff] [blame] | 137 | excluded_names = set(['PSA_ALG_AEAD_WITH_DEFAULT_TAG_LENGTH', |
Darryl Green | ec07950 | 2019-01-29 15:48:00 +0000 | [diff] [blame] | 138 | 'PSA_ALG_FULL_LENGTH_MAC', |
| 139 | 'PSA_ALG_ECDH', |
Jaeden Amero | 5e6d24c | 2019-02-21 10:41:29 +0000 | [diff] [blame] | 140 | 'PSA_ALG_FFDH', |
| 141 | 'PSA_ERROR_UNKNOWN_ERROR', |
| 142 | 'PSA_ERROR_OCCUPIED_SLOT', |
| 143 | 'PSA_ERROR_EMPTY_SLOT', |
| 144 | 'PSA_ERROR_INSUFFICIENT_CAPACITY', |
| 145 | ]) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 146 | argument_split_re = re.compile(r' *, *') |
| 147 | def parse_header_line(self, line): |
| 148 | '''Parse a C header line, looking for "#define PSA_xxx".''' |
| 149 | m = re.match(self.header_line_re, line) |
| 150 | if not m: |
| 151 | return |
| 152 | name = m.group(1) |
Gilles Peskine | c68ce96 | 2018-10-19 11:31:52 +0200 | [diff] [blame] | 153 | if re.search(self.excluded_name_re, name) or \ |
| 154 | name in self.excluded_names: |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 155 | return |
| 156 | dest = self.table_by_prefix.get(m.group(2)) |
| 157 | if dest is None: |
| 158 | return |
| 159 | dest.add(name) |
| 160 | if m.group(3): |
| 161 | self.argspecs[name] = re.split(self.argument_split_re, m.group(3)) |
| 162 | |
| 163 | def parse_header(self, filename): |
| 164 | '''Parse a C header file, looking for "#define PSA_xxx".''' |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 165 | with read_file_lines(filename) as lines: |
| 166 | for line in lines: |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 167 | self.parse_header_line(line) |
| 168 | |
| 169 | def add_test_case_line(self, function, argument): |
| 170 | '''Parse a test case data line, looking for algorithm metadata tests.''' |
| 171 | if function.endswith('_algorithm'): |
Darryl Green | b8fe068 | 2019-02-06 13:21:31 +0000 | [diff] [blame] | 172 | # As above, ECDH and FFDH algorithms are excluded for now. |
| 173 | # Support for them will be added in the future. |
Darryl Green | ec07950 | 2019-01-29 15:48:00 +0000 | [diff] [blame] | 174 | if 'ECDH' in argument or 'FFDH' in argument: |
| 175 | return |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 176 | self.algorithms.add(argument) |
| 177 | if function == 'hash_algorithm': |
| 178 | self.hash_algorithms.add(argument) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 179 | elif function in ['mac_algorithm', 'hmac_algorithm']: |
| 180 | self.mac_algorithms.add(argument) |
| 181 | elif function == 'aead_algorithm': |
| 182 | self.aead_algorithms.add(argument) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 183 | elif function == 'key_type': |
| 184 | self.key_types.add(argument) |
| 185 | elif function == 'ecc_key_types': |
| 186 | self.ecc_curves.add(argument) |
| 187 | |
| 188 | # Regex matching a *.data line containing a test function call and |
| 189 | # its arguments. The actual definition is partly positional, but this |
| 190 | # regex is good enough in practice. |
| 191 | test_case_line_re = re.compile('(?!depends_on:)(\w+):([^\n :][^:\n]*)') |
| 192 | def parse_test_cases(self, filename): |
| 193 | '''Parse a test case file (*.data), looking for algorithm metadata tests.''' |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 194 | with read_file_lines(filename) as lines: |
| 195 | for line in lines: |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 196 | m = re.match(self.test_case_line_re, line) |
| 197 | if m: |
| 198 | self.add_test_case_line(m.group(1), m.group(2)) |
| 199 | |
| 200 | def gather_inputs(headers, test_suites): |
| 201 | '''Read the list of inputs to test psa_constant_names with.''' |
| 202 | inputs = Inputs() |
| 203 | for header in headers: |
| 204 | inputs.parse_header(header) |
| 205 | for test_cases in test_suites: |
| 206 | inputs.parse_test_cases(test_cases) |
| 207 | inputs.gather_arguments() |
| 208 | return inputs |
| 209 | |
| 210 | def remove_file_if_exists(filename): |
| 211 | '''Remove the specified file, ignoring errors.''' |
| 212 | if not filename: |
| 213 | return |
| 214 | try: |
| 215 | os.remove(filename) |
| 216 | except: |
| 217 | pass |
| 218 | |
Gilles Peskine | cf9c18e | 2018-10-19 11:28:42 +0200 | [diff] [blame] | 219 | def run_c(options, type, names): |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 220 | '''Generate and run a program to print out numerical values for names.''' |
Gilles Peskine | c4cd2ad | 2019-02-13 18:42:53 +0100 | [diff] [blame] | 221 | if type == 'status': |
| 222 | cast_to = 'long' |
| 223 | printf_format = '%ld' |
| 224 | else: |
| 225 | cast_to = 'unsigned long' |
| 226 | printf_format = '0x%08lx' |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 227 | c_name = None |
| 228 | exe_name = None |
| 229 | try: |
Gilles Peskine | 95ab71a | 2019-01-04 19:46:59 +0100 | [diff] [blame] | 230 | c_fd, c_name = tempfile.mkstemp(prefix='tmp-{}-'.format(type), |
| 231 | suffix='.c', |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 232 | dir='programs/psa') |
| 233 | exe_suffix = '.exe' if platform.system() == 'Windows' else '' |
| 234 | exe_name = c_name[:-2] + exe_suffix |
| 235 | remove_file_if_exists(exe_name) |
| 236 | c_file = os.fdopen(c_fd, 'w', encoding='ascii') |
Gilles Peskine | 95ab71a | 2019-01-04 19:46:59 +0100 | [diff] [blame] | 237 | c_file.write('/* Generated by test_psa_constant_names.py for {} values */' |
| 238 | .format(type)) |
| 239 | c_file.write(''' |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 240 | #include <stdio.h> |
| 241 | #include <psa/crypto.h> |
| 242 | int main(void) |
| 243 | { |
| 244 | ''') |
| 245 | for name in names: |
Gilles Peskine | c4cd2ad | 2019-02-13 18:42:53 +0100 | [diff] [blame] | 246 | c_file.write(' printf("{}\\n", ({}) {});\n' |
| 247 | .format(printf_format, cast_to, name)) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 248 | c_file.write(''' return 0; |
| 249 | } |
| 250 | ''') |
| 251 | c_file.close() |
| 252 | cc = os.getenv('CC', 'cc') |
| 253 | subprocess.check_call([cc] + |
| 254 | ['-I' + dir for dir in options.include] + |
| 255 | ['-o', exe_name, c_name]) |
Gilles Peskine | cf9c18e | 2018-10-19 11:28:42 +0200 | [diff] [blame] | 256 | if options.keep_c: |
| 257 | sys.stderr.write('List of {} tests kept at {}\n' |
| 258 | .format(type, c_name)) |
| 259 | else: |
| 260 | os.remove(c_name) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 261 | output = subprocess.check_output([exe_name]) |
| 262 | return output.decode('ascii').strip().split('\n') |
| 263 | finally: |
| 264 | remove_file_if_exists(exe_name) |
| 265 | |
| 266 | normalize_strip_re = re.compile(r'\s+') |
| 267 | def normalize(expr): |
| 268 | '''Normalize the C expression so as not to care about trivial differences. |
| 269 | Currently "trivial differences" means whitespace.''' |
| 270 | expr = re.sub(normalize_strip_re, '', expr, len(expr)) |
| 271 | return expr.strip().split('\n') |
| 272 | |
| 273 | def do_test(options, inputs, type, names): |
| 274 | '''Test psa_constant_names for the specified type. |
| 275 | Run program on names. |
| 276 | Use inputs to figure out what arguments to pass to macros that take arguments.''' |
| 277 | names = sorted(itertools.chain(*map(inputs.distribute_arguments, names))) |
Gilles Peskine | cf9c18e | 2018-10-19 11:28:42 +0200 | [diff] [blame] | 278 | values = run_c(options, type, names) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 279 | output = subprocess.check_output([options.program, type] + values) |
| 280 | outputs = output.decode('ascii').strip().split('\n') |
| 281 | errors = [(type, name, value, output) |
| 282 | for (name, value, output) in zip(names, values, outputs) |
| 283 | if normalize(name) != normalize(output)] |
| 284 | return len(names), errors |
| 285 | |
| 286 | def report_errors(errors): |
| 287 | '''Describe each case where the output is not as expected.''' |
| 288 | for type, name, value, output in errors: |
| 289 | print('For {} "{}", got "{}" (value: {})' |
| 290 | .format(type, name, output, value)) |
| 291 | |
| 292 | def run_tests(options, inputs): |
| 293 | '''Run psa_constant_names on all the gathered inputs. |
| 294 | Return a tuple (count, errors) where count is the total number of inputs |
| 295 | that were tested and errors is the list of cases where the output was |
| 296 | not as expected.''' |
| 297 | count = 0 |
| 298 | errors = [] |
| 299 | for type, names in [('status', inputs.statuses), |
| 300 | ('algorithm', inputs.algorithms), |
| 301 | ('ecc_curve', inputs.ecc_curves), |
| 302 | ('key_type', inputs.key_types), |
| 303 | ('key_usage', inputs.key_usage_flags)]: |
| 304 | c, e = do_test(options, inputs, type, names) |
| 305 | count += c |
| 306 | errors += e |
| 307 | return count, errors |
| 308 | |
| 309 | if __name__ == '__main__': |
| 310 | parser = argparse.ArgumentParser(description=globals()['__doc__']) |
| 311 | parser.add_argument('--include', '-I', |
| 312 | action='append', default=['include'], |
| 313 | help='Directory for header files') |
| 314 | parser.add_argument('--program', |
| 315 | default='programs/psa/psa_constant_names', |
| 316 | help='Program to test') |
Gilles Peskine | cf9c18e | 2018-10-19 11:28:42 +0200 | [diff] [blame] | 317 | parser.add_argument('--keep-c', |
| 318 | action='store_true', dest='keep_c', default=False, |
| 319 | help='Keep the intermediate C file') |
| 320 | parser.add_argument('--no-keep-c', |
| 321 | action='store_false', dest='keep_c', |
| 322 | help='Don\'t keep the intermediate C file (default)') |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 323 | options = parser.parse_args() |
Gilles Peskine | 6d194bd | 2019-01-04 19:44:59 +0100 | [diff] [blame] | 324 | headers = [os.path.join(options.include[0], 'psa', h) |
| 325 | for h in ['crypto.h', 'crypto_extra.h', 'crypto_values.h']] |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 326 | test_suites = ['tests/suites/test_suite_psa_crypto_metadata.data'] |
| 327 | inputs = gather_inputs(headers, test_suites) |
| 328 | count, errors = run_tests(options, inputs) |
| 329 | report_errors(errors) |
| 330 | if errors == []: |
| 331 | print('{} test cases PASS'.format(count)) |
| 332 | else: |
| 333 | print('{} test cases, {} FAIL'.format(count, len(errors))) |
| 334 | exit(1) |