- Feb 23, 2017
-
-
Sebastian Dröge authored
-
Sebastian Dröge authored
-
- Feb 22, 2017
-
-
Let libsoup handle redirection automatically. And then, to figure out redirection uri, extract it on "restarted" callback which will be fired before soup_session_send() is returned. https://bugzilla.gnome.org/show_bug.cgi?id=778428
-
Nicolas Dufresne authored
Update the image size according the amount of data we are going to read/write. This workaround bugs in driver where the sizeimage provided by TRY/S_FMT represent the buffer length (maximum size) rather then the expected bytesused (buffer size). https://bugzilla.gnome.org/show_bug.cgi?id=775564
-
- Feb 21, 2017
-
-
Set MAY_BE_LEAKED flag on static pads returned by gst_v4l2_object_get_*_caps() functions. Made functions thread safe by using g_once_init[enter|leave] funtions. https://bugzilla.gnome.org/show_bug.cgi?id=778453
-
In gst_v4l2_allocator_qbuf(), the memory is referenced after the buffer is queued. Once queued (VIDIOC_QBUF), the buffer might be handled by the V4L2 driver (e.g. decoded) and dequeued (gst_v4l2_allocator_dqbuf), through a different thread, before the memory is referenced (gst_memory_ref). In this case, in gst_v4l2_allocator_dqbuf(), the memory is unreferenced (gst_memory_unref) before having been referenced: the memory refcount reaches 0, and the memory is freed. So, to avoid this crossing case, in gst_v4l2_allocator_qbuf(), the memory shall be referenced before the buffer is queued. https://bugzilla.gnome.org/show_bug.cgi?id=777399
-
After commit 1ea9735a I see these error while using the webcam integrated in my laptop: GStreamer-CRITICAL **: gst_value_list_get_size: assertion 'GST_VALUE_HOLDS_LIST (value)' failed The issue is gst_v4l2src_value_simplify() was doing its job of generating a single value, rather than the original list. That why, when getting the list size, a critical warning was raised. This patch takes advantage of the compiler optimizations to verify first if the list was simplified, thus use it directly, otherwise, if it is a list, verify its size. https://bugzilla.gnome.org/show_bug.cgi?id=776106
-
If for some reason we fail to probe formats (all try_fmt calls fail, for example), this is not a critical error, but we end up with an empty list of interlace modes. This causes all subsequent negotiation to fail. This patch fixes interlace-mode setting to be skipped if we failed to detect any. https://bugzilla.gnome.org/show_bug.cgi?id=775702
-
- Feb 20, 2017
-
-
Thibault Saunier authored
-
- Feb 19, 2017
-
-
Sebastian Dröge authored
This goes around the inefficient control message based filtering and does all the filtering kernel-side. Unfortunately this is Linux-only and there is no IPv6 variant of it (yet).
-
Some radio streams uses StreamTitle='' to reset the title after a track stopped playing, e.g. while the host talks between tracks or during news segments. This change forces an empty tag object to be distributed if StreamTitle or StreamUrl is received with empty value, thus allowing downstream elements to get notified about this. https://bugzilla.gnome.org/show_bug.cgi?id=778437
-
- Feb 07, 2017
-
-
Jan Schmidt authored
Used signed calculations when measuring the max_ts of an input fragment, so as to calculate the correct duration and offset when buffers have timestamps preceding their segment
-
- Jan 31, 2017
-
-
Tim-Philipp Müller authored
Fixes crash with fuzzed file. https://bugzilla.gnome.org/show_bug.cgi?id=777940
-
- Jan 30, 2017
-
-
Sebastian Dröge authored
-
Sebastian Dröge authored
-
Sebastian Dröge authored
-
- Jan 27, 2017
-
-
The n_frames field (frames per second) should follow the nominal frame rate for drop-frame timecodes. Also, the trak's timescale (and duration, accordingly) should follow the STSD entry's timescale and frame duration (fps_n and fps_d accordingly), not the other way around. https://bugzilla.gnome.org/show_bug.cgi?id=777832
-
Fix a regression introduced by commit 183695c6 (refactor to use Soup's sync API). The code previously attempted to reconnect when the server closed the connection early, for example when the stream was put in pause for some time. Reintroduce this feature by checking if EOS is received before the expected content size is downloaded. In this case, do the request starting at the previous read position. https://bugzilla.gnome.org/show_bug.cgi?id=776720
-
- Jan 26, 2017
-
-
Sebastian Dröge authored
This reverts commit 67f6d335. It causes problems in certain scenarios and needs further investigation https://bugzilla.gnome.org/show_bug.cgi?id=764947#c9
-
- Jan 25, 2017
-
-
In case wavparse receives a manually injected FLUSH_STOP event while operating in pull mode we get criticals because we'd try to clear a NULL adapter. https://bugzilla.gnome.org/show_bug.cgi?id=777123
-
-
-
- Jan 19, 2017
-
-
Sebastian Dröge authored
Otherwise we could read more chunks than there are available, doing an out of bounds read and potentially crash. https://bugzilla.gnome.org/show_bug.cgi?id=777469
-
Sebastian Dröge authored
This reverts commit 99d5d757. It broke playback of various valid files.
-
Sebastian Dröge authored
Otherwise we could read more chunks than there are available, doing an out of bounds read and potentially crash. https://bugzilla.gnome.org/show_bug.cgi?id=777469
-
- Jan 18, 2017
-
-
The current code configures libsoup to handle redirections transparently, without informing the caller, thus preventing the element to record the redirect code and location uri. Fix this by always setting the SOUP_MESSAGE_NO_REDIRECT, preventing libsoup from handling the redirection. When we receive a redirection request and libsoup can safely handle it, return a custom error which triggers a retry with the new URI. https://bugzilla.gnome.org/show_bug.cgi?id=777222
-
- Jan 16, 2017
-
-
When reset, don't restart request pad numberings, as request pads can survive across state changes. Only restart at 0 if all request pads are handed back first. https://bugzilla.gnome.org/show_bug.cgi?id=777174
-
The seqh buffer allocated in qtdemux_parse_svq3_stsd_data() needs to be freed by the caller after use. https://bugzilla.gnome.org/show_bug.cgi?id=777157 Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
-
The buffer memory type provided to the VIDIOC_CREATE_BUFS ioctl shall be set with the value ("memory") given as input parameter of the gst_v4l2_allocator_probe() function. https://bugzilla.gnome.org/show_bug.cgi?id=777327
-
- Jan 09, 2017
-
-
Otherwise some messages that are emitted by the element on NULL->READY will not reach the application. https://bugzilla.gnome.org/show_bug.cgi?id=764947
-
time in segment should be PTS based (not DTS). https://bugzilla.gnome.org/show_bug.cgi?id=765498
-
- Jan 08, 2017
-
-
Jan Schmidt authored
If a fragmented stream doesn't have a tfdt, don't reset the output timestamps at each fragment boundary by erroneously using the default value of 0. Introduced by commit 69fc48 https://bugzilla.gnome.org/show_bug.cgi?id=754230
-
- Dec 22, 2016
-
-
Sebastian Dröge authored
Also consume the data entry by entry to get complicated indexing out of the code. https://bugzilla.gnome.org/show_bug.cgi?id=776107
-
-
Add check for node length of video sample description and its fields and for the XiTh atom. Also unify the code a bit. https://bugzilla.gnome.org/show_bug.cgi?id=775794
-
- Dec 13, 2016
-
-
Sebastian Dröge authored
That is, whenever we go through start/stop we have to ensure that on the next opportunity the buffers are reallocated again. Otherwise the buffers might be NULL because the element was reused with the same configuration as before (i.e. set_caps() wouldn't have reinited the buffers). https://bugzilla.gnome.org/show_bug.cgi?id=775898
-