v4l2allocator: Use qdata instead of parenting to DmabufMemory

Parenting V4l2Memory to DmabufMemory was in conflict with recent
optimization in DmabufMemory to avoid dup(), and didn't work with
memory sharing. Instead, use a qdata and it's destroy notify.

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

po: Update translations

v4l2bufferpool: Workaround elements not requesting any buffers

This is a workaround for element that don't request buffers when
they should.

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

tests: fix pipeline leak in videocrop test

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

examples: client-rtpaux: Release reference to parent when done

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

rtspsrc: fix query leak

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

wavenc: Return not-negotiated if we got no caps or caps negotiation failed

And do it always, not inside a g_return_val_if_fail().

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

v4l2src: Ensure internal pool activation

Before we would hit an assertion "'gst_buffer_pool_is_active (bpool)' failed"
if the internal pool was not used to push buffer downstrea, hence not
given to the baseclass.

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

videomixer: fix double unlock in segment seek segment code path

We only want to unlock if we push an event downstream and
jump to done_unlock label afterwards. We would also unlock
in case of a segment seek and then unlock again later, and
nothing good can come of that.

(This code looks a bit dodgy anyway though, shouldn't it
also bail out with FLOW_EOS here in case of a segment seek
scenario, just without the event?)

tests: qtmux: suppress glib criticals caused by testing deprecated dts methods

avidemux, wavparse: Print invalid fourcc in hex

Previously this was printed as characters which caused later processing
of the error message to sometimes warn about non-UTF-8 characters.

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

v4l2object: Pool might be NULL in decide allocation

If special stride is needed and downstream don't support VideoMeta,
pool might be NULL in order to let the baseclass create a generic
pool­. This would lead to assertion with on Exynos with:

gst-launch-1.0 -v filesrc location=mov ! qtdemux ! h264parse ! \
                    v4l2video8dec ! fakesink

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

v4l2bufferpool: Handle FD error during poll

This will ensure we fail earlier if something unrecoverable
happens.

v4l2bufferpool: Wait before polling if queue is empty

In kernel before 3.17, polling during queue underrun would unblock right
away and trigger POLLERR. As we are not handling POLLERR, we would endup
blocking in DQBUF call, which won't be unblocked correctly when going
to NULL state. A deadlock at start caused by locking error in libv4l2 was
also seen before this patch. Instead, we wait until the queue is no longer
empty before polling.

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

rtspsrc: fix for mikey api change

v4l2: fix probing and enumeration of stepwise frame sizes

The code enumerating STEPWISE framesizes would start from
(min_w, min_h) and then add (step_w, step_h) to get the
next framesize. However, it should really allow any width
from min_w to max_w with step_w and same for heights.
Secondly, we would add and probe each individual stepped
frame size to the caps as separate structure, which would
lead to hundreds if not thousands of structs ending up in
the probed caps. Use integer ranges with steps instead.

This was particularly noticable with the Raspberry Pi Cam.

https://bugzilla.gnome.org/show_bug.cgi?id=724521
https://bugzilla.gnome.org/show_bug.cgi?id=732458
https://bugzilla.gnome.org/show_bug.cgi?id=726521

v4l2object: drop workaround for misbehaving TRY_FMT

This workaround from 2011 was causing 25 S_FMT ioctls to be sent
to my UVC webcam from under gst_v4l2_object_get_caps as it probes
all the formats. In total, this adds up to about 5 seconds of
execution time, or a 10 second delay while starting up cheese.

These ioctls come from a workaround from 2011 where TRY_FMT might
make changes to hardware settings, so S_FMT was used to restore
the original config:
https://bugzilla.gnome.org/show_bug.cgi?id=649067

The driver bug is now assumed fixed. Remove the workaround to fix the
long startup delay.

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

videomixer: reset QoS on segment event

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

matroskademux: send gap events instead of segment tricks

This fixes missing frames from being time skipped.

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

rtpsession: Fix memory leaks in unit test

rtpbin: Don't leak caps

pulsesrc: Fix compiler warning when compiling with G_DISABLE_ASSERT

rtpssrcdemux: Fix compiler warning when compiling with G_DISABLE_ASSERT

matroskamux: Fix compiler warnings when compiling with G_DISABLE_ASSERT

deinterlace: Fix compiler warnings when compiling with G_DISABLE_ASSERT

pulse: fix compiler warnings when compiling with -DG_DISABLE_ASSERT

Compiler complains about uninitialised variables in the impossible
'default' code path in device provider source/sink switch-case.

v4l2: fix compiler warnings when compiling with -DG_DISABLE_ASSERT

Compiler complains about uninitialised variables in the impossible
'default' code path in device provider source/sink switch-case.

tests: matroskaparse: fail on errors and disable pull mode test

Actually look for error messages on the bus and fail if there
is one before the EOS message. Disable pull mode test which is
pointless as long as matroskaparse only supports push mode
(pull mode support has not been ported over to 1.0).

