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

Clean up all known races/platform-specific UB at the time PR was opened #9708

Merged
merged 10 commits into from
Feb 11, 2017
+105 −45
Merged

Clean up all known races/platform-specific UB at the time PR was opened #9708

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
321d0fc
net: fix a few races. Credit @TheBlueMatt
theuni Feb 6, 2017
644f123
Make nTimeConnected const in CNode
TheBlueMatt Feb 6, 2017
ae683c1
Avoid copying CNodeStats to make helgrind OK with buggy std::string
TheBlueMatt Feb 6, 2017
512731b
Access fRelayTxes with cs_filter lock in copyStats
TheBlueMatt Feb 6, 2017
96f42d8
Make nStartingHeight atomic
TheBlueMatt Feb 6, 2017
0f31872
Make nServices atomic
TheBlueMatt Feb 6, 2017
22b4966
Move [clean|str]SubVer writes/copyStats into a lock
TheBlueMatt Feb 6, 2017
d8f2b8a
Make nTimeBestReceived atomic
TheBlueMatt Feb 6, 2017
036073b
Move CNode::addrName accesses behind locked accessors
TheBlueMatt Feb 6, 2017
db2dc7a
Move CNode::addrLocal access behind locked accessors
TheBlueMatt Feb 6, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Avoid copying CNodeStats to make helgrind OK with buggy std::string
  • Loading branch information
@TheBlueMatt
TheBlueMatt committed Feb 10, 2017
commit ae683c1b1960b32134f5a5a29504691c91f39cf3
5 changes: 2 additions & 3 deletions src/net.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2420,9 +2420,8 @@ void CConnman::GetNodeStats(std::vector<CNodeStats>& vstats)
vstats.reserve(vNodes.size());
for(std::vector<CNode*>::iterator it = vNodes.begin(); it != vNodes.end(); ++it) {
CNode* pnode = *it;
CNodeStats stats;
pnode->copyStats(stats);
vstats.push_back(stats);
vstats.emplace_back();
pnode->copyStats(vstats.back());
}
}

Expand Down