diff options
| author | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-02-08 09:35:13 +0100 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-02-08 09:39:38 +0100 |
| commit | d57d10ee96a64b76dbf4a4cfe0c358f4ee4b7de2 (patch) | |
| tree | a9bfedd33b75d79016385e30319c62f474b509cf /src/wallet | |
| parent | 6db4fa7ad32985cf85c903966be69972bcb6b1d0 (diff) | |
| parent | 02fc8863630a20e75230f8bc3ba1051c480ae560 (diff) | |
| download | bitcoin-d57d10ee96a64b76dbf4a4cfe0c358f4ee4b7de2.tar.xz | |
Merge #12368: Hold mempool.cs for the duration of ATMP.
02fc886 Add braces to meet code style on line-after-the-one-changed. (Matt Corallo)
85aa839 Hold mempool.cs for the duration of ATMP. (Matt Corallo)
Pull request description:
This resolves an issue where getrawmempool() can race mempool
notification signals. Intuitively we use mempool.cs as a "read
lock" on the mempool with cs_main being the write lock, so holding
the read lock intermittently while doing write operations is
somewhat strange.
This also avoids the introduction of cs_main in getrawmempool()
which reviewers objected to in the previous fix in #12273
Tree-SHA512: 29464b9ca3890010ae13b7dc1c53487cc2bc9c3cf3d32a14cb09c8aa33848f57959d8991ea096beebcfb72f062e4e1962f104aefe4252c7db87633bbfe4ab317
Diffstat (limited to 'src/wallet')
0 files changed, 0 insertions, 0 deletions