Deadlock on DRing::hangUp

changed the description

Here's what I figured out.

T1 = Thread 26
T2 = Thread 1

T1 trie to acquire callMutex_ at src/sipcall.cpp:658:

658     std::lock_guard<std::recursive_mutex> lk {callMutex_};

but it's already acquired by T2 at src/sipcall.cpp:689:

689     std::lock_guard<std::recursive_mutex> lk {callMutex_};

Meanwhile, T2 tries to acquire the group lock of the invite session dialog at sip/sipcall.cpp:692:

692         auto ret = pjsip_inv_end_session(inviteSession_.get(), status, nullptr, &tdata);

but it's already acquire by T1 at src/jamidht/channeled_transport.cpp:237

237         auto eaten = pjsip_tpmgr_receive_packet(trData_.base.tpmgr, &rdata_);

Ooops deadlock

assigned to @sblin

added sprintto review ~21 labels

mentioned in commit 86a21e06

closed

reopened

closed

removed sprintdone label

Designs