Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Check for crypto/rand errors and ReadFull io.Readers #417

Merged
merged 2 commits into from Mar 16, 2023
Merged

Check for crypto/rand errors and ReadFull io.Readers #417

merged 2 commits into from Mar 16, 2023

Commits on Mar 16, 2023

  1. Check for crypto/rand errors and ReadFull io.Readers 

    In practice crypto/rand.Read never returns an error, but that is not
guaranteed. Check for those errors.

In contrast to crypto/rand.Reader, a user-provided io.Reader,
might not fill the buffer without returning an error. Though marginal,
we should deal with that corner-case as well.
    @bwesterb
    bwesterb committed Mar 16, 2023
    Copy the full SHA
    7857037 View commit details
    Browse the repository at this point in the history

  2. Update go-ristretto dep 

    New version panic()s if reads from crypto/rand fail.
    @bwesterb
    bwesterb committed Mar 16, 2023
    Copy the full SHA
    3d45470 View commit details
    Browse the repository at this point in the history