- 14 Dec, 2016 8 commits
-
-
Reynaldo H. Verdejo Pinochet authored
-
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
-
Víctor Manuel Jáquez Leal authored
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
-
Havard Graff authored
* Changed PCMU->TEST for common macros * Changed verify-functions (lost & rtx) into macros. * Remove option to add marker-bit for test-buffers (not used anywhere) * Add new push_test_buffer function that makes sure there are correlation between dts and the time on the clock. (classic test-mistake) * Established a generic starting-point for tests with the construct_deterministic_initial_state function and use it where applicable, which removes lots of "boilerplate" everywhere. * Add basic lost-event test * Remove as much "magic constants" as possible. * Remove 3 tests that no longer are testing anything that others don't, and was completely unmaintainable. * Remove unnecessary use of the testclock * Verify each test is testing what it actually says it does (and modify where it doesn't) In general, make the tests much smaller, better, more maintainable and readable. https://bugzilla.gnome.org/show_bug.cgi?id=774409
-
Tim-Philipp Müller authored
Likely extremely bitrotten, and we should not ship this anyway.
-
-
Heekyoung Seo authored
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
-
Heekyoung Seo authored
Add support for xvid video and mp2 audio, add m2v1 fourcc. https://bugzilla.gnome.org/show_bug.cgi?id=775794
-
- 13 Dec, 2016 1 commit
-
-
Sebastian Dröge authored
This is a C99 feature.
-
- 11 Dec, 2016 1 commit
-
-
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
-
- 10 Dec, 2016 1 commit
-
-
Tim-Philipp Müller authored
-
- 09 Dec, 2016 3 commits
-
-
Thibault Saunier authored
-
Sebastian Dröge authored
I.e., don't just forward the event but delay it if we don't have caps on the srcpad yet.
-
-
- 08 Dec, 2016 1 commit
-
-
Arun Raghavan authored
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
-
- 07 Dec, 2016 1 commit
-
-
- 06 Dec, 2016 1 commit
-
-
Sebastian Dröge authored
Otherwise we might leak arbitrary information from the uninitialized memory if not every pixel is written. https://scarybeastsecurity.blogspot.gr/2016/12/1days-0days-pocs-more-gstreamer-flic.html
-
- 05 Dec, 2016 2 commits
-
-
Matt Staples authored
Redirect on PLAY wasn't doing the necessary session cleanup. Fixed by removing code from gst_rtspsrc_send that changed the state varable upon encountering a redirect. Better to let the redirect handlers in gst_rtspsrc_retrieve_sdp and gst_rtspsrc_play do their own state-dependent cleanup. https://bugzilla.gnome.org/show_bug.cgi?id=775543
-
Aleix Conchillo Flaque authored
Allow CMD_CLOSE to cancel all commands not only CMD_PAUSE and ignore CMD_WAIT while closing. https://bugzilla.gnome.org/show_bug.cgi?id=748360
-
- 03 Dec, 2016 1 commit
-
-
Edward Hervey authored
From f980fd9 to 39ac2f5
-
- 02 Dec, 2016 2 commits
-
-
Edward Hervey authored
When providing items with a seqnum, there is a (very small) probability that an element with the same seqnum already exists. Don't forget to free that item if it wasn't inserted. And avoid returning undefined values when dealing with duplicate items
-
Edward Hervey authored
We might have non-printable characters in the unknown fourcc, replace them with '_', in the same way we do it for unknown tags.
-
- 01 Dec, 2016 11 commits
-
-
-
Sebastian Dröge authored
gst_tag_image_data_to_image_sample() does not take ownership of the passed memory, so don't set it to NULL to allow us to free it later. https://bugzilla.gnome.org/show_bug.cgi?id=775472
-
Sebastian Dröge authored
Especially, simplify the code a bit.
-
Sebastian Dröge authored
1024 bytes is quite small, let's do 4096 bytes (or one page). Also remove redundant if, we're always in that case when getting here.
-
-
-
Sebastian Dröge authored
And actually read the size of the cmvd atom from the right position. https://bugzilla.gnome.org/show_bug.cgi?id=775455
-
Sebastian Dröge authored
Handle errors cleanly, deallocate all memory and return the actual size of the inflated data. https://bugzilla.gnome.org/show_bug.cgi?id=775455
-
Sebastian Dröge authored
Also error out cleanly if mapping the buffer failed. https://bugzilla.gnome.org/show_bug.cgi?id=775450
-
Sebastian Dröge authored
We can't simply assume that the length of the tag value as given inside the stream is correct but should also check against the amount of data we have actually available. https://bugzilla.gnome.org/show_bug.cgi?id=775451
-
-
- 30 Nov, 2016 1 commit
-
-
Vivia Nikolaidou authored
Read interlacing and TFF/BFF information from the 'fiel' atom and pass it into the caps https://bugzilla.gnome.org/show_bug.cgi?id=775414
-
- 29 Nov, 2016 2 commits
-
-
Sebastian Dröge authored
qtdemux.c: In function ‘qtdemux_parse_trak’: qtdemux.c:10184:38: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 9 has type ‘gint {aka const int}’ [-Werror=format=] GST_DEBUG_OBJECT (qtdemux, "Found jpeg: len %u, need %lu", len, ^
-
Scott D Phillips authored
off_t is a signed integer type provided by sys/types.h on posix systems. Replace with gint for building on non-posix systems (like windows). https://bugzilla.gnome.org/show_bug.cgi?id=775287
-
- 28 Nov, 2016 4 commits
-
-
Scott D Phillips authored
The functions from math.h may be implemented in libm. https://bugzilla.gnome.org/show_bug.cgi?id=774876
-
Nirbheek Chauhan authored
This reverts commit 05a89613. Let's not put in stuff that needs unreleased Meson. This can go in for the next cycle.
-
-
Sebastian Dröge authored
-