From bd67ef18e9ae3b122e491d28b396a62fa75565fd Mon Sep 17 00:00:00 2001 From: =?utf8?q?Sebastian=20Dr=C3=B6ge?= Date: Mon, 25 May 2020 15:36:38 +0300 Subject: [PATCH] audiobuffersplit: Unset DISCONT flag if not discontinuous And also set/unset the RESYNC flag accordingly. It can happen that the flag is preserved by GstAdapter from the input buffer. For example if a big input buffer is split into many small ones, each of the small ones would have the flag set. All other buffer flags seem safe to keep here if they were set, including the GAP flag. Also ensure that the buffer is actually writable before changing any flags or metadata on it. Part-of: --- gst/audiobuffersplit/gstaudiobuffersplit.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/gst/audiobuffersplit/gstaudiobuffersplit.c b/gst/audiobuffersplit/gstaudiobuffersplit.c index bd66e9e..7d2738a 100644 --- a/gst/audiobuffersplit/gstaudiobuffersplit.c +++ b/gst/audiobuffersplit/gstaudiobuffersplit.c @@ -379,10 +379,15 @@ gst_audio_buffer_split_output (GstAudioBufferSplit * self, gboolean force, size = MIN (size, avail); buffer = gst_adapter_take_buffer (self->adapter, size); + buffer = gst_buffer_make_writable (buffer); /* After a reset we have to set the discont flag */ if (self->current_offset == 0) - GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DISCONT); + GST_BUFFER_FLAG_SET (buffer, + GST_BUFFER_FLAG_DISCONT | GST_BUFFER_FLAG_RESYNC); + else + GST_BUFFER_FLAG_UNSET (buffer, + GST_BUFFER_FLAG_DISCONT | GST_BUFFER_FLAG_RESYNC); resync_time_diff = gst_util_uint64_scale (self->current_offset, GST_SECOND, rate); -- 2.7.4