Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Add mutex around peers.get<T>() #1291

Merged
merged 1 commit into from
Oct 11, 2018
Merged

Add mutex around peers.get<T>() #1291

merged 1 commit into from
Oct 11, 2018

Conversation

rkeene
Copy link
Contributor

@rkeene rkeene commented Oct 11, 2018 •
edited
Loading

There seems to be a missing mutex around this call to a huge boost multi-index item

@rkeene rkeene added the bug label Oct 11, 2018
@rkeene rkeene added this to the V16.2 milestone Oct 11, 2018
@rkeene rkeene self-assigned this Oct 11, 2018
@rkeene
Copy link
Contributor Author

rkeene commented Oct 11, 2018 •
edited
Loading

Originally broken in [ae55a19] (V15.0+)

@rkeene rkeene merged commit f17bee5 into nanocurrency:master Oct 11, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@PlasmaPower PlasmaPower PlasmaPower approved these changes

Assignees

@rkeene rkeene

Labels
bug
Projects
None yet
Milestone
V16.2
Development

Successfully merging this pull request may close these issues.
2 participants
@rkeene @PlasmaPower