From 78642a2b294ed4eb6d07ba56959a9446e7520883 Mon Sep 17 00:00:00 2001 From: Richard Fuchs Date: Wed, 15 Sep 2021 14:46:26 -0400 Subject: [PATCH] TT#14008 remove redundant check for FILLED sinks We may have multiple subscribers, some of which may be dead/unused. We don't care if we have these since we don't forward to them anyway. possibly relevant for #1337 Change-Id: I3cded5080aa2005e9dd615cccf60bd4cba5feb7d (cherry picked from commit 6c9fe540cf194dcff21b20379009771d7d6ec8a0) --- daemon/media_socket.c | 21 --------------------- 1 file changed, 21 deletions(-) diff --git a/daemon/media_socket.c b/daemon/media_socket.c index c4839591f..2add72ef6 100644 --- a/daemon/media_socket.c +++ b/daemon/media_socket.c @@ -2082,27 +2082,6 @@ static void media_packet_kernel_check(struct packet_handler_ctx *phc) { return; } - GQueue *sinks = phc->sinks; - // we might get called before the sinks are set via media_packet_rtcp_demux - if (!sinks) { - sinks = &phc->mp.stream->rtp_sinks; - if (!sinks->length) - sinks = &phc->mp.stream->rtcp_sinks; - } - - for (GList *l = sinks->head; l; l = l->next) { - struct sink_handler *sh = l->data; - - if (MEDIA_ISSET(sh->sink->media, ASYMMETRIC)) - PS_SET(sh->sink, CONFIRMED); - - if (!PS_ISSET(sh->sink, FILLED)) { - __C_DBG("sink not FILLED for stream %s:%d", sockaddr_print_buf(&phc->mp.stream->endpoint.address), - phc->mp.stream->endpoint.port); - return; - } - } - mutex_lock(&phc->mp.stream->in_lock); kernelize(phc->mp.stream); mutex_unlock(&phc->mp.stream->in_lock);