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

v4l2bufferpool: Cleanup poll method and retry on EINTR/EAGAIN

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

flvdemux: set RESYNC buffer flag when bridging large PTS gaps

So downstream gets notified when this happens.

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

rtprtx: Reset state on each iteration

Otherwise it didn't wait for the test to finish before checking the results.

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

matroskademux: don't leak doctype string in error code path

CID 1212145.

vp9enc: Don't dereference NULL checks

CID #1197703

vp8enc: Don't dereference NULL variable

CID #1139838

qtdemux: upstream handles seek if fragmented and on time segment

Otherwise we can reject seeks on local files that contain fragmented-like
atoms like 'mvex'. Also improve a message log

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

h264depay: make sure we call handle_nal for each NAL

Call handle_nal for each NAL in the STAP-A RTP packet. This makes
sure we correctly extract the SPS and PPS.

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

souphttpsrc: Add custom sticky event to contain the HTTP request and response headers

This can be useful to e.g. get cookie information downstream.

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

avidemux: remove stream last flow return

GstPad already stores that information

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

qtdemux: remove last flow return from stream struct

It is already stored on GstPad on core

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

flvdemux: Use GstFlowCombiner

Use the flow combiner to have the standard combination results and avoid
repeating the same code

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

matroskademux: use GstFlowCombiner

Use the flow combiner to have the standard combination results and avoid
repeating the same code

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

avidemux: use GstFlowCombiner

Removes flow return combination code to use the newly added GstFlowCombiner

qtdemux: use GstFlowCombiner

Removes the common code to combining flow returns to let it be
handled by core gstutils' GstFlowCombiner

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

v4l2sink: implement gstvideosink.show_frame instead of gstbasesink.render

It allows to show preroll frame. Especially it allows to update the
frame when seeking in PAUSED state.

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

v4l2sink: Cleanup old pad alloc declaration

v4l2bufferpool: Copy already queued buffer

This is required as during preroll we pass the first buffer twice, hence already
queued. It is also useful, to allow filters replaying a previous rendered buffers.
This will require 1 more buffer in sink if last-sample is enabled, since the last
sample will not be the same as the currently queued buffer.

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

v4l2bufferpool: Port to bufferpool flush_start/stop method

Port the buffer pool to use the new flush_start/flush_stop virtual
methods added to GstBufferPool.

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

po: update

po: update POTFILES

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

v4l2bufferpool: Don't queue all the buffers before dequeueing first

For output device, we where queuing all the buffers, and then we would
dequeue one. This means we only have 1 buffer for the pipeline, no matter
the size of the queue. Instead, start dequeued when min_latency is reached.
Eventually, this the min_latency should also be affected by control
MIN_BUFFERS_FOR_OUTPUT (use by encoders).

v4l2object: Simply read back the config to update the query

It's easy to get the min/max outdate when hacking decide allocation. In
order to avoid this, simply read back the choosen value from the config.

v4l2: Cleanup and fix calculation of latency

Calculation of num_buffers (the max latency in buffers) was
up-side-down.  If we can allcoate, then our maximum latency match
pool maximum number of buffers. Also renamed it to max latency. Finally
introduced a min_latency for clarity.

Revert "v4l2bufferpool: Port to bufferpool flush_start/stop method"

This reverts commit 2e0fb42e868fc9f6d98b028def80a3e953527307.

Conflicts:
sys/v4l2/gstv4l2allocator.c
sys/v4l2/gstv4l2bufferpool.c
sys/v4l2/gstv4l2videodec.c

v4l2object: Fix configuration of other_pool and importation case

Fix the choice of min/max, don't override the min/max with own pool selected
size, correct other_pool is_active check, start from other_pool config when
configuring the other pool and finally validate the configuration.

v4l2object: Use proposed allocator as default

v4l2bufferpool: Fix USERPTR map flags

We need to map READ only for output and write only for capture, we where
doing the opposite. This fixing USERPTR with glimagesink

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

qtdemux: parse tkhd transformation matrix and add tags if appropriate

Handle the transformation matrix cases where there are only simple rotations
(90, 180 or 270 degrees) and use a tag for those cases. This is a common scenario
when recording with mobile devices

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

v4l2bufferpool: Prevent num_queued from going negative

v4l2videodec: don't stop if loop returned FLUSHING

The decodeing thread returning flushing isn't an error, we should simply
try starting the task again. If it's actually flushing, it will stop again by itself.

v4l2videodec: Handle early task stop

v4l2videodec: Handle gst_pad_start_task() failure

v4l2videodec: Add trace for FLUSH_START/STOP handling

v4l2videodec: Fix use of atomic value

v4l2bufferpool: Improve debugging

No need to use obj->element, the pool now have a significant name. Also don't
warn if flushing.

v4l2videodec: Fix handle_frame error handling

v4l2bufferpool: Add a trace when _start() is called

v4l2allocator: Add debug assert to detect calls in the wrong state

v4l2allocator: Reset count when stopped

v4l2allocator: Return a GstFlowReturn instead of boolean in alloc

v4l2object: Don't leak config structure

v4l2bufferpool: Port to bufferpool flush_start/stop method

qtdemux: add tag mappings for _swr, _mak and _mod tags

swr -> Application name
mak -> device manufacturer
mod -> device model

ximagesrc: Fix ximage leaks when buffer has more then one ximage

From time to time, when the image_pool list has more then 1 element
and I suppose at start, all but 1 pooled ximage are leaked. This is
due to broken algorithm in gst_ximagesink_src_ximage_get(). There was
also a risk of use after free for the case where the ximage size has
changed.

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

Back to development

Release 1.3.2

Update .po files

Automatic update of common submodule

From 211fa5f to 1f5d3c3

vp8enc: Don't dereference NULL variable

CID #1139838

vp9enc: Don't dereference NULL checks

CID #1197703

v4l2bufferpool: Explicitly cast enum "subtype" to its "supertype"

gstv4l2bufferpool.c:608:18: error: implicit conversion from enumeration type
      'enum _GstV4l2BufferPoolAcquireFlags' to different enumeration type
      'GstBufferPoolAcquireFlags' [-Werror,-Wenum-conversion]
  params.flags = GST_V4L2_POOL_ACQUIRE_FLAG_RESURECT;
               ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

goom: Use fabs() instead of abs() to calculate the floating point absolute value

tentacle3d.c:268:7: error: using integer absolute value function 'abs' when
      argument is of floating point type [-Werror,-Wabsolute-value]
  if (abs (tmp - fx_data->rot) > abs (tmp - (fx_data->rot + 2.0 * G_PI))) {
      ^

debugutils: Properly calculate the difference with unsigned types

tests.c:161:16: error: taking the absolute value of unsigned type
      'unsigned long' has no effect [-Werror,-Wabsolute-value]
    t->diff += labs (GST_BUFFER_TIMESTAMP (buffer) - t->expected);

v4l2videodec: Handle flush while in start_streaming

We need to handle the case where a flush occure while the streaming
thread is being brought up. In this case, the flushing state of the poll
object is cleared. To solve this, we simply set the capture poll to flushing
again, this way we know the thread will exit. The decoder streamlock
is used to synchronize with handle frame.

v4l2allocator: Don't trace twice the same message

rtspsrc: always use a random ssrc for the internal session

Use a random SSRC different than 0 for the internal session SSRC.

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

rtpsession: update last_activity when sending RTP

Also update last_activity when doing something with the internal
source to make sure don't timeout early.

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