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

Fix lazy mutex unlock via mutex #2382

Merged
merged 1 commit into from
Nov 4, 2019
Merged

Fix lazy mutex unlock via mutex #2382

merged 1 commit into from
Nov 4, 2019

Conversation

cryptocode
Copy link
Contributor

@cryptocode cryptocode commented Nov 2, 2019 •
edited
Loading

lock_guard d'tor would call unlock again, which is undefined behaviour

@cryptocode cryptocode requested a review from SergiySW November 2, 2019 13:23
@cryptocode cryptocode self-assigned this Nov 2, 2019
@cryptocode cryptocode added this to the V20.0 milestone Nov 2, 2019
@cryptocode cryptocode added the bug label Nov 2, 2019
@cryptocode cryptocode merged commit e489cf4 into nanocurrency:master Nov 4, 2019
@cryptocode cryptocode deleted the lazy_requeue-lockfix branch November 4, 2019 16:04
argakiig pushed a commit that referenced this pull request Nov 8, 2019
@cryptocode
Fix lazy mutex unlock via mutex (#2382)
5b70a75
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SergiySW SergiySW SergiySW approved these changes

Assignees

@cryptocode cryptocode

Labels
bug
Projects
None yet
Milestone
V20.0
Development

Successfully merging this pull request may close these issues.
2 participants
@cryptocode @SergiySW