GHSA-xg2h-wx96-xgxr
RandomAlphaNumeric and CryptoRandomAlphaNumeric are not as random as they should be
Packages
| Ecosystem | Package | Affected | Fixed in |
|---|---|---|---|
| Go | github.com/Masterminds/goutils | < 1.1.1 | 1.1.1 |
Details
As written in the advisory (Markdown source, shown as plain text).
### Impact
A security-sensitive bug was discovered by Open Source Developer *Erik Sundell of Sundell Open Source Consulting AB*.
The functions `RandomAlphaNumeric(int)` and `CryptoRandomAlphaNumeric(int)` are not as random as they should be. Small values of `int` in the functions above will return a smaller subset of results than they should. For example, `RandomAlphaNumeric(1)` will always return a digit in the 0-9 range, while `RandomAlphaNumeric(4)` will return around ~7 million of the ~13M possible permutations.
This is considered a security release because programs that rely upon random generators for passwords are at an increased risk of brute force-style password guessing. There is also a higher probability of collision.
The problem was the result of a mistaken regular expression that only accepted random strings if they contained a digit from `[0-9]`. That restriction has been removed.
### Patches
This issue has been corrected in v1.1.1.
### Workarounds
If you cannot upgrade to v1.1.1, you can work around the issue by calling `RandomAlphaNumericCustom(N, true, true)`|`CryptoRandomAlphaNumericCustom(N, true, true)` instead. (Where `N` is the desired length, and `true` is the literal boolean `true`.)
References
External links from the advisory. They lead to third-party sites.
- https://github.com/Masterminds/goutils/security/advisories/GHSA-xg2h-wx96-xgxr
- https://nvd.nist.gov/vuln/detail/CVE-2021-4238
- https://github.com/Masterminds/goutils/commit/869801f20f9f1e7ecdbdb6422049d8241270d5e1
- https://github.com/Masterminds/goutils/commit/f1923532a168b8203bfe956d8cd3b17ebece5982
- https://github.com/Masterminds/goutils
- https://github.com/Masterminds/goutils/releases/tag/v1.1.1
- https://pkg.go.dev/vuln/GO-2022-0411