Page MenuHomePhabricator
Differential D11736

[avalanche] Hold the peer manager lock during the peer cleanup
ClosedPublic
Actions

Authored by Fabien on Jul 11 2022, 14:10.

Details

Reviewers
sdulfari
Group Reviewers
Restricted Project
Commits
rABC4459e2e60d30: [avalanche] Hold the peer manager lock during the peer cleanup
Summary

The cleanup task is accessing and modifying the peer structure, so it needs to hold the cs_peerManager lock.

Test Plan

With Clang and Debug:

ninja all check-all

With D11735:

./contrib/teamcity/build-configurations.py build-tsan

Diff Detail

Repository
rABC Bitcoin ABC
Branch
avalanche_periodic_cleanup_processor
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 19613
Build 38945: Build Difflint-circular-dependencies · build-without-wallet · build-clang-tidy · build-debug · build-diff · build-clang
Build 38944: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
avalanche/
6 lines
45 lines
9 lines
test/
52 lines
2 lines

Diff 34328

View Options

src/avalanche/peermanager.h

Loading...
View Options

src/avalanche/peermanager.cpp

Loading...
View Options

src/avalanche/processor.cpp

Loading...
View Options

src/avalanche/test/peermanager_tests.cpp

Loading...
View Options

src/avalanche/test/proofpool_tests.cpp

Loading...