v4l2: rework sink buffer refcounting

This is a followup patch for #700781, which is not quite correct.
The buffer handling is quite complicated here.
The original code intended to the the following:

- gst_v4l2_buffer_pool_process() calls QBUF and adds the buffer to the
  local list.
- The sink calls gst_buffer_unref() which returns the buffer to the pool
  but not the 'free list'.
- Some time later DQBUF returns the buffer and
  gst_v4l2_buffer_pool_release_buffer() puts in on the 'free list'.

If the buffer must be copied then (parent_class)->acquire_buffer() is
called directly to keep the buffer in the pool.

This has two problems:
1. If gst_v4l2_buffer_pool_release_buffer() is called before the buffer is
   returned to the pool, then the buffer is put on the 'free list' twice.
   This can happen if a reference to the buffer is kept outside the sink,
   of if DQBUF returns the buffer, that was just queued with QBUF.
2. If buffers are copied, then all buffers are in the pool at all times. As
   a result gst_v4l2_buffer_pool_stop() and gst_v4l2_buffer_pool_dqbuf()
   can access pool->buffers at the same time, which can lead to memory
   corruption.

The patch for #700781 fixes those problems, but with the side effect that
there are always buffers outside the pool (because they are queued) and
the pool is never stopped.
This patch fixes this by releasing the reference to the buffer after
handling it (to avoid problem 2.) so it can be returned to the pool.
gst_v4l2_buffer_pool_release_buffer() is only called if the buffer is
already in the pool (to avoid problem 1.).

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

qtdemux: make sure taglist is writable before adding tags

Avoids assertions

qtdemux: effectively skip tracks that weren't listed on the 1st moov

Without this, stream is NULL and the code will try to access it, leading
to segfaults.

qtdemux: skip redundant check

!got_moov is already checked the line above

tests: cleanup level tests

Split out a few more tests to avoid checking the same stuff over and over again.

level: remove unused variables in instance struct

level: add a test for continous timestamps

A test that checks that msg[n].ts + msg[n].dur == msg[n+1].ts.

wavenc: add tags & toc support

Write tags as LIST INFO chunk. Format the toc as cue + LIST adtl chunk. Remove
old #ifdef'ed code.

Revert "rtph264pay: Restructuring to allow for adding optional caps"

This reverts commit 61666898cfe89a1b21d3e6850ab44f5b1633ed79.

This commit changes what the set_sps_pps() function does, not it doesn't
set caps anymore (and should have been renamed). The main problem is that
not all call sites are updated and thus leak the string.

Revert "rtph264pay/depay: Add frame dimensions a payloaded caps"

This reverts commit 3dca756a5dba55266256f239e3e12a3d058e185a.

The H264 RTP spec has no attributes for width and height.

Revert "rtph264pay/depay: Add optional framerate caps for use in SDP"

This reverts commit d8825e2a5c0bfb883ff88e2c9da499c800ebca0a.

There is no framerate attribute in the h264 RTP spec.

Revert "rtpjpegpay/depay: Replace framesize caps with width/height"

This reverts commit 0075d111b475ca27895ee9476154260b6902940b.

Extra application/x-rtp are SDP fields, which are strings.

Revert "rtpjpegpay/depay: Replace framerate caps field with fraction"

This reverts commit 9fd25a810b859e0ec205176578735100d83de4af.

We deal with sdp attributes in application/sdp, which are always strings.

rtspsrc: add extra TLS url protocols

We also support TLS protocols now.

videomixer: Add FIXME comment about the DURATION query from adder

Currently the code just takes with maximum upstream duration, which
is wrong. It should be the maximum upstream duration in running time.

videomixer: Set a reference to mix->current_caps as the QUERY_CAPS result.

level: misc cleanups

Fix some oudated comments. Sort out some confusion of interval_frames and num_frames.

v4l2: Only conditionally use V4L2_CTRL_TYPE_INTEGER_MENU, it's not available in older versions

v4l2: add a property for arbitrary v4l2 controls

This makes it possible to set any controls that can be set with
VIDIOC_S_CTRL.
The controls are set when the property is set (if the device is open)
and when the device is opened.

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

