diff options
author | Theo Buehler <tb@cvs.openbsd.org> | 2023-11-06 14:50:13 +0000 |
---|---|---|
committer | Theo Buehler <tb@cvs.openbsd.org> | 2023-11-06 14:50:13 +0000 |
commit | 63118232f51f339316721d16e769c923fd80b6f0 (patch) | |
tree | 430ed097b4dd48c95f63ee60bd1ed4b3efa61daa /usr.sbin/config | |
parent | 7b5a837e39cc3913d1e8cd048952f6b5662cc953 (diff) |
Fix a for loop bug introduced in the concurrency refactor
Due to Go's idiosyncratic semantics of for loops, tests would only run
some of the test groups in the JSON file because by the time the closure
is called, the array index could be changed. For example, on fast 8 core
machines, the CMAC tests would run the last test group with key size 320
eight times rather than each of the eight test groups once.
Make a copy of the pointer before passing it to the closure to avoid this
issue.
Simpler version of my initial fix from jsing
Diffstat (limited to 'usr.sbin/config')
0 files changed, 0 insertions, 0 deletions