Differential D8443

net: Add missing cs_vNodes lock
ClosedPublic
Actions

Authored by Fabien on Nov 18 2020, 09:08.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABC781244d1ff80: net: Add missing cs_vNodes lock

Summary

Backport of core PR18458.

Test Plan

With Clang as a compiler:

cmake -GNinja .. -DCMAKE_BUILD_TYPE=Debug
ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Nov 18 2020, 09:08

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 18 2020, 09:08

Fabien requested review of this revision.Nov 18 2020, 09:08

teamcity edited the summary of this revision. (Show Details)Nov 18 2020, 09:08

@bot build-tsan

Harbormaster completed remote builds in B14087: Diff 25831.Nov 18 2020, 09:21

teamcity awarded a token.Nov 18 2020, 09:21

majcosta accepted this revision.Nov 18 2020, 09:34

This revision is now accepted and ready to land.Nov 18 2020, 09:34

Closed by commit rABC781244d1ff80: net: Add missing cs_vNodes lock (authored by MarcoFalke <falke.marco@gmail.com>, committed by Fabien). · Explain WhyNov 18 2020, 10:08

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC781244d1ff80: net: Add missing cs_vNodes lock.

Revision Contents
Changeset List

Path

Size

src/

14 lines

19 lines

5 lines

1 line

test/

denialofservice_tests.cpp

1 line

Diff 25833

View Options

src/init.cpp

View Options

src/net.h

View Options

src/net.cpp

View Options

src/net_processing.h

View Options

src/test/denialofservice_tests.cpp

net: Add missing cs_vNodes lockClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 25833

src/init.cpp

src/net.h

src/net.cpp

src/net_processing.h

src/test/denialofservice_tests.cpp

net: Add missing cs_vNodes lock
ClosedPublic
Actions

Revision Contents
Changeset List