blob: 9cdc290b9fc047b086c0f14b7566a2a4ad421b87 [file] [log] [blame]
Simon Glass8729d582016-07-03 09:40:46 -06001# SPDX-License-Identifier: GPL-2.0+
Tom Rini83d290c2018-05-06 17:58:06 -04002# Copyright (c) 2016, Google Inc.
Simon Glass8729d582016-07-03 09:40:46 -06003#
4# U-Boot Verified Boot Test
5
6"""
7This tests verified boot in the following ways:
8
9For image verification:
10- Create FIT (unsigned) with mkimage
11- Check that verification shows that no keys are verified
12- Sign image
13- Check that verification shows that a key is now verified
14
15For configuration verification:
16- Corrupt signature and check for failure
17- Create FIT (with unsigned configuration) with mkimage
Simon Glass72f52262016-07-31 17:35:04 -060018- Check that image verification works
Simon Glass8729d582016-07-03 09:40:46 -060019- Sign the FIT and mark the key as 'required' for verification
20- Check that image verification works
21- Corrupt the signature
22- Check that image verification no-longer works
23
24Tests run with both SHA1 and SHA256 hashing.
25"""
26
Simon Glassd5f3aad2021-02-15 17:08:08 -070027import shutil
Teddy Reed72239fc2018-06-09 11:38:05 -040028import struct
Simon Glassb0086772020-03-18 11:44:05 -060029import pytest
Simon Glass8729d582016-07-03 09:40:46 -060030import u_boot_utils as util
Simon Glassc0219712020-03-18 11:43:59 -060031import vboot_forge
Simon Glassd5f3aad2021-02-15 17:08:08 -070032import vboot_evil
Simon Glass8729d582016-07-03 09:40:46 -060033
Simon Glassd5f3aad2021-02-15 17:08:08 -070034# Only run the full suite on a few combinations, since it doesn't add any more
35# test coverage.
Simon Glass1b090032020-03-18 11:44:00 -060036TESTDATA = [
Simon Glassd5f3aad2021-02-15 17:08:08 -070037 ['sha1', '', None, False, True],
38 ['sha1', '', '-E -p 0x10000', False, False],
39 ['sha1', '-pss', None, False, False],
40 ['sha1', '-pss', '-E -p 0x10000', False, False],
41 ['sha256', '', None, False, False],
42 ['sha256', '', '-E -p 0x10000', False, False],
43 ['sha256', '-pss', None, False, False],
44 ['sha256', '-pss', '-E -p 0x10000', False, False],
45 ['sha256', '-pss', None, True, False],
46 ['sha256', '-pss', '-E -p 0x10000', True, True],
Simon Glass1b090032020-03-18 11:44:00 -060047]
48
Michal Simek04a47862016-07-18 08:49:08 +020049@pytest.mark.boardspec('sandbox')
Simon Glass8729d582016-07-03 09:40:46 -060050@pytest.mark.buildconfigspec('fit_signature')
Stephen Warren2d26bf62017-09-18 11:11:49 -060051@pytest.mark.requiredtool('dtc')
52@pytest.mark.requiredtool('fdtget')
53@pytest.mark.requiredtool('fdtput')
54@pytest.mark.requiredtool('openssl')
Simon Glassd5f3aad2021-02-15 17:08:08 -070055@pytest.mark.parametrize("sha_algo,padding,sign_options,required,full_test",
56 TESTDATA)
57def test_vboot(u_boot_console, sha_algo, padding, sign_options, required,
58 full_test):
Simon Glass8729d582016-07-03 09:40:46 -060059 """Test verified boot signing with mkimage and verification with 'bootm'.
60
61 This works using sandbox only as it needs to update the device tree used
62 by U-Boot to hold public keys from the signing process.
63
64 The SHA1 and SHA256 tests are combined into a single test since the
65 key-generation process is quite slow and we want to avoid doing it twice.
66 """
67 def dtc(dts):
Simon Glass72f52262016-07-31 17:35:04 -060068 """Run the device tree compiler to compile a .dts file
Simon Glass8729d582016-07-03 09:40:46 -060069
70 The output file will be the same as the input file but with a .dtb
71 extension.
72
73 Args:
74 dts: Device tree file to compile.
75 """
76 dtb = dts.replace('.dts', '.dtb')
Simon Glassec70f8a2016-07-31 17:35:05 -060077 util.run_and_log(cons, 'dtc %s %s%s -O dtb '
78 '-o %s%s' % (dtc_args, datadir, dts, tmpdir, dtb))
Simon Glass8729d582016-07-03 09:40:46 -060079
Simon Glassd5f3aad2021-02-15 17:08:08 -070080 def run_bootm(sha_algo, test_type, expect_string, boots, fit=None):
Simon Glass8729d582016-07-03 09:40:46 -060081 """Run a 'bootm' command U-Boot.
82
83 This always starts a fresh U-Boot instance since the device tree may
84 contain a new public key.
85
86 Args:
Simon Glassac9a23c2016-07-31 17:35:06 -060087 test_type: A string identifying the test type.
88 expect_string: A string which is expected in the output.
89 sha_algo: Either 'sha1' or 'sha256', to select the algorithm to
90 use.
Tom Rinide4be9e2016-09-18 09:46:58 -040091 boots: A boolean that is True if Linux should boot and False if
92 we are expected to not boot
Simon Glassd5f3aad2021-02-15 17:08:08 -070093 fit: FIT filename to load and verify
Simon Glass8729d582016-07-03 09:40:46 -060094 """
Simon Glassd5f3aad2021-02-15 17:08:08 -070095 if not fit:
96 fit = '%stest.fit' % tmpdir
Simon Glass27c087d2016-07-31 17:35:08 -060097 cons.restart_uboot()
Simon Glass851271a2016-07-31 17:35:07 -060098 with cons.log.section('Verified boot %s %s' % (sha_algo, test_type)):
99 output = cons.run_command_list(
Simon Glassd5f3aad2021-02-15 17:08:08 -0700100 ['host load hostfs - 100 %s' % fit,
Simon Glassb0086772020-03-18 11:44:05 -0600101 'fdt addr 100',
102 'bootm 100'])
103 assert expect_string in ''.join(output)
Tom Rinide4be9e2016-09-18 09:46:58 -0400104 if boots:
Simon Glassb0086772020-03-18 11:44:05 -0600105 assert 'sandbox: continuing, as we cannot run' in ''.join(output)
Philippe Reynesce5172c2019-09-18 16:04:53 +0200106 else:
Simon Glass3156ee32020-03-18 11:44:04 -0600107 assert('sandbox: continuing, as we cannot run'
108 not in ''.join(output))
Simon Glass8729d582016-07-03 09:40:46 -0600109
110 def make_fit(its):
Simon Glass72f52262016-07-31 17:35:04 -0600111 """Make a new FIT from the .its source file.
Simon Glass8729d582016-07-03 09:40:46 -0600112
113 This runs 'mkimage -f' to create a new FIT.
114
115 Args:
Simon Glass72f52262016-07-31 17:35:04 -0600116 its: Filename containing .its source.
Simon Glass8729d582016-07-03 09:40:46 -0600117 """
118 util.run_and_log(cons, [mkimage, '-D', dtc_args, '-f',
119 '%s%s' % (datadir, its), fit])
120
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200121 def sign_fit(sha_algo, options):
Simon Glass8729d582016-07-03 09:40:46 -0600122 """Sign the FIT
123
124 Signs the FIT and writes the signature into it. It also writes the
125 public key into the dtb.
Simon Glassac9a23c2016-07-31 17:35:06 -0600126
127 Args:
128 sha_algo: Either 'sha1' or 'sha256', to select the algorithm to
129 use.
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200130 options: Options to provide to mkimage.
Simon Glass8729d582016-07-03 09:40:46 -0600131 """
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200132 args = [mkimage, '-F', '-k', tmpdir, '-K', dtb, '-r', fit]
133 if options:
134 args += options.split(' ')
Simon Glassac9a23c2016-07-31 17:35:06 -0600135 cons.log.action('%s: Sign images' % sha_algo)
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200136 util.run_and_log(cons, args)
Simon Glass8729d582016-07-03 09:40:46 -0600137
Thirupathaiah Annapureddyfeaeee82020-08-16 23:01:10 -0700138 def sign_fit_norequire(sha_algo, options):
139 """Sign the FIT
140
141 Signs the FIT and writes the signature into it. It also writes the
142 public key into the dtb. It does not mark key as 'required' in dtb.
143
144 Args:
145 sha_algo: Either 'sha1' or 'sha256', to select the algorithm to
146 use.
147 options: Options to provide to mkimage.
148 """
149 args = [mkimage, '-F', '-k', tmpdir, '-K', dtb, fit]
150 if options:
151 args += options.split(' ')
152 cons.log.action('%s: Sign images' % sha_algo)
153 util.run_and_log(cons, args)
154
Teddy Reed72239fc2018-06-09 11:38:05 -0400155 def replace_fit_totalsize(size):
156 """Replace FIT header's totalsize with something greater.
157
158 The totalsize must be less than or equal to FIT_SIGNATURE_MAX_SIZE.
159 If the size is greater, the signature verification should return false.
160
161 Args:
162 size: The new totalsize of the header
163
164 Returns:
165 prev_size: The previous totalsize read from the header
166 """
167 total_size = 0
168 with open(fit, 'r+b') as handle:
169 handle.seek(4)
170 total_size = handle.read(4)
171 handle.seek(4)
172 handle.write(struct.pack(">I", size))
173 return struct.unpack(">I", total_size)[0]
174
Simon Glassda76ed22020-03-18 11:44:07 -0600175 def create_rsa_pair(name):
176 """Generate a new RSA key paid and certificate
177
178 Args:
179 name: Name of of the key (e.g. 'dev')
180 """
181 public_exponent = 65537
182 util.run_and_log(cons, 'openssl genpkey -algorithm RSA -out %s%s.key '
183 '-pkeyopt rsa_keygen_bits:2048 '
184 '-pkeyopt rsa_keygen_pubexp:%d' %
185 (tmpdir, name, public_exponent))
186
187 # Create a certificate containing the public key
188 util.run_and_log(cons, 'openssl req -batch -new -x509 -key %s%s.key '
189 '-out %s%s.crt' % (tmpdir, name, tmpdir, name))
190
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200191 def test_with_algo(sha_algo, padding, sign_options):
Simon Glass72f52262016-07-31 17:35:04 -0600192 """Test verified boot with the given hash algorithm.
Simon Glass8729d582016-07-03 09:40:46 -0600193
194 This is the main part of the test code. The same procedure is followed
195 for both hashing algorithms.
196
197 Args:
Simon Glassac9a23c2016-07-31 17:35:06 -0600198 sha_algo: Either 'sha1' or 'sha256', to select the algorithm to
199 use.
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200200 padding: Either '' or '-pss', to select the padding to use for the
201 rsa signature algorithm.
202 sign_options: Options to mkimage when signing a fit image.
Simon Glass8729d582016-07-03 09:40:46 -0600203 """
Simon Glassbcbd0c82016-07-31 17:35:02 -0600204 # Compile our device tree files for kernel and U-Boot. These are
205 # regenerated here since mkimage will modify them (by adding a
206 # public key) below.
Simon Glass8729d582016-07-03 09:40:46 -0600207 dtc('sandbox-kernel.dts')
208 dtc('sandbox-u-boot.dts')
209
210 # Build the FIT, but don't sign anything yet
Simon Glassac9a23c2016-07-31 17:35:06 -0600211 cons.log.action('%s: Test FIT with signed images' % sha_algo)
Simon Glassb0086772020-03-18 11:44:05 -0600212 make_fit('sign-images-%s%s.its' % (sha_algo, padding))
Tom Rinide4be9e2016-09-18 09:46:58 -0400213 run_bootm(sha_algo, 'unsigned images', 'dev-', True)
Simon Glass8729d582016-07-03 09:40:46 -0600214
215 # Sign images with our dev keys
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200216 sign_fit(sha_algo, sign_options)
Tom Rinide4be9e2016-09-18 09:46:58 -0400217 run_bootm(sha_algo, 'signed images', 'dev+', True)
Simon Glass8729d582016-07-03 09:40:46 -0600218
219 # Create a fresh .dtb without the public keys
220 dtc('sandbox-u-boot.dts')
221
Simon Glassac9a23c2016-07-31 17:35:06 -0600222 cons.log.action('%s: Test FIT with signed configuration' % sha_algo)
Simon Glassb0086772020-03-18 11:44:05 -0600223 make_fit('sign-configs-%s%s.its' % (sha_algo, padding))
Tom Rinide4be9e2016-09-18 09:46:58 -0400224 run_bootm(sha_algo, 'unsigned config', '%s+ OK' % sha_algo, True)
Simon Glass8729d582016-07-03 09:40:46 -0600225
226 # Sign images with our dev keys
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200227 sign_fit(sha_algo, sign_options)
Tom Rinide4be9e2016-09-18 09:46:58 -0400228 run_bootm(sha_algo, 'signed config', 'dev+', True)
Simon Glass8729d582016-07-03 09:40:46 -0600229
Simon Glassac9a23c2016-07-31 17:35:06 -0600230 cons.log.action('%s: Check signed config on the host' % sha_algo)
Simon Glass8729d582016-07-03 09:40:46 -0600231
Simon Glass477f5592020-03-18 11:43:58 -0600232 util.run_and_log(cons, [fit_check_sign, '-f', fit, '-k', dtb])
Simon Glass8729d582016-07-03 09:40:46 -0600233
Simon Glassd5f3aad2021-02-15 17:08:08 -0700234 if full_test:
235 # Make sure that U-Boot checks that the config is in the list of hashed
236 # nodes. If it isn't, a security bypass is possible.
237 ffit = '%stest.forged.fit' % tmpdir
238 shutil.copyfile(fit, ffit)
239 with open(ffit, 'rb') as fd:
240 root, strblock = vboot_forge.read_fdt(fd)
241 root, strblock = vboot_forge.manipulate(root, strblock)
242 with open(ffit, 'w+b') as fd:
243 vboot_forge.write_fdt(root, strblock, fd)
244 util.run_and_log_expect_exception(
245 cons, [fit_check_sign, '-f', ffit, '-k', dtb],
246 1, 'Failed to verify required signature')
Simon Glassc0219712020-03-18 11:43:59 -0600247
Simon Glassd5f3aad2021-02-15 17:08:08 -0700248 run_bootm(sha_algo, 'forged config', 'Bad Data Hash', False, ffit)
249
250 # Try adding an evil root node. This should be detected.
251 efit = '%stest.evilf.fit' % tmpdir
252 shutil.copyfile(fit, efit)
253 vboot_evil.add_evil_node(fit, efit, evil_kernel, 'fakeroot')
254
255 util.run_and_log_expect_exception(
256 cons, [fit_check_sign, '-f', efit, '-k', dtb],
257 1, 'Failed to verify required signature')
258 run_bootm(sha_algo, 'evil fakeroot', 'Bad Data Hash', False, efit)
259
260 # Try adding an @ to the kernel node name. This should be detected.
261 efit = '%stest.evilk.fit' % tmpdir
262 shutil.copyfile(fit, efit)
263 vboot_evil.add_evil_node(fit, efit, evil_kernel, 'kernel@')
264
265 util.run_and_log_expect_exception(
266 cons, [fit_check_sign, '-f', efit, '-k', dtb],
267 1, 'Node name contains @')
268 run_bootm(sha_algo, 'evil kernel@', 'Bad Data Hash', False, efit)
Simon Glassc0219712020-03-18 11:43:59 -0600269
270 # Create a new properly signed fit and replace header bytes
271 make_fit('sign-configs-%s%s.its' % (sha_algo, padding))
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200272 sign_fit(sha_algo, sign_options)
Teddy Reed72239fc2018-06-09 11:38:05 -0400273 bcfg = u_boot_console.config.buildconfig
274 max_size = int(bcfg.get('config_fit_signature_max_size', 0x10000000), 0)
275 existing_size = replace_fit_totalsize(max_size + 1)
Simon Glass3156ee32020-03-18 11:44:04 -0600276 run_bootm(sha_algo, 'Signed config with bad hash', 'Bad Data Hash',
277 False)
Teddy Reed72239fc2018-06-09 11:38:05 -0400278 cons.log.action('%s: Check overflowed FIT header totalsize' % sha_algo)
279
280 # Replace with existing header bytes
281 replace_fit_totalsize(existing_size)
282 run_bootm(sha_algo, 'signed config', 'dev+', True)
283 cons.log.action('%s: Check default FIT header totalsize' % sha_algo)
284
Simon Glass8729d582016-07-03 09:40:46 -0600285 # Increment the first byte of the signature, which should cause failure
Simon Glassec70f8a2016-07-31 17:35:05 -0600286 sig = util.run_and_log(cons, 'fdtget -t bx %s %s value' %
287 (fit, sig_node))
Simon Glass8729d582016-07-03 09:40:46 -0600288 byte_list = sig.split()
289 byte = int(byte_list[0], 16)
Simon Glassbcbd0c82016-07-31 17:35:02 -0600290 byte_list[0] = '%x' % (byte + 1)
Simon Glass8729d582016-07-03 09:40:46 -0600291 sig = ' '.join(byte_list)
Simon Glassec70f8a2016-07-31 17:35:05 -0600292 util.run_and_log(cons, 'fdtput -t bx %s %s value %s' %
293 (fit, sig_node, sig))
Simon Glass8729d582016-07-03 09:40:46 -0600294
Simon Glass3156ee32020-03-18 11:44:04 -0600295 run_bootm(sha_algo, 'Signed config with bad hash', 'Bad Data Hash',
296 False)
Simon Glass8729d582016-07-03 09:40:46 -0600297
Simon Glassac9a23c2016-07-31 17:35:06 -0600298 cons.log.action('%s: Check bad config on the host' % sha_algo)
Simon Glassb0086772020-03-18 11:44:05 -0600299 util.run_and_log_expect_exception(
300 cons, [fit_check_sign, '-f', fit, '-k', dtb],
301 1, 'Failed to verify required signature')
Simon Glass8729d582016-07-03 09:40:46 -0600302
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200303 def test_required_key(sha_algo, padding, sign_options):
Philippe Reynesce5172c2019-09-18 16:04:53 +0200304 """Test verified boot with the given hash algorithm.
305
Simon Glass3156ee32020-03-18 11:44:04 -0600306 This function tests if U-Boot rejects an image when a required key isn't
307 used to sign a FIT.
Philippe Reynesce5172c2019-09-18 16:04:53 +0200308
309 Args:
Simon Glass3156ee32020-03-18 11:44:04 -0600310 sha_algo: Either 'sha1' or 'sha256', to select the algorithm to use
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200311 padding: Either '' or '-pss', to select the padding to use for the
312 rsa signature algorithm.
313 sign_options: Options to mkimage when signing a fit image.
Philippe Reynesce5172c2019-09-18 16:04:53 +0200314 """
315 # Compile our device tree files for kernel and U-Boot. These are
316 # regenerated here since mkimage will modify them (by adding a
317 # public key) below.
318 dtc('sandbox-kernel.dts')
319 dtc('sandbox-u-boot.dts')
320
Philippe Reynesce5172c2019-09-18 16:04:53 +0200321 cons.log.action('%s: Test FIT with configs images' % sha_algo)
Simon Glass3156ee32020-03-18 11:44:04 -0600322
323 # Build the FIT with prod key (keys required) and sign it. This puts the
324 # signature into sandbox-u-boot.dtb, marked 'required'
Simon Glassb0086772020-03-18 11:44:05 -0600325 make_fit('sign-configs-%s%s-prod.its' % (sha_algo, padding))
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200326 sign_fit(sha_algo, sign_options)
Simon Glass3156ee32020-03-18 11:44:04 -0600327
328 # Build the FIT with dev key (keys NOT required). This adds the
329 # signature into sandbox-u-boot.dtb, NOT marked 'required'.
Simon Glassb0086772020-03-18 11:44:05 -0600330 make_fit('sign-configs-%s%s.its' % (sha_algo, padding))
Thirupathaiah Annapureddyfeaeee82020-08-16 23:01:10 -0700331 sign_fit_norequire(sha_algo, sign_options)
Philippe Reynesce5172c2019-09-18 16:04:53 +0200332
Simon Glass3156ee32020-03-18 11:44:04 -0600333 # So now sandbox-u-boot.dtb two signatures, for the prod and dev keys.
334 # Only the prod key is set as 'required'. But FIT we just built has
Thirupathaiah Annapureddyfeaeee82020-08-16 23:01:10 -0700335 # a dev signature only (sign_fit_norequire() overwrites the FIT).
Simon Glass3156ee32020-03-18 11:44:04 -0600336 # Try to boot the FIT with dev key. This FIT should not be accepted by
337 # U-Boot because the prod key is required.
338 run_bootm(sha_algo, 'required key', '', False)
Philippe Reynesce5172c2019-09-18 16:04:53 +0200339
Thirupathaiah Annapureddyfeaeee82020-08-16 23:01:10 -0700340 # Build the FIT with dev key (keys required) and sign it. This puts the
341 # signature into sandbox-u-boot.dtb, marked 'required'.
342 make_fit('sign-configs-%s%s.its' % (sha_algo, padding))
343 sign_fit(sha_algo, sign_options)
344
345 # Set the required-mode policy to "any".
346 # So now sandbox-u-boot.dtb two signatures, for the prod and dev keys.
347 # Both the dev and prod key are set as 'required'. But FIT we just built has
348 # a dev signature only (sign_fit() overwrites the FIT).
349 # Try to boot the FIT with dev key. This FIT should be accepted by
350 # U-Boot because the dev key is required and policy is "any" required key.
351 util.run_and_log(cons, 'fdtput -t s %s /signature required-mode any' %
352 (dtb))
353 run_bootm(sha_algo, 'multi required key', 'dev+', True)
354
355 # Set the required-mode policy to "all".
356 # So now sandbox-u-boot.dtb two signatures, for the prod and dev keys.
357 # Both the dev and prod key are set as 'required'. But FIT we just built has
358 # a dev signature only (sign_fit() overwrites the FIT).
359 # Try to boot the FIT with dev key. This FIT should not be accepted by
360 # U-Boot because the prod key is required and policy is "all" required key
361 util.run_and_log(cons, 'fdtput -t s %s /signature required-mode all' %
362 (dtb))
363 run_bootm(sha_algo, 'multi required key', '', False)
364
Simon Glass8729d582016-07-03 09:40:46 -0600365 cons = u_boot_console
366 tmpdir = cons.config.result_dir + '/'
Stephen Warrenc9ba60c2016-07-18 10:07:25 -0600367 datadir = cons.config.source_dir + '/test/py/tests/vboot/'
Simon Glass8729d582016-07-03 09:40:46 -0600368 fit = '%stest.fit' % tmpdir
369 mkimage = cons.config.build_dir + '/tools/mkimage'
370 fit_check_sign = cons.config.build_dir + '/tools/fit_check_sign'
371 dtc_args = '-I dts -O dtb -i %s' % tmpdir
372 dtb = '%ssandbox-u-boot.dtb' % tmpdir
Philippe Reynesed470972018-11-14 13:51:05 +0100373 sig_node = '/configurations/conf-1/signature'
Simon Glass8729d582016-07-03 09:40:46 -0600374
Simon Glassda76ed22020-03-18 11:44:07 -0600375 create_rsa_pair('dev')
376 create_rsa_pair('prod')
Philippe Reynesce5172c2019-09-18 16:04:53 +0200377
Simon Glass8729d582016-07-03 09:40:46 -0600378 # Create a number kernel image with zeroes
Simon Glassd5f3aad2021-02-15 17:08:08 -0700379 with open('%stest-kernel.bin' % tmpdir, 'wb') as fd:
380 fd.write(500 * b'\0')
381
382 # Create a second kernel image with ones
383 evil_kernel = '%stest-kernel1.bin' % tmpdir
384 with open(evil_kernel, 'wb') as fd:
385 fd.write(500 * b'\x01')
Simon Glass8729d582016-07-03 09:40:46 -0600386
387 try:
388 # We need to use our own device tree file. Remember to restore it
389 # afterwards.
390 old_dtb = cons.config.dtb
391 cons.config.dtb = dtb
Simon Glass1b090032020-03-18 11:44:00 -0600392 if required:
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200393 test_required_key(sha_algo, padding, sign_options)
Simon Glass1b090032020-03-18 11:44:00 -0600394 else:
Philippe Reyneseb7690e2020-04-29 15:26:16 +0200395 test_with_algo(sha_algo, padding, sign_options)
Simon Glass8729d582016-07-03 09:40:46 -0600396 finally:
Simon Glass27c087d2016-07-31 17:35:08 -0600397 # Go back to the original U-Boot with the correct dtb.
Simon Glass8729d582016-07-03 09:40:46 -0600398 cons.config.dtb = old_dtb
Simon Glass27c087d2016-07-31 17:35:08 -0600399 cons.restart_uboot()