[Bug 787922] splitmuxsink: add a "split-now" action signal

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Sep 27 14:54:02 UTC 2018


https://bugzilla.gnome.org/show_bug.cgi?id=787922

--- Comment #36 from Vivia Nikolaidou <vivia at ahiru.eu> ---
Thanks, please review again.

(In reply to Sebastian Dröge (slomo) from comment #32)
> Review of attachment 373772 [details] [review]:
> 
> ::: gst/multifile/gstsplitmuxsink.c
> @@ +1869,3 @@
> +  if (splitmux->reference_ctx->in_running_time >= time_to_split) {
> +    GST_OBJECT_LOCK (splitmux);
> +    splitmux->time_to_split = GST_CLOCK_TIME_NONE;
> 
> This is racy: Between taking the local copy of this value above, and
> resetting it here the signal could've been called again. You would then
> overwrite a *new* value here where splitting would happen.
> 
> Maybe just compare against the local version and only reset if it a) changed
> and b) the changed value is after the current split position (i.e. in the
> future, we still have the opportunity to split there)

Using a GstQueueArray now to queue up possible split times. This shouldn't be
an issue.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list