Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 2 | """Test the program psa_constant_names. |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 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, |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 7 | or 1 (with a Python backtrace) if there was an operational error. |
| 8 | """ |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 9 | |
Bence Szépkúti | 700ee44 | 2020-05-26 00:33:31 +0200 | [diff] [blame^] | 10 | # Copyright (C) 2018-2020, Arm Limited, All Rights Reserved |
| 11 | # |
| 12 | # This file is part of Mbed TLS (https://tls.mbed.org) |
| 13 | |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 14 | import argparse |
Gilles Peskine | a5000f1 | 2019-11-21 17:51:11 +0100 | [diff] [blame] | 15 | from collections import namedtuple |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 16 | import itertools |
| 17 | import os |
| 18 | import platform |
| 19 | import re |
| 20 | import subprocess |
| 21 | import sys |
| 22 | import tempfile |
| 23 | |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 24 | class ReadFileLineException(Exception): |
| 25 | def __init__(self, filename, line_number): |
| 26 | message = 'in {} at {}'.format(filename, line_number) |
| 27 | super(ReadFileLineException, self).__init__(message) |
| 28 | self.filename = filename |
| 29 | self.line_number = line_number |
| 30 | |
| 31 | class read_file_lines: |
Gilles Peskine | 54f5445 | 2019-05-27 18:31:59 +0200 | [diff] [blame] | 32 | # Dear Pylint, conventionally, a context manager class name is lowercase. |
| 33 | # pylint: disable=invalid-name,too-few-public-methods |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 34 | """Context manager to read a text file line by line. |
| 35 | |
| 36 | ``` |
| 37 | with read_file_lines(filename) as lines: |
| 38 | for line in lines: |
| 39 | process(line) |
| 40 | ``` |
| 41 | is equivalent to |
| 42 | ``` |
| 43 | with open(filename, 'r') as input_file: |
| 44 | for line in input_file: |
| 45 | process(line) |
| 46 | ``` |
| 47 | except that if process(line) raises an exception, then the read_file_lines |
| 48 | snippet annotates the exception with the file name and line number. |
| 49 | """ |
Gilles Peskine | 49af2d3 | 2019-12-06 19:20:13 +0100 | [diff] [blame] | 50 | def __init__(self, filename, binary=False): |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 51 | self.filename = filename |
| 52 | self.line_number = 'entry' |
Gilles Peskine | 54f5445 | 2019-05-27 18:31:59 +0200 | [diff] [blame] | 53 | self.generator = None |
Gilles Peskine | 49af2d3 | 2019-12-06 19:20:13 +0100 | [diff] [blame] | 54 | self.binary = binary |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 55 | def __enter__(self): |
Gilles Peskine | 49af2d3 | 2019-12-06 19:20:13 +0100 | [diff] [blame] | 56 | self.generator = enumerate(open(self.filename, |
| 57 | 'rb' if self.binary else 'r')) |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 58 | return self |
| 59 | def __iter__(self): |
| 60 | for line_number, content in self.generator: |
| 61 | self.line_number = line_number |
| 62 | yield content |
| 63 | self.line_number = 'exit' |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 64 | def __exit__(self, exc_type, exc_value, exc_traceback): |
| 65 | if exc_type is not None: |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 66 | raise ReadFileLineException(self.filename, self.line_number) \ |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 67 | from exc_value |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 68 | |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 69 | class Inputs: |
Gilles Peskine | 8c8694c | 2019-11-21 19:22:45 +0100 | [diff] [blame] | 70 | # pylint: disable=too-many-instance-attributes |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 71 | """Accumulate information about macros to test. |
Gilles Peskine | 4408dfd | 2019-11-21 17:16:21 +0100 | [diff] [blame] | 72 | |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 73 | This includes macro names as well as information about their arguments |
| 74 | when applicable. |
| 75 | """ |
| 76 | |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 77 | def __init__(self): |
Gilles Peskine | 2bcfc71 | 2019-11-21 19:49:26 +0100 | [diff] [blame] | 78 | self.all_declared = set() |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 79 | # Sets of names per type |
| 80 | self.statuses = set(['PSA_SUCCESS']) |
| 81 | self.algorithms = set(['0xffffffff']) |
Gilles Peskine | f65ed6f | 2019-12-04 17:18:41 +0100 | [diff] [blame] | 82 | self.ecc_curves = set(['0xff']) |
| 83 | self.dh_groups = set(['0xff']) |
| 84 | self.key_types = set(['0xffff']) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 85 | self.key_usage_flags = set(['0x80000000']) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 86 | # Hard-coded value for unknown algorithms |
Darryl Green | 61b7f61 | 2019-02-04 16:00:21 +0000 | [diff] [blame] | 87 | self.hash_algorithms = set(['0x010000fe']) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 88 | self.mac_algorithms = set(['0x02ff00ff']) |
Gilles Peskine | 882e57e | 2019-04-12 00:12:07 +0200 | [diff] [blame] | 89 | self.ka_algorithms = set(['0x30fc0000']) |
| 90 | self.kdf_algorithms = set(['0x200000ff']) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 91 | # For AEAD algorithms, the only variability is over the tag length, |
| 92 | # and this only applies to known algorithms, so don't test an |
| 93 | # unknown algorithm. |
| 94 | self.aead_algorithms = set() |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 95 | # Identifier prefixes |
| 96 | self.table_by_prefix = { |
| 97 | 'ERROR': self.statuses, |
| 98 | 'ALG': self.algorithms, |
Gilles Peskine | 98a710c | 2019-11-21 18:58:36 +0100 | [diff] [blame] | 99 | 'ECC_CURVE': self.ecc_curves, |
| 100 | 'DH_GROUP': self.dh_groups, |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 101 | 'KEY_TYPE': self.key_types, |
| 102 | 'KEY_USAGE': self.key_usage_flags, |
| 103 | } |
Gilles Peskine | 8c8694c | 2019-11-21 19:22:45 +0100 | [diff] [blame] | 104 | # Test functions |
| 105 | self.table_by_test_function = { |
Gilles Peskine | 8fa1348 | 2019-11-25 17:10:12 +0100 | [diff] [blame] | 106 | # Any function ending in _algorithm also gets added to |
| 107 | # self.algorithms. |
| 108 | 'key_type': [self.key_types], |
Gilles Peskine | f8210f2 | 2019-12-02 17:26:44 +0100 | [diff] [blame] | 109 | 'block_cipher_key_type': [self.key_types], |
| 110 | 'stream_cipher_key_type': [self.key_types], |
Gilles Peskine | 228abc5 | 2019-12-03 17:24:19 +0100 | [diff] [blame] | 111 | 'ecc_key_family': [self.ecc_curves], |
Gilles Peskine | 8fa1348 | 2019-11-25 17:10:12 +0100 | [diff] [blame] | 112 | 'ecc_key_types': [self.ecc_curves], |
Gilles Peskine | 228abc5 | 2019-12-03 17:24:19 +0100 | [diff] [blame] | 113 | 'dh_key_family': [self.dh_groups], |
Gilles Peskine | 8fa1348 | 2019-11-25 17:10:12 +0100 | [diff] [blame] | 114 | 'dh_key_types': [self.dh_groups], |
| 115 | 'hash_algorithm': [self.hash_algorithms], |
| 116 | 'mac_algorithm': [self.mac_algorithms], |
| 117 | 'cipher_algorithm': [], |
| 118 | 'hmac_algorithm': [self.mac_algorithms], |
| 119 | 'aead_algorithm': [self.aead_algorithms], |
| 120 | 'key_derivation_algorithm': [self.kdf_algorithms], |
| 121 | 'key_agreement_algorithm': [self.ka_algorithms], |
| 122 | 'asymmetric_signature_algorithm': [], |
| 123 | 'asymmetric_signature_wildcard': [self.algorithms], |
| 124 | 'asymmetric_encryption_algorithm': [], |
| 125 | 'other_algorithm': [], |
Gilles Peskine | 8c8694c | 2019-11-21 19:22:45 +0100 | [diff] [blame] | 126 | } |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 127 | # macro name -> list of argument names |
| 128 | self.argspecs = {} |
| 129 | # argument name -> list of values |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 130 | self.arguments_for = { |
| 131 | 'mac_length': ['1', '63'], |
| 132 | 'tag_length': ['1', '63'], |
| 133 | } |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 134 | |
Gilles Peskine | ffe2d6e | 2019-11-21 17:17:01 +0100 | [diff] [blame] | 135 | def get_names(self, type_word): |
| 136 | """Return the set of known names of values of the given type.""" |
| 137 | return { |
| 138 | 'status': self.statuses, |
| 139 | 'algorithm': self.algorithms, |
| 140 | 'ecc_curve': self.ecc_curves, |
| 141 | 'dh_group': self.dh_groups, |
| 142 | 'key_type': self.key_types, |
| 143 | 'key_usage': self.key_usage_flags, |
| 144 | }[type_word] |
| 145 | |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 146 | def gather_arguments(self): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 147 | """Populate the list of values for macro arguments. |
Gilles Peskine | 4408dfd | 2019-11-21 17:16:21 +0100 | [diff] [blame] | 148 | |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 149 | Call this after parsing all the inputs. |
| 150 | """ |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 151 | self.arguments_for['hash_alg'] = sorted(self.hash_algorithms) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 152 | self.arguments_for['mac_alg'] = sorted(self.mac_algorithms) |
Gilles Peskine | 882e57e | 2019-04-12 00:12:07 +0200 | [diff] [blame] | 153 | self.arguments_for['ka_alg'] = sorted(self.ka_algorithms) |
Gilles Peskine | 1754208 | 2019-01-04 19:46:31 +0100 | [diff] [blame] | 154 | self.arguments_for['kdf_alg'] = sorted(self.kdf_algorithms) |
Gilles Peskine | 434899f | 2018-10-19 11:30:26 +0200 | [diff] [blame] | 155 | self.arguments_for['aead_alg'] = sorted(self.aead_algorithms) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 156 | self.arguments_for['curve'] = sorted(self.ecc_curves) |
Gilles Peskine | dcaefae | 2019-05-16 12:55:35 +0200 | [diff] [blame] | 157 | self.arguments_for['group'] = sorted(self.dh_groups) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 158 | |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 159 | @staticmethod |
| 160 | def _format_arguments(name, arguments): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 161 | """Format a macro call with arguments..""" |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 162 | return name + '(' + ', '.join(arguments) + ')' |
| 163 | |
| 164 | def distribute_arguments(self, name): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 165 | """Generate macro calls with each tested argument set. |
Gilles Peskine | 4408dfd | 2019-11-21 17:16:21 +0100 | [diff] [blame] | 166 | |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 167 | If name is a macro without arguments, just yield "name". |
| 168 | If name is a macro with arguments, yield a series of |
| 169 | "name(arg1,...,argN)" where each argument takes each possible |
| 170 | value at least once. |
| 171 | """ |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 172 | try: |
| 173 | if name not in self.argspecs: |
| 174 | yield name |
| 175 | return |
| 176 | argspec = self.argspecs[name] |
| 177 | if argspec == []: |
| 178 | yield name + '()' |
| 179 | return |
| 180 | argument_lists = [self.arguments_for[arg] for arg in argspec] |
| 181 | arguments = [values[0] for values in argument_lists] |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 182 | yield self._format_arguments(name, arguments) |
Gilles Peskine | 54f5445 | 2019-05-27 18:31:59 +0200 | [diff] [blame] | 183 | # Dear Pylint, enumerate won't work here since we're modifying |
| 184 | # the array. |
| 185 | # pylint: disable=consider-using-enumerate |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 186 | for i in range(len(arguments)): |
| 187 | for value in argument_lists[i][1:]: |
| 188 | arguments[i] = value |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 189 | yield self._format_arguments(name, arguments) |
Gilles Peskine | f96ed66 | 2018-10-19 11:29:56 +0200 | [diff] [blame] | 190 | arguments[i] = argument_lists[0][0] |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 191 | except BaseException as e: |
| 192 | raise Exception('distribute_arguments({})'.format(name)) from e |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 193 | |
Gilles Peskine | 5a994c1 | 2019-11-21 16:46:51 +0100 | [diff] [blame] | 194 | def generate_expressions(self, names): |
| 195 | return itertools.chain(*map(self.distribute_arguments, names)) |
| 196 | |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 197 | _argument_split_re = re.compile(r' *, *') |
| 198 | @classmethod |
| 199 | def _argument_split(cls, arguments): |
| 200 | return re.split(cls._argument_split_re, arguments) |
| 201 | |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 202 | # Regex for interesting header lines. |
| 203 | # Groups: 1=macro name, 2=type, 3=argument list (optional). |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 204 | _header_line_re = \ |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 205 | re.compile(r'#define +' + |
Gilles Peskine | 98a710c | 2019-11-21 18:58:36 +0100 | [diff] [blame] | 206 | r'(PSA_((?:(?:DH|ECC|KEY)_)?[A-Z]+)_\w+)' + |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 207 | r'(?:\(([^\n()]*)\))?') |
| 208 | # Regex of macro names to exclude. |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 209 | _excluded_name_re = re.compile(r'_(?:GET|IS|OF)_|_(?:BASE|FLAG|MASK)\Z') |
Gilles Peskine | c68ce96 | 2018-10-19 11:31:52 +0200 | [diff] [blame] | 210 | # Additional excluded macros. |
Gilles Peskine | 5c196fb | 2019-05-17 12:04:41 +0200 | [diff] [blame] | 211 | _excluded_names = set([ |
| 212 | # Macros that provide an alternative way to build the same |
| 213 | # algorithm as another macro. |
| 214 | 'PSA_ALG_AEAD_WITH_DEFAULT_TAG_LENGTH', |
| 215 | 'PSA_ALG_FULL_LENGTH_MAC', |
| 216 | # Auxiliary macro whose name doesn't fit the usual patterns for |
| 217 | # auxiliary macros. |
| 218 | 'PSA_ALG_AEAD_WITH_DEFAULT_TAG_LENGTH_CASE', |
Gilles Peskine | 5c196fb | 2019-05-17 12:04:41 +0200 | [diff] [blame] | 219 | ]) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 220 | def parse_header_line(self, line): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 221 | """Parse a C header line, looking for "#define PSA_xxx".""" |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 222 | m = re.match(self._header_line_re, line) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 223 | if not m: |
| 224 | return |
| 225 | name = m.group(1) |
Gilles Peskine | 2bcfc71 | 2019-11-21 19:49:26 +0100 | [diff] [blame] | 226 | self.all_declared.add(name) |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 227 | if re.search(self._excluded_name_re, name) or \ |
| 228 | name in self._excluded_names: |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 229 | return |
| 230 | dest = self.table_by_prefix.get(m.group(2)) |
| 231 | if dest is None: |
| 232 | return |
| 233 | dest.add(name) |
| 234 | if m.group(3): |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 235 | self.argspecs[name] = self._argument_split(m.group(3)) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 236 | |
Gilles Peskine | 49af2d3 | 2019-12-06 19:20:13 +0100 | [diff] [blame] | 237 | _nonascii_re = re.compile(rb'[^\x00-\x7f]+') |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 238 | def parse_header(self, filename): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 239 | """Parse a C header file, looking for "#define PSA_xxx".""" |
Gilles Peskine | 49af2d3 | 2019-12-06 19:20:13 +0100 | [diff] [blame] | 240 | with read_file_lines(filename, binary=True) as lines: |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 241 | for line in lines: |
Gilles Peskine | 49af2d3 | 2019-12-06 19:20:13 +0100 | [diff] [blame] | 242 | line = re.sub(self._nonascii_re, rb'', line).decode('ascii') |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 243 | self.parse_header_line(line) |
| 244 | |
Gilles Peskine | 49af2d3 | 2019-12-06 19:20:13 +0100 | [diff] [blame] | 245 | _macro_identifier_re = re.compile(r'[A-Z]\w+') |
Gilles Peskine | 2bcfc71 | 2019-11-21 19:49:26 +0100 | [diff] [blame] | 246 | def generate_undeclared_names(self, expr): |
| 247 | for name in re.findall(self._macro_identifier_re, expr): |
| 248 | if name not in self.all_declared: |
| 249 | yield name |
| 250 | |
| 251 | def accept_test_case_line(self, function, argument): |
| 252 | #pylint: disable=unused-argument |
| 253 | undeclared = list(self.generate_undeclared_names(argument)) |
| 254 | if undeclared: |
| 255 | raise Exception('Undeclared names in test case', undeclared) |
| 256 | return True |
| 257 | |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 258 | def add_test_case_line(self, function, argument): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 259 | """Parse a test case data line, looking for algorithm metadata tests.""" |
Gilles Peskine | 8c8694c | 2019-11-21 19:22:45 +0100 | [diff] [blame] | 260 | sets = [] |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 261 | if function.endswith('_algorithm'): |
Gilles Peskine | 8c8694c | 2019-11-21 19:22:45 +0100 | [diff] [blame] | 262 | sets.append(self.algorithms) |
Gilles Peskine | 7961668 | 2019-11-21 20:08:10 +0100 | [diff] [blame] | 263 | if function == 'key_agreement_algorithm' and \ |
| 264 | argument.startswith('PSA_ALG_KEY_AGREEMENT('): |
| 265 | # We only want *raw* key agreement algorithms as such, so |
| 266 | # exclude ones that are already chained with a KDF. |
| 267 | # Keep the expression as one to test as an algorithm. |
| 268 | function = 'other_algorithm' |
Gilles Peskine | 8fa1348 | 2019-11-25 17:10:12 +0100 | [diff] [blame] | 269 | sets += self.table_by_test_function[function] |
Gilles Peskine | 2bcfc71 | 2019-11-21 19:49:26 +0100 | [diff] [blame] | 270 | if self.accept_test_case_line(function, argument): |
| 271 | for s in sets: |
| 272 | s.add(argument) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 273 | |
| 274 | # Regex matching a *.data line containing a test function call and |
| 275 | # its arguments. The actual definition is partly positional, but this |
| 276 | # regex is good enough in practice. |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 277 | _test_case_line_re = re.compile(r'(?!depends_on:)(\w+):([^\n :][^:\n]*)') |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 278 | def parse_test_cases(self, filename): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 279 | """Parse a test case file (*.data), looking for algorithm metadata tests.""" |
Gilles Peskine | a0a315c | 2018-10-19 11:27:10 +0200 | [diff] [blame] | 280 | with read_file_lines(filename) as lines: |
| 281 | for line in lines: |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 282 | m = re.match(self._test_case_line_re, line) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 283 | if m: |
| 284 | self.add_test_case_line(m.group(1), m.group(2)) |
| 285 | |
Gilles Peskine | 84a4581 | 2019-11-21 19:50:33 +0100 | [diff] [blame] | 286 | def gather_inputs(headers, test_suites, inputs_class=Inputs): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 287 | """Read the list of inputs to test psa_constant_names with.""" |
Gilles Peskine | 84a4581 | 2019-11-21 19:50:33 +0100 | [diff] [blame] | 288 | inputs = inputs_class() |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 289 | for header in headers: |
| 290 | inputs.parse_header(header) |
| 291 | for test_cases in test_suites: |
| 292 | inputs.parse_test_cases(test_cases) |
| 293 | inputs.gather_arguments() |
| 294 | return inputs |
| 295 | |
| 296 | def remove_file_if_exists(filename): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 297 | """Remove the specified file, ignoring errors.""" |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 298 | if not filename: |
| 299 | return |
| 300 | try: |
| 301 | os.remove(filename) |
Gilles Peskine | 54f5445 | 2019-05-27 18:31:59 +0200 | [diff] [blame] | 302 | except OSError: |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 303 | pass |
| 304 | |
Gilles Peskine | b86b6d3 | 2019-11-21 17:26:10 +0100 | [diff] [blame] | 305 | def run_c(type_word, expressions, include_path=None, keep_c=False): |
Gilles Peskine | 5a994c1 | 2019-11-21 16:46:51 +0100 | [diff] [blame] | 306 | """Generate and run a program to print out numerical values for expressions.""" |
Gilles Peskine | b86b6d3 | 2019-11-21 17:26:10 +0100 | [diff] [blame] | 307 | if include_path is None: |
| 308 | include_path = [] |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 309 | if type_word == 'status': |
Gilles Peskine | c4cd2ad | 2019-02-13 18:42:53 +0100 | [diff] [blame] | 310 | cast_to = 'long' |
| 311 | printf_format = '%ld' |
| 312 | else: |
| 313 | cast_to = 'unsigned long' |
| 314 | printf_format = '0x%08lx' |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 315 | c_name = None |
| 316 | exe_name = None |
| 317 | try: |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 318 | c_fd, c_name = tempfile.mkstemp(prefix='tmp-{}-'.format(type_word), |
Gilles Peskine | 95ab71a | 2019-01-04 19:46:59 +0100 | [diff] [blame] | 319 | suffix='.c', |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 320 | dir='programs/psa') |
| 321 | exe_suffix = '.exe' if platform.system() == 'Windows' else '' |
| 322 | exe_name = c_name[:-2] + exe_suffix |
| 323 | remove_file_if_exists(exe_name) |
| 324 | c_file = os.fdopen(c_fd, 'w', encoding='ascii') |
Gilles Peskine | 95ab71a | 2019-01-04 19:46:59 +0100 | [diff] [blame] | 325 | c_file.write('/* Generated by test_psa_constant_names.py for {} values */' |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 326 | .format(type_word)) |
Gilles Peskine | 95ab71a | 2019-01-04 19:46:59 +0100 | [diff] [blame] | 327 | c_file.write(''' |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 328 | #include <stdio.h> |
| 329 | #include <psa/crypto.h> |
| 330 | int main(void) |
| 331 | { |
| 332 | ''') |
Gilles Peskine | 5a994c1 | 2019-11-21 16:46:51 +0100 | [diff] [blame] | 333 | for expr in expressions: |
Gilles Peskine | c4cd2ad | 2019-02-13 18:42:53 +0100 | [diff] [blame] | 334 | c_file.write(' printf("{}\\n", ({}) {});\n' |
Gilles Peskine | 5a994c1 | 2019-11-21 16:46:51 +0100 | [diff] [blame] | 335 | .format(printf_format, cast_to, expr)) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 336 | c_file.write(''' return 0; |
| 337 | } |
| 338 | ''') |
| 339 | c_file.close() |
| 340 | cc = os.getenv('CC', 'cc') |
| 341 | subprocess.check_call([cc] + |
Gilles Peskine | b86b6d3 | 2019-11-21 17:26:10 +0100 | [diff] [blame] | 342 | ['-I' + dir for dir in include_path] + |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 343 | ['-o', exe_name, c_name]) |
Gilles Peskine | b86b6d3 | 2019-11-21 17:26:10 +0100 | [diff] [blame] | 344 | if keep_c: |
Gilles Peskine | cf9c18e | 2018-10-19 11:28:42 +0200 | [diff] [blame] | 345 | sys.stderr.write('List of {} tests kept at {}\n' |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 346 | .format(type_word, c_name)) |
Gilles Peskine | cf9c18e | 2018-10-19 11:28:42 +0200 | [diff] [blame] | 347 | else: |
| 348 | os.remove(c_name) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 349 | output = subprocess.check_output([exe_name]) |
| 350 | return output.decode('ascii').strip().split('\n') |
| 351 | finally: |
| 352 | remove_file_if_exists(exe_name) |
| 353 | |
Gilles Peskine | 42a0a0a | 2019-05-27 18:29:47 +0200 | [diff] [blame] | 354 | NORMALIZE_STRIP_RE = re.compile(r'\s+') |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 355 | def normalize(expr): |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 356 | """Normalize the C expression so as not to care about trivial differences. |
Gilles Peskine | 4408dfd | 2019-11-21 17:16:21 +0100 | [diff] [blame] | 357 | |
Gilles Peskine | a3b93ff | 2019-06-03 11:23:56 +0200 | [diff] [blame] | 358 | Currently "trivial differences" means whitespace. |
| 359 | """ |
Gilles Peskine | 5a6dc89 | 2019-11-21 16:48:07 +0100 | [diff] [blame] | 360 | return re.sub(NORMALIZE_STRIP_RE, '', expr) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 361 | |
Gilles Peskine | b86b6d3 | 2019-11-21 17:26:10 +0100 | [diff] [blame] | 362 | def collect_values(inputs, type_word, include_path=None, keep_c=False): |
Gilles Peskine | c231711 | 2019-11-21 17:17:39 +0100 | [diff] [blame] | 363 | """Generate expressions using known macro names and calculate their values. |
| 364 | |
| 365 | Return a list of pairs of (expr, value) where expr is an expression and |
| 366 | value is a string representation of its integer value. |
| 367 | """ |
| 368 | names = inputs.get_names(type_word) |
| 369 | expressions = sorted(inputs.generate_expressions(names)) |
Gilles Peskine | b86b6d3 | 2019-11-21 17:26:10 +0100 | [diff] [blame] | 370 | values = run_c(type_word, expressions, |
| 371 | include_path=include_path, keep_c=keep_c) |
Gilles Peskine | c231711 | 2019-11-21 17:17:39 +0100 | [diff] [blame] | 372 | return expressions, values |
| 373 | |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 374 | class Tests: |
| 375 | """An object representing tests and their results.""" |
Gilles Peskine | 4408dfd | 2019-11-21 17:16:21 +0100 | [diff] [blame] | 376 | |
Gilles Peskine | a5000f1 | 2019-11-21 17:51:11 +0100 | [diff] [blame] | 377 | Error = namedtuple('Error', |
| 378 | ['type', 'expression', 'value', 'output']) |
| 379 | |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 380 | def __init__(self, options): |
| 381 | self.options = options |
| 382 | self.count = 0 |
| 383 | self.errors = [] |
Gilles Peskine | 4408dfd | 2019-11-21 17:16:21 +0100 | [diff] [blame] | 384 | |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 385 | def run_one(self, inputs, type_word): |
| 386 | """Test psa_constant_names for the specified type. |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 387 | |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 388 | Run the program on the names for this type. |
| 389 | Use the inputs to figure out what arguments to pass to macros that |
| 390 | take arguments. |
| 391 | """ |
| 392 | expressions, values = collect_values(inputs, type_word, |
| 393 | include_path=self.options.include, |
| 394 | keep_c=self.options.keep_c) |
| 395 | output = subprocess.check_output([self.options.program, type_word] + |
| 396 | values) |
| 397 | outputs = output.decode('ascii').strip().split('\n') |
| 398 | self.count += len(expressions) |
| 399 | for expr, value, output in zip(expressions, values, outputs): |
Gilles Peskine | 3255848 | 2019-12-03 19:03:35 +0100 | [diff] [blame] | 400 | if self.options.show: |
| 401 | sys.stdout.write('{} {}\t{}\n'.format(type_word, value, output)) |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 402 | if normalize(expr) != normalize(output): |
Gilles Peskine | a5000f1 | 2019-11-21 17:51:11 +0100 | [diff] [blame] | 403 | self.errors.append(self.Error(type=type_word, |
| 404 | expression=expr, |
| 405 | value=value, |
| 406 | output=output)) |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 407 | |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 408 | def run_all(self, inputs): |
| 409 | """Run psa_constant_names on all the gathered inputs.""" |
| 410 | for type_word in ['status', 'algorithm', 'ecc_curve', 'dh_group', |
| 411 | 'key_type', 'key_usage']: |
| 412 | self.run_one(inputs, type_word) |
Gilles Peskine | 4408dfd | 2019-11-21 17:16:21 +0100 | [diff] [blame] | 413 | |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 414 | def report(self, out): |
| 415 | """Describe each case where the output is not as expected. |
| 416 | |
| 417 | Write the errors to ``out``. |
| 418 | Also write a total. |
| 419 | """ |
Gilles Peskine | a5000f1 | 2019-11-21 17:51:11 +0100 | [diff] [blame] | 420 | for error in self.errors: |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 421 | out.write('For {} "{}", got "{}" (value: {})\n' |
Gilles Peskine | a5000f1 | 2019-11-21 17:51:11 +0100 | [diff] [blame] | 422 | .format(error.type, error.expression, |
| 423 | error.output, error.value)) |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 424 | out.write('{} test cases'.format(self.count)) |
| 425 | if self.errors: |
| 426 | out.write(', {} FAIL\n'.format(len(self.errors))) |
| 427 | else: |
| 428 | out.write(' PASS\n') |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 429 | |
Gilles Peskine | 69f93b5 | 2019-11-21 16:49:50 +0100 | [diff] [blame] | 430 | HEADERS = ['psa/crypto.h', 'psa/crypto_extra.h', 'psa/crypto_values.h'] |
| 431 | TEST_SUITES = ['tests/suites/test_suite_psa_crypto_metadata.data'] |
| 432 | |
Gilles Peskine | 54f5445 | 2019-05-27 18:31:59 +0200 | [diff] [blame] | 433 | def main(): |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 434 | parser = argparse.ArgumentParser(description=globals()['__doc__']) |
| 435 | parser.add_argument('--include', '-I', |
| 436 | action='append', default=['include'], |
| 437 | help='Directory for header files') |
Gilles Peskine | cf9c18e | 2018-10-19 11:28:42 +0200 | [diff] [blame] | 438 | parser.add_argument('--keep-c', |
| 439 | action='store_true', dest='keep_c', default=False, |
| 440 | help='Keep the intermediate C file') |
| 441 | parser.add_argument('--no-keep-c', |
| 442 | action='store_false', dest='keep_c', |
| 443 | help='Don\'t keep the intermediate C file (default)') |
Gilles Peskine | 8f5a501 | 2019-11-21 16:49:10 +0100 | [diff] [blame] | 444 | parser.add_argument('--program', |
| 445 | default='programs/psa/psa_constant_names', |
| 446 | help='Program to test') |
Gilles Peskine | 3255848 | 2019-12-03 19:03:35 +0100 | [diff] [blame] | 447 | parser.add_argument('--show', |
| 448 | action='store_true', |
| 449 | help='Keep the intermediate C file') |
| 450 | parser.add_argument('--no-show', |
| 451 | action='store_false', dest='show', |
| 452 | help='Don\'t show tested values (default)') |
Gilles Peskine | 2482702 | 2018-09-25 18:49:23 +0200 | [diff] [blame] | 453 | options = parser.parse_args() |
Gilles Peskine | 69f93b5 | 2019-11-21 16:49:50 +0100 | [diff] [blame] | 454 | headers = [os.path.join(options.include[0], h) for h in HEADERS] |
| 455 | inputs = gather_inputs(headers, TEST_SUITES) |
Gilles Peskine | 2460933 | 2019-11-21 17:44:21 +0100 | [diff] [blame] | 456 | tests = Tests(options) |
| 457 | tests.run_all(inputs) |
| 458 | tests.report(sys.stdout) |
| 459 | if tests.errors: |
Gilles Peskine | 8b02235 | 2020-03-24 18:36:56 +0100 | [diff] [blame] | 460 | sys.exit(1) |
Gilles Peskine | 54f5445 | 2019-05-27 18:31:59 +0200 | [diff] [blame] | 461 | |
| 462 | if __name__ == '__main__': |
| 463 | main() |