Make the cs_sendProcessing a LOCK instead of a TRY_LOCK

Technically cs_sendProcessing is entirely useless now because it
is only ever taken on the one MessageHandler thread, but because
there may be multiple of those in the future, it is left in place
This commit is contained in:
Matt Corallo 2017-01-12 20:08:52 -08:00
parent d7c58ad514
commit 376b3c2c6e

View file

@ -1871,9 +1871,8 @@ void CConnman::ThreadMessageHandler()
// Send messages
{
TRY_LOCK(pnode->cs_sendProcessing, lockSend);
if (lockSend)
GetNodeSignals().SendMessages(pnode, *this, flagInterruptMsgProc);
LOCK(pnode->cs_sendProcessing);
GetNodeSignals().SendMessages(pnode, *this, flagInterruptMsgProc);
}
if (flagInterruptMsgProc)
return;