Skip to content

Commit ffc8564

Browse files
committed
sim: Parametrize make_bad_secondary_slot_image
Add ImageManipulation parameter to the function to allow creating images broken in various ways. Signed-off-by: Dominik Ermel <[email protected]>
1 parent 4bf4441 commit ffc8564

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

sim/src/image.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -297,15 +297,15 @@ impl ImagesBuilder {
297297
images
298298
}
299299

300-
pub fn make_bad_secondary_slot_image(self) -> Images {
300+
pub fn make_bad_secondary_slot_image(self, img_manipulation : ImageManipulation) -> Images {
301301
let mut bad_flash = self.flash;
302302
let ram = self.ram.clone(); // TODO: Avoid this clone.
303303
let images = self.slots.into_iter().enumerate().map(|(image_num, slots)| {
304304
let dep = BoringDep::new(image_num, &NO_DEPS);
305305
let primaries = install_image(&mut bad_flash, &self.areadesc, &slots, 0,
306306
maximal(32784), &ram, &dep, ImageManipulation::None, Some(0));
307307
let upgrades = install_image(&mut bad_flash, &self.areadesc, &slots, 1,
308-
maximal(41928), &ram, &dep, ImageManipulation::BadSignature, Some(0));
308+
maximal(41928), &ram, &dep, img_manipulation, Some(0));
309309
OneImage {
310310
slots,
311311
primaries,

sim/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ impl RunStatus {
202202

203203
// Creates a badly signed image in the secondary slot to check that
204204
// it is not upgraded to
205-
let bad_secondary_slot_image = run.clone().make_bad_secondary_slot_image();
205+
let bad_secondary_slot_image = run.clone().make_bad_secondary_slot_image(ImageManipulation::BadSignature);
206206

207207
failed |= bad_secondary_slot_image.run_signfail_upgrade();
208208

sim/tests/core.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ macro_rules! sim_test {
4949
};
5050
}
5151

52-
sim_test!(bad_secondary_slot, make_bad_secondary_slot_image(), run_signfail_upgrade());
52+
sim_test!(bad_secondary_slot, make_bad_secondary_slot_image(ImageManipulation::BadSignature), run_signfail_upgrade());
5353
sim_test!(secondary_trailer_leftover, make_erased_secondary_image(), run_secondary_leftover_trailer());
5454
sim_test!(bootstrap, make_bootstrap_image(), run_bootstrap());
5555
sim_test!(oversized_bootstrap, make_oversized_bootstrap_image(), run_oversized_bootstrap());

0 commit comments

Comments
 (0)