rtph263pay: fix leak

CID 1212156

rtph263pay: remove uneeded variable

We just need to save the ebit information in case there is an error decoding.

vp[89]enc: Reset the encoder when flushing

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

matroska: error mode if can't push buffer

If gst_pad_push() fails, inform and return flow error.

matroska: unused value

Value set in ret will be overwritten just before exiting the function.

CID #1226469

rtpjitterbuffer: Drop packets with sequence numbers before the seqnum-base

These are outside the expected range of sequence numbers and should be
clipped, especially for RTSP they might belong to packets from before a seek
or a previous stream in general.

rtspsrc: Don't include payload type in the caps for framesize

When the sdp media attribute framesize are converted to caps
the <payload> should not be included.

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

rtpjitterbuffer: Don't forget to unlock the mutex when receiving GAPs in TCP streams

pulsesink: Make sure to filter caps in all cases during CAPS query

We were skipping the filter step while returning template caps, for
example.

v4l2bufferpool: Don't update buffer for OUTPUT

For output device, we should not update the buffer with flags and
timestamp when we dequeue. The information in the v4l2_buffer is not
meaningful and it breaks the case where the buffer is rendered at
multiple places.

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

souphttpclientsink: Implement cookies property

souphttpclientsink: Implement automatic-redirect property

souphttpclientsink: Implement proxy support

The properties were there before, but not used anywhere.

avidemux: resurrect some flow return handling

aacparse: fix LOAS parsing issue

Fix missing index in syncword searching

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

directsoundsink: fix modulo math with ringbuffer parameters

To get a multiple of bpf use a subtraction and not an addition

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

splitmuxsink: Protect property variables with the object lock.

Use the object lock instead of the splitmux lock to protect
internal property variables, so they're not locked when
switching to a new file.

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

check: add jitterbuffer unit test

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

rtspsrc: Fix handling of interleaved (TCP) streams

We need to set up the transport in any case, not just if we have a container
stream or a non-interleaved stream. Only if we have an interleaved stream and
are retrying, we should not set up the stream again.

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

vp[89]dec: Drop frames that have no output buffer because of errors

finish_frame() assumes that there is an output buffer.

rtspsrc: Don't unref caps we don't own

rtspsrc: Push RTCP caps on the RTCP pads

Otherwise we will get not-negotiated later from rtpbin, and will never be able
to send RTCP packets back to the server. Note that error flow returns from the
RTCP pads are ignored, that's why it didn't fail more visible before.

rtspsrc: Make sure to send SEGMENT events on all pads

rtp: Add Full Intra Request (FIR) packets to statistics

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

rtp: Add Packet Loss Indication (PLI) to statistics

This is helpful to provide statistics in the format defined in
http://w3c.github.io/webrtc-stats/#dictionary-rtcrtpstreamstats-members.

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

matroskamux: Remove duration accumulation logic

Duration accumulation can cause rounding errors and generate wrong
duration with different buffers that share the same timestamp.

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

matroska: Add an helper method to get buffer timestamps

... and replace GST_BUFFER_TIMESTAMP that always return PTS with this method
that return PTS or DTS based on stream type.

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

rtpsession: Add explanation why we have space for 32 hash tables

And also create only one, there's no need yet to create all 32 until
we implement RFC2762.

Revert "rtpsession: Do not use an array of maps if they are not being used"

This reverts commit 1591adf4cd843d13d8622a30c619425691a84128.

https://bugzilla.gnome.org/show_bug.cgi?id=745586#c1:
It's the beginning of an implementation of RFC 2762, which is needed for
large multicast groups. The implementation is not yet complete but why
not leave what is there and implement RFC 2762 instead?

rtpsession: Do not use an array of maps if they are not being used

rtpsession declares an array of maps to store srrcs but only the
the key 0 is being used. This patch replaces the array of maps
for just one map and remove useless parameters in rtpsession

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

avidemux: remove not needed code

In gst_avi_demux_handle_src_query, there is not needed code.
We already check about stream is vbr or not at the upper line.
o, we don't need to check this condition becase stream is not
vbr 100% in this case.

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

tests: gdkpixbufoverlay-test: replace deprecated function

