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

[TSAN] minor race in rpc.send test #2919

Merged
merged 1 commit into from
Sep 9, 2020
Merged

[TSAN] minor race in rpc.send test #2919

merged 1 commit into from
Sep 9, 2020

Conversation

guilhermelawless
Copy link
Contributor

ASSERT_TIMELY sets the deadline on system which isn't thread safe; no need to use the second thread in this test.

@guilhermelawless guilhermelawless added unit test Related to a new, changed or fixed unit test sanitizers Related to thread, address or undefined sanitizers labels Sep 9, 2020
@guilhermelawless guilhermelawless added this to the V22.0 milestone Sep 9, 2020
@guilhermelawless guilhermelawless self-assigned this Sep 9, 2020
@guilhermelawless guilhermelawless merged commit 234397d into nanocurrency:develop Sep 9, 2020
@guilhermelawless guilhermelawless deleted the tsan/rpc-send-race branch September 9, 2020 16:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@SergiySW SergiySW SergiySW approved these changes

Assignees

@guilhermelawless guilhermelawless

Labels
sanitizers Related to thread, address or undefined sanitizers unit test Related to a new, changed or fixed unit test
Projects
None yet
Milestone
V22.0
Development

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