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.

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

crypto/rand: Read hangs when passed buffer larger than 1<<32 - 1 [1.17 backport] #52932

Closed
gopherbot opened this issue May 16, 2022 · 3 comments
Closed

crypto/rand: Read hangs when passed buffer larger than 1<<32 - 1 [1.17 backport] #52932

gopherbot opened this issue May 16, 2022 · 3 comments
Labels
CherryPickApproved Security
Milestone
Go1.17.11

Comments

@gopherbot
Copy link

@gopherbot gopherbot commented May 16, 2022

@rolandshoemaker requested issue #52561 to be considered for backport to the next 1.17 minor release.

@gopherbot please open backport issues, this is a minor security issue.
@gopherbot gopherbot added the CherryPickCandidate label May 16, 2022
@gopherbot gopherbot mentioned this issue May 16, 2022
Closed
@gopherbot gopherbot added this to the Go1.17.11 milestone May 16, 2022
@gopherbot
Copy link
Author

@gopherbot gopherbot commented May 16, 2022

Change https://go.dev/cl/406635 mentions this issue: [release-branch.go1.17] crypto/rand: properly handle large Read on windows

@toothrot toothrot added the CherryPickApproved label May 25, 2022
@gopherbot gopherbot removed the CherryPickCandidate label May 25, 2022
@toothrot
Copy link
Contributor

@toothrot toothrot commented May 25, 2022

Approved. This is a serious issue with no workaround. @rolandshoemaker Does this need security text/labeling?

@gopherbot
Copy link
Author

@gopherbot gopherbot commented May 25, 2022

Closed by merging 2be03d7 to release-branch.go1.17.

gopherbot pushed a commit that referenced this issue May 25, 2022
@rolandshoemaker @toothrot
[release-branch.go1.17] crypto/rand: properly handle large Read on wi…
2be03d7 
…ndows

Use the batched reader to chunk large Read calls on windows to a max of
1 << 31 - 1 bytes. This prevents an infinite loop when trying to read
more than 1 << 32 -1 bytes, due to how RtlGenRandom works.

This change moves the batched function from rand_unix.go to rand.go,
since it is now needed for both windows and unix implementations.

Updates #52561
Fixes #52932
Fixes CVE-2022-30634

Change-Id: Id98fc4b1427e5cb2132762a445b2aed646a37473
Reviewed-on: https://go-review.googlesource.com/c/go/+/402257
Run-TryBot: Roland Shoemaker <roland@golang.org>
Reviewed-by: Filippo Valsorda <filippo@golang.org>
Reviewed-by: Filippo Valsorda <valsorda@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
(cherry picked from commit bb1f441)
Reviewed-on: https://go-review.googlesource.com/c/go/+/406635
Reviewed-by: Damien Neil <dneil@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CherryPickApproved Security
Projects
None yet
Milestone
Go1.17.11
Development

No branches or pull requests
3 participants
@toothrot @dmitshur @gopherbot