Just avoid using the deprecated function entirely,
it's easy enough. Defining the macro is not enough.

tests: gdkpixbufoverlay-test: fix compilation against newer gdk-pixbuf

gdk_pixbuf_new_from_inline() has been deprecated in favour
of GResource.

osxaudiosrc: Allow caps renegotiation

The ringbuffer does allow renegotiation, so we do not have to report
fixed caps once it is acquired (based on a similar patch for the sink
side by Ilya Konstantinov <ilya.konstantinov@gmail.com>).

osxaudiosink: Allow renegotiating caps

Once osxaudiosink's device is open, it fixates on the initial caps and
refuses to accept new caps. This is erroneous since the Audio Unit is
can accept a new ASBD, and GstAudioRingBuffer supports reconfiguration
as well.

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

v4l2allocator: fix fd leak in DMABUF import mode.

Ensure gst_v4l2_buffer_pool_release_buffer() releases the associated
GstV4l2MemoryGroup. In particular, this allows for closing the DMABUF
handles prior to instantiating new ones.

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

vp8enc: Use 0 as duration for the EOS "frame"

vp{8,9}enc: Tell the encoder about actual timestamps and durations of frames

... instead of just counting frames. The values are supposed to be in timebase
units, not frame units. This fixes various quality problems with VP8/VP9
encoding and in general makes the encoder behave better.

Thanks to Nirbheek Chauhan for noticing this bug.

vpxdec: Fix calculation of width in bytes

Right now we only support I420, but vpx seems to support more formats.
This will prevent hard to find bug in the future.

vpxdec: Don't memcpy in frame map failed

This avoid a crash if mapping the frame failed.

v4l2bufferpool: Add missing break

This is cosmetic change.

v4l2: Workaround driver not setting field correctly

As it's very common, handle driver not setting field in buffers
by using the field value from the format. This workaround a long time
bug in UVC driver. For even buggier driver, we simply assume
progressive as before. We also only warn once, to avoid spamming.

qtdemux: fix key unit seek

Unlike many other seek flags, the KEY_UNIT seek
flag is not copied over into the GstSegment,
since it's only relevant for the seek itself,
so we need to pass it explicitly to the seek
handler here.

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

docs/plugins: Updates

matroskamux/demux: initialize dts_only

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

matroskamux: store DTS for V_MS/VFW/FOURCC streams

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

multifile: attempt to fix docs build issue on build bot

interleave: Drop custom latency query handling

This is implemented by the default query handler now.

videomixer: Drop custom latency querying logic

This is now implemented in the default latency query handler.

rtpvorbispay: fix payloader description and author e-mail

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

v4l2: query crop configuration after each call of S_CROP

S_CROP ioctl is write-only and the device can adjust crop rectangle so
we query back the crop configuration after each S_CROP to know what has
been done.

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

matroskademux: V_MS/VFW/FOURCC streams have DTS instead of PTS

When such stream is present demuxer should set DTS on buffers instead
of PTS. This is consistent with how VLC and libav/ffmpeg handle VFW
streams.

Sample file
https://s3.amazonaws.com/MatejK/Samples/Matroska-VFW-DTS-Only.mkv

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

v4l2bufferpool: Check corruption flag on the right buffer

We where checking the buffer we are copying to instead of the buffer we
are copying from.

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

v4l2object: set colorspace in caps for capture devices

This information is set by the driver for a capture device, and so could
be forwarded to pipeline by setting the colorimetry in caps.

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

v4l2bufferpool: fix import_userptr() in single-planar API when n_planes > 1

In the V4L2 single-planar API, when format is semi-planar/planar,
drivers expect the planes to be contiguous in memory.
So this commit change the way we handle semi-planar/planar format
(n_planes > 1) when we use the single-planar API (group->n_mem == 1).

To check that planes are contiguous and have expected size, ie: no
padding. We test the fact that plane 'i' start address + plane 'i'
expected size equals to plane 'i + 1' start address. If not, we return
in error.

Math are done in bufferpool rather than in allocator because the
former is aware of video info.

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

v4l2allocator: let bufferpool calculate image size when importing userptr

