diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2018-03-13 20:15:07 -0400 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2018-03-13 20:15:09 -0400 |
| commit | 18462960c0f13bd07d8f52b61e7d7bc17e991eea (patch) | |
| tree | a6912c605375e83ab1e72194add339ddc12d9b6d /contrib/init | |
| parent | 0630974647dacaf25e7fcb7f9cbb785bb078ede6 (diff) | |
| parent | 33eb9071b98e63fd66c1211254e420535320eebd (diff) | |
| download | bitcoin-18462960c0f13bd07d8f52b61e7d7bc17e991eea.tar.xz | |
Merge #12681: Fix ComputeTimeSmart test failure with -DDEBUG_LOCKORDER
33eb9071b9 Fix ComputeTimeSmart test failure with -DDEBUG_LOCKORDER (Russell Yanofsky)
Pull request description:
Failure looks like:
```
Entering test case "ComputeTimeSmart"
test_bitcoin: sync.cpp:100: void potential_deadlock_detected(const std::pair<void*, void*>&, const LockStack&, const LockStack&): Assertion `false' failed.
unknown location(0): fatal error in "ComputeTimeSmart": signal: SIGABRT (application abort requested)
wallet/test/wallet_tests.cpp(566): last checkpoint
```
Reproducible with:
```
./configure --enable-debug
make -C src test/test_bitcoin && src/test/test_bitcoin --log_level=test_suite --run_test=wallet_tests/ComputeTimeSmart
```
Seems to be caused by acquiring `cs_main` inside `CWallet::ComputeTimeSmart` in #11041.
I think this may be causing timeouts on travis like: https://travis-ci.org/bitcoin/bitcoin/jobs/353005676#L2692
Tree-SHA512: b263cd122ea9c88204d1d8e7e35291c71ea6319f05114c5009235a75dbd0f669bc0394f44afeed0d9eb08c2a956cd7c08f1ac4ef28616932fef9b43eaac5521b
Diffstat (limited to 'contrib/init')
0 files changed, 0 insertions, 0 deletions