Skip to content

Commit f8a8ad8

Browse files
committed
Add more safeguards on the arguments
1 parent 8f0ab52 commit f8a8ad8

2 files changed

Lines changed: 7 additions & 3 deletions

File tree

SofaRegressionProgram.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -178,9 +178,9 @@ def make_parser():
178178
if args.legacy_mode:
179179
print("Legacy regression mode activated.")
180180
reg_prog.legacy_mode = True
181-
182-
replayId = int(args.replay)
183-
if replayId is not None:
181+
182+
if args.replay is not None:
183+
replayId = int(args.replay)
184184
reg_prog.replay_references(replayId)
185185
sys.exit()
186186

tools/RegressionSceneList.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,10 @@ def compare_all_references(self):
152152

153153

154154
def replay_references(self, id_scene):
155+
if (id_scene < 0 or id_scene >= len(self.scenes_data_sets)):
156+
helper.writeError(f'Id of the scene given for replay: {id_scene} is out of range [0, {len(self.scenes_data_sets) - 1}] from input regression list file.')
157+
return
158+
155159
self.scenes_data_sets[id_scene].load_scene()
156160
self.scenes_data_sets[id_scene].add_compare_state()
157161
self.scenes_data_sets[id_scene].replay_references()

0 commit comments

Comments
 (0)