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
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
Make nServices atomic
  • Loading branch information
@TheBlueMatt
TheBlueMatt committed Feb 10, 2017
commit 0f3187261519c7568ef4211ce12b9740a3c1200f
2 changes: 1 addition & 1 deletion src/net.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -564,7 +564,7 @@ class CNode
friend class CConnman;
public:
// socket
ServiceFlags nServices;
std::atomic<ServiceFlags> nServices;
ServiceFlags nServicesExpected;
SOCKET hSocket;
size_t nSendSize; // total size of all vSendMsg entries
Expand Down