ssvlabs / ssv-spec

GNU General Public License v3.0
25 stars 22 forks source link

Fix static analysis issues #480

Closed MatheusFranco99 closed 3 weeks ago

MatheusFranco99 commented 1 month ago

Overview

This PR adds fixes for static analysis issues including:

It also adds a nosec comment for a cryptographically insecure PRNG (pseudo-random number generator) used in tests. We use this PRNG (math/rand) since the secure crypto/rand doesn't allow seeding (which is important for consistent tests).

An existing issue with implicit memory aliasing in a for loop will be automatically fixed by the go update to 1.22.