Thibault Saunier f4a5b6623b adaptivedemux2: Do not submit_transfer when cancelled ... There is a race condition where transfer has not been submitted yet while the request is cancelled which leads to the transfer state going back to `DOWNLOAD_REQUEST_STATE_OPEN` and the user of the request to get signalled about its completion (and the task actually happening after it was cancelled) leading to assertions and misbehaviours. To ensure that this race can't happen, we start differentiating between the UNSENT and CANCELLED states as in the normal case, when entering `submit_request` the state is UNSENT and at that point we need to know that it is not because the request has been cancelled. In practice this case lead to an assertion in `gst_adaptive_demux2_stream_begin_download_uri` because in a previous call to `gst_adaptive_demux2_stream_stop_default` we cancelled the previous request and setup a new one while it had not been submitted yet and then got a `on_download_complete` callback called from that previous cancelled request and then we tried to do `download_request_set_uri` on a request that was still `in_use`, leading to something like: ``` #0: 0x0000000186655ec8 g_assert (request->in_use == FALSE)assert.c:0 #1: 0x00000001127236b8 libgstadaptivedemux2.dylib`download_request_set_uri(request=0x000060000017cc00, uri="https://XXX/chunk-stream1-00002.webm", range_start=0, range_end=-1) at downloadrequest.c:361 #2: 0x000000011271cee8 libgstadaptivedemux2.dylib`gst_adaptive_demux2_stream_begin_download_uri(stream=0x00000001330f1800, uri="https://XXX/chunk-stream1-00002.webm", start=0, end=-1) at gstadaptivedemux-stream.c:1447 #3: 0x0000000112719898 libgstadaptivedemux2.dylib`gst_adaptive_demux2_stream_load_a_fragment [inlined] gst_adaptive_demux2_stream_download_fragment(stream=0x00000001330f1800) at gstadaptivedemux-stream.c:0 #4: 0x00000001127197f8 libgstadaptivedemux2.dylib`gst_adaptive_demux2_stream_load_a_fragment(stream=0x00000001330f1800) at gstadaptivedemux-stream.c:1969 #5: 0x000000011271c2a4 libgstadaptivedemux2.dylib`gst_adaptive_demux2_stream_next_download(stream=0x00000001330f1800) at gstadaptivedemux-stream.c:2112 ``` Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/5611>		2023-11-06 16:21:08 +00:00
..
dash	dash: mpdclient: fix divide by 0 if segment has no duration	2023-04-18 09:03:26 +01:00
hls	hlsdemux2: Don't set a referer when updating playlists	2023-07-18 16:00:49 +02:00
mss
downloadhelper.c	adaptivedemux2: Do not submit_transfer when cancelled	2023-11-06 16:21:08 +00:00
downloadhelper.h
downloadrequest.c	adaptivedemux2: Do not submit_transfer when cancelled	2023-11-06 16:21:08 +00:00
downloadrequest.h	adaptivedemux2: Do not submit_transfer when cancelled	2023-11-06 16:21:08 +00:00
gstadaptivedemux-period.c
gstadaptivedemux-private.h	adaptivedemux2: Handle early key-unit seek	2023-06-23 08:51:05 +02:00
gstadaptivedemux-stream.c	adaptivedemux2: Don't blindly set the main manifest URI as referer	2023-07-18 16:00:49 +02:00
gstadaptivedemux-stream.h
gstadaptivedemux-track.c	adaptivedemux2: Fix early seeking	2023-06-23 08:42:03 +02:00
gstadaptivedemux-types.h
gstadaptivedemux.c	gstadaptivedemux: fix memory leak	2023-08-12 15:17:46 +00:00
gstadaptivedemux.h
gstadaptivedemuxelement.c
gstadaptivedemuxelements.h
gstadaptivedemuxutils.c
gstadaptivedemuxutils.h
gstisoff.c
gstisoff.h
meson.build	adaptivedemux2: Add GStreamer to the deps list	2022-11-23 09:29:14 +00:00
plugin.c