summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2016-12-10docs: design: remove, moved to gst-docsHEADmasterTim-Philipp Müller4-272/+0
2016-12-09meson: Support building without Gst debugThibault Saunier1-0/+24
2016-12-09flxdec: Only send SEGMENT events after CAPSSebastian Dröge2-8/+29
I.e., don't just forward the event but delay it if we don't have caps on the srcpad yet.
2016-12-09flxdec: Unref and unmap buffers in all code paths as neededSebastian Dröge1-3/+2
https://bugzilla.gnome.org/show_bug.cgi?id=775888
2016-12-08v4l2object: Don't set empty interlace-mode listArun Raghavan1-1/+6
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
2016-12-07monoscope: Unref allocation query after finished with itGarima Gaur1-0/+1
https://bugzilla.gnome.org/show_bug.cgi?id=775752
2016-12-06flxdec: Allocate 0-initialized memory for the decoded frameSebastian Dröge1-2/+2
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
2016-12-05rtspsrc: Fix session cleanup when handling redirect on PLAYMatt Staples1-1/+0
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
2016-12-05rtspsrc: always send teardown requestAleix Conchillo Flaque1-3/+9
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
2016-12-03Automatic update of common submoduleEdward Hervey2-1/+1
From f980fd9 to 39ac2f5
2016-12-02jitterbuffer: Don't leak duplicate itemsEdward Hervey2-1/+5
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
2016-12-02qtdemux: Sanitize unknown codec capsEdward Hervey1-18/+21
We might have non-printable characters in the unknown fourcc, replace them with '_', in the same way we do it for unknown tags.
2016-12-01avidemux: Free vprp chunk also if it existed but we made no use of itSebastian Dröge1-3/+2
https://bugzilla.gnome.org/show_bug.cgi?id=775479
2016-12-01matroskademux: Fix memory leak when parsing attachmentsSebastian Dröge1-3/+2
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
2016-12-01matroskademux: Unify zlib/bzip2 decompress loops with the ones from qtdemuxSebastian Dröge1-19/+23
Especially, simplify the code a bit.
2016-12-01qtdemux: Increase inflate buffer in bigger stepsSebastian Dröge1-6/+4
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.
2016-12-01qtdemux: Ensure that size of the pasp atom is as much as we needSebastian Dröge1-2/+8
https://bugzilla.gnome.org/show_bug.cgi?id=775455
2016-12-01qtdemux: Free compressed moov node and it's corresponding decompressed dataSebastian Dröge1-3/+23
https://bugzilla.gnome.org/show_bug.cgi?id=775455
2016-12-01qtdemux: Check size of compressed MOOV header against available dataSebastian Dröge1-1/+11
And actually read the size of the cmvd atom from the right position. https://bugzilla.gnome.org/show_bug.cgi?id=775455
2016-12-01qtdemux: Fix zlib inflate loopSebastian Dröge1-39/+46
Handle errors cleanly, deallocate all memory and return the actual size of the inflated data. https://bugzilla.gnome.org/show_bug.cgi?id=775455
2016-12-01aacparse: Make sure we have enough data in the codec_data to be able to parse itSebastian Dröge1-2/+3
Also error out cleanly if mapping the buffer failed. https://bugzilla.gnome.org/show_bug.cgi?id=775450
2016-12-01qtdemux: Fix out of bounds read in tag parsing codeSebastian Dröge1-2/+2
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
2016-12-01rtp: Fix some memory leaks in usage of gst_pad_get_current_caps()Garima Gaur2-0/+3
https://bugzilla.gnome.org/show_bug.cgi?id=775071
2016-11-30qtdemux: Read interlacing information from 'fiel' atomVivia Nikolaidou1-0/+32
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
2016-11-29qtdemux: Fix compiler warningSebastian Dröge1-1/+1
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, ^
2016-11-29qtdemux: Change off_t type to gintScott D Phillips1-4/+3
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
2016-11-29meson: add libm to has_function checksScott D Phillips1-2/+3
The functions from math.h may be implemented in libm. https://bugzilla.gnome.org/show_bug.cgi?id=774876
2016-11-28Revert "meson: dv plugin now works on MSVC"Nirbheek Chauhan1-1/+4
This reverts commit 05a89613feff70cff416367f5aa807a1d5c68b63. Let's not put in stuff that needs unreleased Meson. This can go in for the next cycle.
2016-11-28avidemux: Ensure that tags are valid UTF-8 before adding them to the taglistSebastian Dröge1-9/+16
https://bugzilla.gnome.org/show_bug.cgi?id=775219
2016-11-28multipartdemux: Post an error message on the bus if we got EOS without ↵Sebastian Dröge1-0/+28
having added any pads
2016-11-28souphttpsrc: Handle non-UTF8 headers and error reasons more gracefullySebastian Dröge1-48/+106
Especially don't put them into GstStructures in one way or another, just ignore them or error out cleanly depending on the importance of their content.
2016-11-28vrawpay: Error out cleanly if mapping the video frame failsSebastian Dröge1-1/+4
Instead of later dereferencing NULL and crashing.
2016-11-27rtprtxsend: Update statistics before pushingEdward Hervey1-2/+3
If an element queries the number of retransmission buffers pushed *while* the push is still taking place (and before the object lock is taken just after) it would end up with the wrong statistic being reported. Increment it just before the push, avoids races when getting statistics https://bugzilla.gnome.org/show_bug.cgi?id=768723
2016-11-26common: use https protocol for common submoduleTim-Philipp Müller1-1/+1
https://bugzilla.gnome.org/show_bug.cgi?id=775110
2016-11-24gstv4l2bufferpool: lock flush_stop against regular qbufPhilipp Zabel1-8/+13
These can be called from different threads and both manipulate the pool->buffers array. Lock them properly and let flush_stop move the array contents into a temporary array on the stack to avoid having to call release_buffer under the object lock. https://bugzilla.gnome.org/show_bug.cgi?id=775015
2016-11-24gstv4l2bufferpool: remove critical error message when process is called on ↵Philipp Zabel1-2/+0
an inactive pool If the pool is inactive, it is guaranteed to also be flushing, so the following check will return GST_FLOW_FLUSHING anyway. This can happen if a v4l2src is blocking on DQBUF in create and is sent an EOS event on another thread. In that case the pool is set to flushing/inactive without locking, the v4l2src is unblocked, and may call pool_process with a valid buffer on the already inactive pool. https://bugzilla.gnome.org/show_bug.cgi?id=775014
2016-11-24v4l2src: release buffer if create failsPhilipp Zabel1-0/+1
gst_base_src_get_range does not expect a buffer to be returned in the error case, so we are leaking a reference here if create fails. https://bugzilla.gnome.org/show_bug.cgi?id=775014
2016-11-23rtpbin: Handle create_session() returning NULL in bundle codeSebastian Dröge1-0/+6
CID 1394492.
2016-11-23qtmux: Make sure to only change DTS of writable buffersSebastian Dröge1-2/+2
And trivial cleanup https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-23qtmux: Error out much earlier if we don't have a valid PTSSebastian Dröge1-3/+3
https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-23qtmux: Only use buffer durations if they are actually validSebastian Dröge1-19/+21
https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-23qtmux: Revert commits that set DTS and duration on buffers unconditionallySebastian Dröge1-28/+2
39f7e52266fde3b3c035e22cbcbb2bb1fa207b17 was setting the buffer duration to 0 if is not valid, under the assumption that this is "the last" buffer and no others are coming next. This is wrong, last_buf is the previous buffer and not the very last one. 4e3c13c87c258c9c95e2217d32ab314d12b5fffc was setting DTS to 0 if there was none. This will set DTS to 0 for all e.g. audio streams, completely messing up calculations if streams don't start at 0. https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-23qtmux: Only write "gap" edit list if there is a non-zero gapSebastian Dröge1-4/+7
https://bugzilla.gnome.org/show_bug.cgi?id=774840
2016-11-23flxdec: rewrite logic based on GstByteReader/WriterMatthew Waters4-260/+427
Solves overreading/writing the given arrays and will error out if the streams asks to do that. Also does more error checking that the stream is valid and won't overrun any allocated arrays. Also mitigate integer overflow errors calculating allocation sizes. https://bugzilla.gnome.org/show_bug.cgi?id=774859
2016-11-23flxdec: Don't unref() parent in the chain functionSebastian Dröge1-1/+0
We don't own the reference here, it is owned by the caller and given to us for the scope of this function. Leftover mistake from 0.10 porting. https://bugzilla.gnome.org/show_bug.cgi?id=774897
2016-11-22vpxdec: libvpx's release buffer is sometimes called with fb->priv==NULLSebastian Dröge1-1/+4
Don't assert on this but just ignore these cases.
2016-11-22matroskademux: Fix cluster searching if we search multiple times in one chunkSebastian Dröge1-2/+11
After finding a cluster id in the byte reader, we skip ahead the reader position by one further byte to be able to continue searching from there inside the same chunk if the cluster candidate was a false positive. We have to accomodate for that additional byte when resuming the search, otherwise all following pulls are off-by-one for every resume and we run into an assertion.
2016-11-22matroska: Add size checks to the parsing of FLAC headersSebastian Dröge1-3/+6
2016-11-22flxdec: fix some warnings comparing unsigned < 0Matthew Waters1-2/+2
bf43f44fcfada5ec4a3ce60cb374340486fe9fac was comparing an unsigned expression to be < 0 which was always false. gstflxdec.c: In function ‘flx_decode_brun’: gstflxdec.c:322:33: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits] if ((glong) row - count < 0) { ^ gstflxdec.c:332:33: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits] if ((glong) row - count < 0) { ^ https://bugzilla.gnome.org/show_bug.cgi?id=774834
2016-11-22qtmux: Enable up to 16 unpositioned raw audio channelsVivia Nikolaidou1-0/+8
https://bugzilla.gnome.org/show_bug.cgi?id=774789