matroskaparse: don't error out if there's not enough data in the adapter

gst_matroska_parse_take() would return FLOW_ERROR instead of
FLOW_EOS in case there's less data in the adapter than requested,
because buffer is NULL in that case which triggers the error
code path. This made the unit test fail (occasionally at least,
because of a bug in the unit test there's a race and it would
happen only sporadically).

videomixer: Update dist generated ORC files

videomixer: Update videoconvert code from -base

And also rename the remaining symbols to prevent conflicts
during static linking.

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

autovideosrc: use videotestsrc as fallback element instead of fakesrc

fakesrc doesn't announce video caps, so most video pipelines will
just error out with not-negotiated if a fallback element is created.

autoaudiosrc: use audiotestsrc as fallback element instead of fakesrc

fakesrc doesn't announce audio caps, so most audio pipelines will
just error out with not-negotiated if a fallback element is created.

Release 1.3.90

Update .po files

Rename GstDeviceMonitor to GstDeviceProvider

videobox: Add unit test

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

videomixer: Declare as Compositor in 'klass'

flvdemux: fix speex caps

Decoder complains about "notification: Invalid mode encountered.
The stream is corrupted" though, even if it works, so there's
probably something wrong with the generated codec headers.

flvmux: fix speex in FLV

Speex in FLV is always mono @ 16kHz, see
http://download.macromedia.com/f4v/video_file_format_spec_v10_1.pdf
section E.4.2.1: "If the SoundFormat indicates Speex, the audio is
compressed mono sampled at 16 kHz, the SoundRate shall be 0, the
SoundSize shall be 1, and the SoundType shall be 0"

Also see https://bugzilla.gnome.org/show_bug.cgi?id=683622

isomp4: Add object type id and fourcc for DTS/DTS-HD

Enables playback for files with DTS audio tracks.
Also add an extra AC-3 variant fourcc from Nero

videomixer2: Solve segmentation fault when src caps are configured

Change function pointers to NULL while holding the lock to avoid
race conditions

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

jitterbuffer: improve SR packet handling

Implement 3 different cases for handling the SR:

 1) we don't have enough timing information to handle the SR packet and
    we need to wait a little for more RTP packets. In that case we keep
    the SR packet around and retry when we get an RTP packet in the
    chain function.

 2) the SR packet has a too old timestamp and should be discarded. It is
    labeled invalid and the last_sr is cleared.

 3) the SR packet is ok and there is enough timing information, proceed
    with processing the SR packet.

Before this patch, case 2) and 1) were handled in the same way,
resulting that SR packets with too old timestamps were checked over and
over again for each RTP packet.

tests: add udpsink test to check client add/remove

tests: port udpsink tests to 1.0

They all seem a bit pointless though.

avimux: Add UYVY format

gstrtpssrcdemux: manage ssrc of RTCP RR packets

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

wavparse: Update offset after parsing adtl chunk

Otherwise we will parse it over and over again without ever
getting past it.

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

osxvideosink: remove legacy code for passing a window handle

"have-ns-view" and the "embed" property was kept in 0.10 for
backwards compatibility but it's no longer used in favor of
the GstVideoOverlay interface

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

Back to development

matroskademux: Don't call GST_DEBUG_OBJECT() and other macros with non-GObject objects

It will crash with latest GLib GIT and was never supposed to work before
either.

Release 1.3.3

Update .po files

po: Update translations

pulse, v4l2: update for device "klass" -> "device-class" rename

multiudpsink: optimisation: avoid unnecessary memory ref/unrefs

We know the buffer will stay valid and we will also not
modify the buffer, we just want to send out the data.

multiudpsink: avoid some unnecessary run-time type checks

rtspsrc: pass the stream id when asking for crypto params

This way the app can choose different parameters for each stream.

rtspsrc: add support for key length parameters

This patch adds supports for the incoming key management parameters for
encryption and authentication key lengths.

It also adds a new signal request-rtcp-key that allows the user to
provide the crypto parameters and key for the RTCP stream.

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

vp8depay: fix header size checking

Use a different variable name to make it clear that we are calculating
the header size.
Correctly check that we have enough bytes to read the header bits. We
were checking if there were 5 bytes available in the header while we
only needed 3, causing the packet to be discarded as too small.

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

rtph264pay: propagate the GST_BUFFER_FLAG_DISCONT flag

Similarly to what we did with the DELTA_UNIT flag, this patch
propagates the DISCONT flag to the first RTP packet being used to transfer a
DISCONT buffer.

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

rtph264pay: propagate the GST_BUFFER_FLAG_DELTA_UNIT flag

Downstream elements may be interested knowing if a RTP packet is the start
of a key frame (to implement a RTP extension as defined in the
ONVIF Streaming Spec for example).

We do this by checking the GST_BUFFER_FLAG_DELTA_UNIT flag we receive from
upstream and propagate it to the *first* RTP packet outputted to transfer this
buffer.

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