Offset are relative to the buffer and there is no guarantee substracting
them will give us the plane size. So we let bufferpool make the math as
it is more aware of video info than allocator and pass a size array to
allocator import function.

Pointed out by Nicolas Dufresne <nicolas.dufresne@collabora.com>

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

v4l2object: recognize and distinguish all bayer arrangements

Up to now, v4l2src recognized only "bggr" amongst the bayer arrangements.
Recognize now also the "rggb", "gbrg" and "grbg" arrangements.

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

v4l2bufferpool: set v4l2_buffer.field when queuing buffer in an output device

According to the current specification, application must set this field
for an output device.

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

osxaudiosrc: iOS resampling causes stuttering

Fixes stuttering audio when iOS AU is resampling. To make AU resample,
one has to request a rate that differs from AVAudioSession's
sampleRate. The resampling itself is not the culprit, but rather our
API misuse.

AudioUnitRender modifies the mDataByteSize members with the
actual read bytes count. Therefore, they must be reinitialized
before each AudioUnitRender. (The buffers themselves can be
preallocated.)

The "stutter" was caused by one AudioUnitRender making the buffer
too small for other AudioUnitRender invocations, making them fail
with -50 (paramErr). By way of luck, when AU didn't resample, all
AudioUnitRender invocations read the same number of bytes.

(This patch addresses some non-interleaved audio concerns, but
at this moment the elements do not support non-interleaved audio
and non-interleaved is untested.)

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

rtspsrc: improve error message when unauthorized

Make use of NOT_AUTHORIZED error code instead of falling back to generic
READ error.

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

ximagesrc: remove pointless g_return_val_if_fail()

ximage won't ever be NULL here because the dispose
function is called via ximage->dispose().

qtdemux: All segment resulting from a seek should have the same seqnum

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

v4l2: Enable copy when no known allocation params

When there is no allocation parameters in the query, enable copy
threshold. When this threshold is reached, the buffer pool will start
copying when the pool reaches a critical level. If the driver supports
CREATE_BUFS, this will be used instead.

v4l2bufferpool: Update allocator flags

When we hit emulated formats, we disable CREATE_BUFS since libv4l2
cope very badly with it. Also clear the allocator flags so we will
never try to allocate more buffers. This fixes failure when the copy
threshold is reached as we where calling CREATE_BUFS, which lead to
libv4l2 instability.

v4l2bufferpool: Use specific debug category

The pool has grown enough that it is now handy to seperate v4l2object
trace from v4l2bufferpool trace.

rtpvp8pay: default encoding name to VP8

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

rtpvp8pay: make caps writable before truncating them

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

rtpvp8pay: negotiate encoding name

Chrome uses a different one than gstreamer.

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

rtpsession: Send initial events on sync_rtcp pad when using RTP/RTCP muxing

Otherwise we will just send buffers on the pad without any events beforehand
and will get g_warnings() about that.

jack: case missing break statement

commit b1098c2ea5eabea7af08ce51d22b867eaed2bbe2 added a new case in
gst_jack_audio_src_get_property() but forgot to add the break statement to it.

Revert "v4l2: fraction is reversed"

This reverts commit b91fe36644b15ae070d72b9e8a9c7087e82aef12.

v4l2: fraction is reversed

In the fraction 1 / 2. 1 is the numerator and 2 is the denominator.
The arguments of fraction gst_value_set_fractions() are value,
numerator and denominator.
Also, gst_value_set_fraction() fails if denominator is 0 for obvious
reasons.

v4l2pool: Deactivate other pool

When importing buffers from a downstream pool, we need to deactivate
that pool to ensure it will be usable again later. Relying on the
refcount to reach zero does not work, since elements like xvimagesink
keeps a reference on their proposed pool.

qtmux: remove not needed condition

gst_buffer_replace can handle NULL inputs by itself

qtdemux: prefer the tfdt timestamp over the buffer's that is less accurate

The tfdt should be more accurate as the buffer timestamp is provided
by the fragmented format manifest and it might just be an approximation.

rtpjitterbuffer: When resetting the jitterbuffer because of packet discont, don't flush sticky events

