summaryrefslogtreecommitdiff
path: root/usr.sbin/config
diff options
context:
space:
mode:
authorTheo Buehler <tb@cvs.openbsd.org>2023-11-06 14:50:13 +0000
committerTheo Buehler <tb@cvs.openbsd.org>2023-11-06 14:50:13 +0000
commit63118232f51f339316721d16e769c923fd80b6f0 (patch)
tree430ed097b4dd48c95f63ee60bd1ed4b3efa61daa /usr.sbin/config
parent7b5a837e39cc3913d1e8cd048952f6b5662cc953 (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