-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Race condition: use of "range + closure + goroutine" combination #40
Labels
kind/bug
A bug in existing code (including security flaws)
Comments
There is no bug here, the code in question is correct, and the variables are properly captured by the closure. |
You're right. |
ghost
mentioned this issue
Sep 20, 2016
Closed
ribasushi
pushed a commit
that referenced
this issue
Jul 4, 2021
Also, regenerate with the latest gogo protobuf package. fixes #40
longfeiWan9
pushed a commit
to longfeiWan9/go-ipfs
that referenced
this issue
Nov 18, 2021
feat: Build script
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
bug located at: https://github.com/jbenet/go-ipfs/blob/7968b45e58e6dfee3e732276a6cb7eab889a4333/bitswap/bitswap.go#L182
see: https://code.google.com/p/go-wiki/wiki/CommonMistakes#Using_Closures_with_Goroutines
The text was updated successfully, but these errors were encountered: