Fix a MIDI-related deadlock in the GUI code.

[nageru] / shared / midi_device.cpp

diff --git a/shared/midi_device.cpp b/shared/midi_device.cpp
index e4b1a116e17a27ecff7b7768d6e19fa7cf3fcdfb..9d77e218f5078f37b5aa3eeba8c9bb87d84a8226 100644 (file)
--- a/shared/midi_device.cpp
+++ b/shared/midi_device.cpp
@@ -73,7 +73,7 @@ void MIDIDevice::thread_func()
 
        // The sequencer object is now ready to be used from other threads.
        {
-               lock_guard<mutex> lock(mu);
+               lock_guard<recursive_mutex> lock(mu);
                alsa_seq = seq;
                alsa_queue_id = queue_id;
        }
@@ -97,7 +97,7 @@ void MIDIDevice::thread_func()
                while (snd_seq_query_next_port(seq, pinfo) >= 0) {
                        constexpr int mask = SND_SEQ_PORT_CAP_READ | SND_SEQ_PORT_CAP_SUBS_READ;
                        if ((snd_seq_port_info_get_capability(pinfo) & mask) == mask) {
-                               lock_guard<mutex> lock(mu);
+                               lock_guard<recursive_mutex> lock(mu);
                                subscribe_to_port_lock_held(seq, *snd_seq_port_info_get_addr(pinfo));
                        }
                }
@@ -154,7 +154,7 @@ void MIDIDevice::handle_event(snd_seq_t *seq, snd_seq_event_t *event)
                return;
        }
 
-       lock_guard<mutex> lock(mu);
+       lock_guard<recursive_mutex> lock(mu);
        switch (event->type) {
        case SND_SEQ_EVENT_CONTROLLER: {
                receiver->controller_received(event->data.control.param, event->data.control.value);