Fix deadlock on janus_videoroom_rtp_forwarder_rtcp_receive (#3469)

spscream · Alexander Malaev · web-flow · commit 51fe38a452e1 · 2024-10-29T11:46:32.000+01:00
Co-authored-by: Alexander Malaev &lt;amalaev@ccsteam.ru&gt;
diff --git a/src/plugins/janus_videoroom.c b/src/plugins/janus_videoroom.c
@@ -13271,6 +13271,7 @@ static void janus_videoroom_rtp_forwarder_rtcp_receive(janus_rtp_forwarder *rf,
 				janus_mutex_lock(&p->rtp_forwarders_mutex);
 				if(g_hash_table_size(p->rtp_forwarders) == 0) {
 					janus_mutex_unlock(&p->rtp_forwarders_mutex);
+					janus_mutex_unlock(&p->streams_mutex);
 					return;
 				}
 				gboolean found = FALSE;

Original file line number	Diff line number	Diff line change
`@@ -13271,6 +13271,7 @@ static void janus_videoroom_rtp_forwarder_rtcp_receive(janus_rtp_forwarder *rf,`
`13271`	`13271`	`janus_mutex_lock(&p->rtp_forwarders_mutex);`
`13272`	`13272`	`if(g_hash_table_size(p->rtp_forwarders) == 0) {`
`13273`	`13273`	`janus_mutex_unlock(&p->rtp_forwarders_mutex);`
	`13274`	`+ janus_mutex_unlock(&p->streams_mutex);`
`13274`	`13275`	`return;`
`13275`	`13276`	`}`
`13276`	`13277`	`gboolean found = FALSE;`