From 4d2ce9949a784da3da8ee60ef53a043b83d42797 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com>
Date: Fri, 13 Sep 2024 10:58:27 +0300
Subject: [PATCH] aggregator: Immediately return NONE from
 simple_get_next_time() on non-TIME segments

Otherwise this gives critical warnings.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/7514>
---
 subprojects/gstreamer/libs/gst/base/gstaggregator.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/subprojects/gstreamer/libs/gst/base/gstaggregator.c b/subprojects/gstreamer/libs/gst/base/gstaggregator.c
index d60aa14531..6b5caf8484 100644
--- a/subprojects/gstreamer/libs/gst/base/gstaggregator.c
+++ b/subprojects/gstreamer/libs/gst/base/gstaggregator.c
@@ -4010,6 +4010,11 @@ gst_aggregator_simple_get_next_time (GstAggregator * self)
   GstSegment *segment = &srcpad->segment;
 
   GST_OBJECT_LOCK (self);
+  if (segment->format != GST_FORMAT_TIME) {
+    GST_OBJECT_UNLOCK (self);
+    return GST_CLOCK_TIME_NONE;
+  }
+
   if (segment->position == -1 || segment->position < segment->start)
     next_time = segment->start;
   else