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

TSAN error in confirmation height unbounded #3106

Merged
merged 1 commit into from Mar 1, 2021
Merged

TSAN error in confirmation height unbounded #3106

merged 1 commit into from Mar 1, 2021

Conversation

wezrule
Copy link
Contributor

@wezrule wezrule commented Mar 1, 2021

Error found by @SergiySW in the node.auto_bootstrap test
block_cache was not protected in a couple places

@wezrule wezrule added bug sanitizers Related to thread, address or undefined sanitizers labels Mar 1, 2021
@wezrule wezrule added this to the V22.0 milestone Mar 1, 2021
@wezrule wezrule requested a review from SergiySW March 1, 2021 13:41
@wezrule wezrule self-assigned this Mar 1, 2021
@wezrule wezrule merged commit 8a5dfee into nanocurrency:develop Mar 1, 2021
@wezrule wezrule deleted the tsan_block_cache branch March 1, 2021 15:07
@zhyatt zhyatt added this to Pending testing evaluation in V22.0DB11 via automation Mar 8, 2021
@zhyatt zhyatt moved this from Pending testing evaluation to Pending general testing in V22.0DB11 Mar 8, 2021
@zhyatt zhyatt moved this from Pending general testing to Testing completed in V22.0DB11 Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clemahieu clemahieu clemahieu approved these changes

@SergiySW SergiySW SergiySW approved these changes

Assignees

@wezrule wezrule

Labels
bug sanitizers Related to thread, address or undefined sanitizers
Projects
No open projects
V22.0DB11
Testing completed
Milestone
V22.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@wezrule @clemahieu @SergiySW