deliberately add a wrong negative lock annotation to test that various clang versions can detect the issue (see D14876).
Warnings generated by clang14 on a debug build:
[412/541] Building CXX object src/CMakeFiles/server.dir/net.cpp.o /home/pierre/dev/bitcoin-abc/src/net.cpp:3124:9: warning: calling function 'DeleteNode' requires negative capability '!m_nodes_mutex' [-Wthread-safety-analysis] DeleteNode(pnode); ^ /home/pierre/dev/bitcoin-abc/src/net.cpp:3138:9: warning: calling function 'DeleteNode' requires negative capability '!m_nodes_mutex' [-Wthread-safety-analysis] DeleteNode(pnode); ^ 2 warnings generated. [417/541] Building CXX object src/CMakeFiles/server.dir/net_processing.cpp.o /home/pierre/dev/bitcoin-abc/src/net_processing.cpp:6314:9: warning: cannot call function 'ProcessMessage' while mutex 'm_recent_confirmed_transactions_mutex' is held [-Wthread-safety-analysis] ProcessMessage(config, *pfrom, msg.m_type, vRecv, msg.m_time, ^ 1 warning generated.
Depends on D14876