HomePhabricator

tests: Add missing locks to tests

Description

tests: Add missing locks to tests

Summary:
Add missing locks to tests to satisfy lock requirements (such as
EXCLUSIVE_LOCKS_REQUIRED(...) (Clang Thread Safety Analysis),
AssertLockHeld(...) and implicit lock assumptions).

This doesn't clear all the thread sanitizer issues, so running with the
tsan enabled still output a lot of errors.

Backport of core PR11623
Completes T543

Test Plan:

make check

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Subscribers: teamcity, schancel

Differential Revision: https://reviews.bitcoinabc.org/D2617

Details

Provenance
practicalswift <practicalswift@users.noreply.github.com>Authored on Nov 6 2017, 22:08
FabienCommitted on Feb 28 2019, 10:59
FabienPushed on Feb 28 2019, 10:59
Reviewer
Restricted Project
Differential Revision
D2617: tests: Add missing locks to tests
Parents
rABCbf562f11084b: Use cached [compact] blocks to respond to getdata messages
Branches
Unknown
Tags
Unknown

Event Timeline

Fabien committed rABCafadcafdac8f: tests: Add missing locks to tests (authored by practicalswift <practicalswift@users.noreply.github.com>).Feb 28 2019, 10:59