We will otherwise flush away STREAM_START, CAPS or SEGMENT events and will
confuse downstream with buffers that come before such events.

jack: Add property port-pattern to specify which JACK ports to connect to

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

isomp4: Redefine gst_isoff_ symbols to gst_isoff_qt_

We need different symbol names, because these symbols are also present
in the fragmented plugin ... which will cause conflicts when doing
static linking

goom2k1: use fractional part of float division

splitmuxsin: remove dead code

Every instance of goto beach has buf_info equal NULL. Don't check
for a condition that never happens.

CID #1268399

splitmux-test: Parse error message

The test had a function to print the error, but was not parsing it.
This was causing warning about dbg_info being used uninitialized. If
the test was testing any errors, this would have crashed.

spectrum: Fix min and max for bands property

The number of FFTs is calculated with the following formula:

  guint nfft = 2 * bands - 2;

nfft is passed to gst_fft_f32_new() as the len argument and is of type
unsigned integer. This method required that len is at leas 1, then
maximum G_MAXINT, as other values would be negative. If we extrapolate
from the formula above it means we need "bands" to be between 2 and
((guint)G_MAXINT + 2) / 2).

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

v4l2allocator: Fix freeing of shared memory

When memory (that has been shared using gst_memory_share()) are freed,
the memory (or the DMABUF FD) should not bee freed. These memories have
a parent. This also removes the extra _v4l2mem_free function and avoid
calling close twice on the DMABUF FD.

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

qtdemux: do not use sparse streams in push-based seeking

Using the sparse streams can make the push-based seeking return
too far in the stream. It also can lead to issues as the
sparse streams will be ignored when restarting playback and,
 if the sparse stream is the one that has the earliest sample,
it will confuse qtdemux's offsets as one stream will have
an earlier offset than the demuxer's one which might lead to
early EOS.

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

pulsesink: Enhance code readability in pulsesink_query

In pulsesink_query function, we use a switch for the query
type. In the CAPS case, there is no 'break', instead we
return right away. Use a break and return at the end of
the function instead for better code readability.

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

splitmuxsink: flag as sink from the start

qtdemux: Initial 'sidx' atom parsing support

Parse the 'sidx' atom and update the total duration according to the
parser result. The isoff parser code is imported from
gst-plugins-bad's dashdemux and a gst_isoff_sidx_parser_add_data()
function was factored out of the gst_isoff_sidx_parser_add_buffer()
function.

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

flvdemux: Use gst_video_guess_framerate()

Use gst_video_guess_framerate() from libgstvideo to guess
sensible common framerates where possible from the
floating point fps in the stream.

Improve and fix LATENCY query handling

This now follows the design docs everywhere, especially the maximum latency
handling.

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

rtpsession: Handle first RTCP packet and early feedback correctly

According to RFC 4585 section 3.5.3 step 1 we are not allowed to send
an early RTCP packet for the very first one. It must be a regular one.

Also make sure to not use last_rtcp_send_time in any calculations until
we actually sent an RTCP packet already. In specific this means that we
must not use it for forward reconsideration of the current RTCP send time.
Instead we don't do any forward reconsideration for the first RTCP packet.

rtph263depay: fix compilation with gcc 5.0

splitmuxsink: fix example pipeline properly

x264enc might not have a max-key-int property, but it
has a key-int-max property...

splitmux: fix typo

splitmux: update example pipeline

Element x264enc doesn't have a max-key-int property

splitmux: fix memory leak

If execution goes to the beach in line 981, buf_info goes out of scope without
the memory being free'd. Handle this case.

CID #1268403

rtspsrc: fix awkward if clause

splitmux: Add unit test for file splitting

Add a unit test for file splitting, and fix the leaks in the
splitmuxsink it found

wavparse: fix which stop variable is used in assignment

Assignment is done to variable segment.stop when the intention was to assign to
local variable stop. Instead of overwriting it, the value is now clamped and
segment.stop is set to it soon after.

CID #1265773

splitmux: Fix memory leaks until the test valgrinds clean

splitmux: Handle early EOS during part preparation

Handle the case where a short file reaches EOS while we're still
waiting for no-more-pads, and make sure we continue to the internal
READY state for real playback to work properly later.