This repository has been archived by the owner. It is now read-only.

fix racing condition #135

Merged
merged 1 commit into from Nov 9, 2017

Conversation

Projects
None yet
2 participants
@ilovesoup
Contributor

ilovesoup commented Nov 9, 2017

Seems implemented incorrect double lock resource allocation.
Here is the fix.


This change is Reviewable

@ilovesoup

This comment has been minimized.

Show comment
Hide comment
@ilovesoup

ilovesoup Nov 9, 2017

Contributor

@birdstorm PTAL

Contributor

ilovesoup commented Nov 9, 2017

@birdstorm PTAL

@birdstorm

LGTM

@ilovesoup ilovesoup merged commit caf9b33 into master Nov 9, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@ilovesoup ilovesoup deleted the fix_racing branch Nov 23, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.