diff options
| author | MacroFake <falke.marco@gmail.com> | 2022-08-30 11:34:52 +0200 |
|---|---|---|
| committer | MacroFake <falke.marco@gmail.com> | 2022-08-30 11:35:11 +0200 |
| commit | cfda740b332c77800f9bb2506d840dad3f4938c0 (patch) | |
| tree | 85bacbd46a9e0fa8c9d9a9717ea8a90507e2c450 /src/test/util_tests.cpp | |
| parent | e191fac4f3c37820f0618f72f0a8e8b524531ab8 (diff) | |
| parent | 9816dc96b77fd9780bb97891cd45a1b9798db8f5 (diff) | |
Merge bitcoin/bitcoin#25174: net/net_processing: Add thread safety related annotations for CNode and Peer
9816dc96b77fd9780bb97891cd45a1b9798db8f5 net: note CNode members that are treated as const (Anthony Towns)
ef26f2f421071986a3878a1a94b0149ae8e16fcd net: mark CNode unique_ptr members as const (Anthony Towns)
bbec32c9ad2fe213314db9d39aa1eacff2e0bc23 net: mark TransportSerializer/m_serializer as const (Anthony Towns)
06ebdc886fcb4ca22f695bafe0956cff6d70a250 net/net_processing: add missing thread safety annotations (Anthony Towns)
Pull request description:
Adds `GUARDED_BY` and `const` annotations to document how we currently ensure various members of `CNode` and `Peer` aren't subject to race conditions.
ACKs for top commit:
MarcoFalke:
review ACK 9816dc96b77fd9780bb97891cd45a1b9798db8f5 📍
jonatack:
utACK 9816dc96b77fd9780bb97891cd45a1b9798db8f5
hebasto:
ACK 9816dc96b77fd9780bb97891cd45a1b9798db8f5, I have reviewed the code and it looks OK. In particular, I verified the usage of variables which got `GUARDED_BY` annotations.
Tree-SHA512: fa95bca72435d79caadc736ee7687e505dbe8fbdb20690809e97666664a8d0dea39a7d17cf16f0437d7f5746b9ad98a466b26325d2913252c5d2b520b384b785
Diffstat (limited to 'src/test/util_tests.cpp')
0 files changed, 0 insertions, 0 deletions