From 9e2eeab1c68acbbd048c1de3475866a9b59cc42d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= Date: Thu, 27 Apr 2023 16:33:25 +0300 Subject: [PATCH] Revert "splitmuxsink: Avoid assertion when WAITING_GOP_COLLECT on reference context" This reverts commit f29c19be5821d26a6b7682d0c69ff31d0815e072. If this is called for the reference context then we would run into an infinite loop, which is not really better than an assertion. By fixing up DTS to never be ahead of the PTS in the previous commit this situation should be impossible to hit now. Part-of: --- .../gst-plugins-good/gst/multifile/gstsplitmuxsink.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/subprojects/gst-plugins-good/gst/multifile/gstsplitmuxsink.c b/subprojects/gst-plugins-good/gst/multifile/gstsplitmuxsink.c index 0d661854ad..012ca3e43f 100644 --- a/subprojects/gst-plugins-good/gst/multifile/gstsplitmuxsink.c +++ b/subprojects/gst-plugins-good/gst/multifile/gstsplitmuxsink.c @@ -3184,13 +3184,11 @@ handle_mq_input (GstPad * pad, GstPadProbeInfo * info, MqStreamCtx * ctx) } break; case SPLITMUX_INPUT_STATE_WAITING_GOP_COLLECT:{ - /* We're collecting a GOP, this is normally only called for non-reference + /* We're collecting a GOP, this is only ever called for non-reference * contexts as the reference context would be waiting inside * check_completed_gop() */ - if (G_UNLIKELY (ctx->is_reference)) { - check_completed_gop (splitmux, ctx); - break; - } + + g_assert (!ctx->is_reference); /* If we overran the target timestamp, it might be time to process * the GOP, otherwise bail out for more data. */