level: fix discontinuities in timestamps

gdkpixbufdec: Keep serialized events in order, and don't send SEGMENT before CAPS

rtspsrc: create and push stream-start in TCP mode

rtspsrc: remove some obsolete code

It is not needed to do a state change from the _play() function on
ourselves. The state change function already did that and we don't want to
interfere with that (or use hacks to avoid interference).

rtspsrc: set RTCP caps on the RTCP pads

rtpsession: send stream-start and segment events

Also send stream-start and segment event on the RTCP pad.
We don't need to send anything on the sync_src pad because we
already forwarded all incomming events.

rtspsrc: add signal to handle server requests

Add a signal to be notified of a server request. The signal handler can then
construct the response message for the server.

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

videomixer: Maintain z-order when new pad are added

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

vp8enc: Add property to manually specify the timebase of the encoder

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

videomixer: Always handle flush_stop_pending atomically

It is not protected with the COLLECT_PADS_STREAM_LOCK anymore

tests: videomixer: Add a testsuite for videomixer

This is mostly copy pasted from -base/tests/check/elements/adder.c

videomixer: Do not take COLLECT_PADS_STREAM_LOCK when unnecessary

Collectpad takes the lock itself when receiving serialized events
and we should not take it for not serialized ones

flxdec: Properly skip non-frame chunks

flxdec: Flush data from adapter after reading it

Otherwise we're going in an infinite loop, reading the same data
over and over again.

goom2k1: fix more duplicated symbols

rtpjpegpay/depay: Replace framerate caps field with fraction

The previous implementation had the formatting of SDP attributes happen
in each RTP payloader, now instead the constituent values are propagated
as caps fields. This allows for applications to do SDP offer/answer
based on caps negotiation.

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

rtpjpegpay/depay: Replace framesize caps with width/height

The previous implementation had the formatting of SDP attributes happen
in each RTP payloader, now instead the constituent values are propagated
as caps fields. This allows for applications to do SDP offer/answer
based on caps negotiation.

Keep parsing a-framerate, x-framerate and x-dimensions in rtpjpegdepay
to be backwards compatible with previous payloaders.

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

rtph264pay/depay: Add optional framerate caps for use in SDP

This allows for applications to format SDP attributes and still do SDP
offer/answer based on caps negotiation.

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

rtph264pay/depay: Add frame dimensions a payloaded caps

This allows for applications to format SDP attributes and still do SDP
offer/answer based on caps negotiation.

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

rtph264pay: Restructuring to allow for adding optional caps

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

(dyn|multi)udpsink: Add properties to specify the bind address and port

By default we use the any addresses and a random port for binding the socket.

(dyn|multi)udpsink: Bind socket before using it

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

(multi)udpsink: Add missing getters for socket-v6 and used-socket-v6 properties

videomixer: Don't hold stream-lock while pushing non-serialized events

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

videomixer: Don't hold object lock while sending events

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

deinterlace: The return value of gst_pad_set_caps() is not relevant anymore

Caps can fail to be set because the pad is not linked yet for example.

qtdemux: Add error if file has playready drm

videomixer: Send a reconfigure event upstream if sinkpad caps are not usable

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

v4l2: keep a reference to all queued buffers

Without this, a queued buffer may be required, filled and queued before it
is dequeued.
Calling gst_buffer_pool_acquire_buffer() ensures that the buffer is set up
correctly and gst_buffer_unref() calls buffer_release().

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

mulawdec: Handle NULL buffers in handle_frame

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

rtpjpegpay/depay: Add framesize caps for use in SDP

The format of the value adheres to RFC6064 and it is meant to be parsed
and included in the SDP sent by gst-rtsp-server to its clients.

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

rtpjpegpay: Add optional framerate caps for use in SDP

The format of the value adheres to RFC4566 and it is meant to be parsed
and included in the SDP sent by gst-rtsp-server to its clients.

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

videomixer: When all sinkpads are eos, update output segment stop and forward it

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

videomixer: Don't reset the output segment on flush stop