gstrtpmp4gpay: propagate the GST_BUFFER_FLAG_DISCONT flag

Propagate the DISCONT flag to the first RTP packet being used to transfer
a DISCONT buffer.

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

rtpjpegpay: propagate the GST_BUFFER_FLAG_DISCONT flag

Propagate the DISCONT flag to the first RTP packet being used to transfer
a DISCONT buffer.

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

avidemux: don't leak flow combiner

rtpjp2kpay: pre-allocate buffer-list of the right size

rtpjpegpay: pre-allocate buffer list of the right size

rtpmp4vpay: pre-allocate buffer list of the right size

rtpvp8pay: allocate bitreader on the stack

rtpvp8pay: post error message on bus on error and don't use g_message()

rtpvp8pay: couple of minor optimisations

Pre-allocate buffer list of the right size to avoid re-allocs.
Avoid plenty of double runtime cast checks and re-doing the
same calculation over and over again in rtp_vp8_calc_payload_len().
Only call gst_buffer_get_size() once.

rtpgstpay: pre-allocate buffer list of the right size

To avoid re-allocs.

rtph264pay: pre-allocate bufferlist of the right size

To avoid unnecessary re-allocs.

rtph264pay: push single buffer directly, no need to wrap it in a bufferlist

No point in a buffer list if we just have one single
buffer to push. Fix up unit test to handle that case
as well.

rtpvrawpay: make chunks per frame configurable

Bit of a misnomer because it's really chunks per field
and not per frame, but we're going to ignore that for
the time being.

rtpvrawpay: remove unused variables

rtpvrawpay: pre-allocate buffer lists of sufficient size

Avoids unnecessary reallocs when appending buffers
to the bufferlist.

rtpvrawpay: micro-optimise variable access in inner loop

Store some values that don't change during the execution
of the inner loops locally, so the compiler knows that too.

rtpvrawpay: use buffer lists

Collect buffers to send out in buffer lists instead of
pushing out single buffers one at a time. For HD video
each frame might easily add up to a couple of thousand
packets, multiply that by the frame rate and that's a
lot of push() and sendmsg() calls per second.

A good reason to push out buffers as early as possible is
latency, so we don't accumulate the whole frame in a single
buffer list, but instead push it out in a few chunks, which
is hopefully a reasonable compromise.

udp: improve element descriptions for dynudpsink and multiudpsink

udp: remove suppression of compiler warnings for deprecated GLib API

Not needed any more.

videobox: Fix caps negotiation issue

Make sure that if AYUV is received it will detect that it can produce
both RGB and YUV formats

Signed-off-by: Ravi Kiran K N <ravi.kiran@samsung.com>
https://bugzilla.gnome.org/show_bug.cgi?id=725248

rtptheoradepay: fix double frees

Fix double-frees introduced to fix another coverity report.

CID 1223053

dynudpsink: return FLUSHING when sendto got canceled, not an error

oss: simplify probed caps before returning them

Exposes all formats in the first structure if the
rest is the same for all of them.

oss: make sure 16-bit formats are before 8-bit formats in probed caps

Probe supported formats in order of desirability rather than in
what order they may happen to be in the formats bitmask. Fixes
accidentally exposure of 8-bit formats in caps before 16-bit formats
(in case where U16 was not supported S8 might be listed before S16).

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

v4l2bufferpool: Cleanly handle v4l2_allocator_new failure

rtptheordepay: fix leaks

Coverity 1212163

rtpg729pay: leak fixes

Coverity 1212159

rtph263pay: fix leak

Coverity 1212157

rtph263pay: fix leaks

Coverity 1212149

rtpdvpay: catch failures to map buffer

Coverity 1139741

multipartdemux: guard against having no MIME type

The code would previously crash trying to insert a NULL string
into a hash table.
It does seem a little broken that indexing is done by MIME type
and not by index though, unless the spec says there cannot be
two parts with the same MIME type.

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

multipartdemux: Send stream-start event

This event was not sent. Send it before caps, this requires the pad to
be parented. This removes warning like: "Got data flow before
stream-start event".

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

qtdemux: avoid looping indefinitely in broken svq3 files

Abort if an atom with size 0 is read from within the svq3 stsd
atoms

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

flacdec: add const where appropriate

speexenc: add missing va_end in variadic function

Coverity 1139944

flvdemux: Attempt upstream seek first

If we have an upstream element that can handle the seek (such as
rtmpsrc), try to do that first before attempting it ourself.

wavparse: do not include codec_data on raw audio caps

If the wav header contains an extended chunk, we want to keep
the codec_data field, but not for raw audio.

This fixes some elements (such as adder) from failing to intersect
raw audio caps which would otherwise be intersectable.

flvdemux: Query duration upstream first

Upstream elements (like rtmpsrc) might be able to provide the duration
more accurately than flvdemux. Especially with index-less vod files