smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-20 21:56:30 +01:00
Code Issues Projects Releases Wiki Activity

Fix adeadlock (or rather NDEADLCK)

Browse source 
ControlProtocolManager::set_state() already takes the protocols_lock.
However effectively this is a NO-OP. During ::set_state no CPI should
be instantiated and  ControlProtocolManager::teardown() returns early.
This commit is contained in:
Robin Gareus 2017-04-21 03:37:26 +02:00
parent bdd960cd99
commit 661d258687
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
libs/ardour/control_protocol_manager.cc
View file

@ -469,7 +469,7 @@ ControlProtocolManager::set_state (const XMLNode& node, int /*version*/)
} }
} else { } else {
if (_session) { if (_session) {
teardown (*cpi, true); teardown (*cpi, false);
} else { } else {
cpi->requested = false; cpi->requested = false;
} }