Only init it when getting from READY to PAUSED, and change it on seek events.

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

v4l2: Don't stop streaming when set_caps is called with unchanged caps

This can happen if other parts of the pipeline are reconfigured.
Stop streaming even for a short amount of time can be quite visible, so it
should be avoided if possible.

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

tests: Re-enable videomixer test

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

videomixer: Send caps event from the streaming thread

This way we avoid races in caps negotiation and we make sure
that the caps are sent after stream-start.

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

videomixer: Do not send flush_stop when receiving a seek

There is no reason to send a flush-stop when receiving a seek event.
In the case of a flushing seek, we could eventually want to, but in
the code path were we check if the seek is "flushing", we have the
following comment that makes sense:

"we can't send FLUSH_STOP here since upstream could start pushing data
after we unlock mix->collect.
We set flush_stop_pending to TRUE instead and send FLUSH_STOP after
forwarding the seek upstream or from gst_videomixer_collected,
whichever happens first."

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

videomixer2: Protect flush_stop_pending with the collectpad stream lock

And make sure to expect a flush-stop after a flush-start

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

rtpmp4apay: clear config buffer before using it

This is necessary because parts of the memory are only modified with "|="

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

qtdemux: Do not expect EOS after a segment event if upstream is mss

In case qtdemux is handling a mss stream, do not mark the stream to wait
for EOS after a segment. Even if it seems to be the last one according to
the current streams information.

MSS handling is different here because there is another demuxer driving
the pipeline

qtdemux: only set channels and rate if qtdemux knows it

Setting both of those to 0 is pointless and means that qtdemux
doesn't know the real value. Avoid setting it in this case.

qtdemux: set alac caps using info from codec buffer

The samplerate field in the STSD atom is not right for some ALAC files
(usually when audio is 96kHz/24bits), so the audio caps must be
extracted from the codec data.

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

avidemux: do not push discont buffers if they aren't discont

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

Automatic update of common submodule

From 5edcd85 to 098c0d7

videocrop: Add support for GRAY16_LE/GRAY16_BE

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

rgvolume: Send all events through the proxypads instead of just sending to the target

Otherwise the sticky events are missing on the proxypads.

rgvolume: Fix event handling in the unit test

rglimiter: Fix event handling in unit tests

rganalysis: Fix event handling in unit test

qtmux: Fix event handling in unit test

multifile: Fix event handling in unit test

mulaw: Fix event handling in unit test

matroskaparse: Make sure to send a segment event before dataflow

v4l2: only add interlace-mode to the caps for raw formats

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

v4l2: copy and set the actual size of the content

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

interleave: Fix event handling in unit test

deinterlace: Improve handling of min/max buffer numbers of the buffer pool

deinterlace: set caps for buffer pool config

multifilesink: Let the base class do get_times

This will make sync=TRUE work, the default is still sync=FALSE

interleave: Send stream-start before caps event

rtpmux: Send stream-start before caps

rtpjitterbuffer-test: Send stream-start before caps followed by segment

rtpbin-test: Send missing stream-start and segment events

tests: Fix some more event handling in tests

icydemux: Fix event handling in unit test

icydemux: Fix sticky event handling

flvmux: Push sticky events in the right order

deinterleave: Fix event handling in test

deinterleave: Fix sticky event handling

deinterleave: Code style fixes

rtpgstpay: First let baseclass handle events, then put them into the stream

Fixes handling of sticky events.

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

shapewipe-test: Send inital events

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

spectrum-test: Send inital events

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

videofilter-test: Send inital events

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

wavpackparse-test: Send inital events

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

y4menc-test: Send inital events

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

multipartdemux: fix example pipeline

Need jpegparse.

tests: Fix some more unit tests

tests: Fix parser tests

shapewipe: Can't map twice the same buffer for writing

I took the opportunity to simplify that code a bit. We now use
gst_buffer_make_writable() to make the buffer writable and map twice the
same buffer, with first map being read/write, and second read only. This
get rid of the critical:

GStreamer-CRITICAL **: gst_structure_set_name: assertion `IS_MUTABLE

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