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

v4l2: Cleanup M2M properties

M2M devices were sharing the same properties as src and sink. Most of
these made no sense. This patch reduces the number of propeties and
makes io-mode clearer by having capture-io-mode and output-io-mode. This
also accidently fixed a bug in gstv4l2transform io-mode code, where the
capture io-mode could not be set.

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

v4l2bufferpool: Update pool limit with hardware requiremenst

If the driver need more buffers than requested by the config,
update the pool min/max values. The minimum value for the pool
could be provided either by the driver or by the pool. This is
best effort for drivers that don't support
CID V4L2_CID_MIN_BUFFERS_FOR_CAPTURE.

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

v4l2videodec: Handle start_streaming error

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

v4l2videodec: Print the flow return causing the loop to leave

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

v4l2videodec: Don't lock the decoder when stopping task

That src pad task may need to take the lock when being pulled
down. takeing that lock can lead to a deadlock.

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

v4l2transform: Don't leak pool if activation failed

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

v4l2: Split flush in start/stop_streaming

This allow calling start streaming later for capture device. Currently it breaks
in dmabuf-import because downstream is holding a buffer that will only be
released after stream-start.

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

v4l2transform: Flush buffer pools on flush stop

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

v4l2allocator: Fix use of atomic active marker

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

v4l2bufferpool: Don't deactivate otherpool

We should not stop the otherpool unless we also stop our own
pool, otherwise it will never get restarted.

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

v4l2bufferpool: Also update num_buffers for import cases

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

rtpbin: update rtp encoder/decoder docs

Use %u in RTP encoder/decoder pads to match other rtpbin pads.

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

tests/check: rtpsession: test internal sources timing out

rtpsession: remove unused if branch

1) sources that have sent BYE in the past cannot be senders, since
they would have timed out to being receivers in the meantime...
2) sources that have sent BYE are now being removed earlier inside
this function

rtpsession: cleanup sources that have sent BYE

rtpsession: unify nested if clauses

rtpsession: timeout internal sources that are inactive for a long time and send BYE

rtpjitterbuffer: don't stop looping if event found in the queue

If we are inserting a packet into the jitter queue we need to keep
looping through the items until the right position is found. Currently,
the code stops as soon as an event is found in the queue.

Regarding events, we should only move packets before an event if there
is another packet before the event that has a larger seqnum.

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

matroskamux: fix the memory leak of language attribute

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

v4l2object: Fix regression in offset extrapolation

When extrapolating the offset, we need to use the extrapolate
stride rather then the base stride. This should fix support for format
with more then two planes (I420, Y42B, etc).

v4l2bufferpool: Use default VideoInfo for frame operation

When doing frame operation, we need to use the default VideoInfo
and let the frame API read the video meta in order to get the stride
and offset right. Currently we where using the specialized VideoInfo
which reflects what the HW is setup to.

v4l2src: minor GValue handling optimisation in probing code

v4l2src: avoid lists with one single framerate in probed caps

Simplify framerate field if possible, so we don't end up with
e.g. framerate = (fraction) { 30/1 }. Maybe the helper function
should be moved to core, but we can do this later.

qtdemux: Fix leak of palette_data in error cases

CID #1212151

qtmux: Free node_header in error cases

CID #1212134

flvdemux: Don't use WARNING for not-linked flow return

Pollutes debug logs for no reason. It's only an error if all pads
return not-linked

flvdemux: Skip unknown tags in push-mode

We add a new mode (SKIP) in push-mode to skip tags that we don't known about

Partially fixes https://bugzilla.gnome.org/show_bug.cgi?id=670712

flacdec: Add support for variable block size files and remove dead code

This dead code wasn't used since the 1.0 port and would need to
be modified heavily for variable block size support.

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

v4l2transform: Fix NULL check copy paste error

CID 1212129

v4l2transform: Fix potential deadlock due to missing break

CID 1212131

rtpjitterbuffer: increment accepted packets after loss

When we detect a lost packet, expect packets with higher
seqnum on the input.

Also update the unit test.

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

Add new test case.

shapewipe: no need to activate pads

Activation will happen in the state change

v4l2object: Don't leak config structure

this fixes a leak of the config structure and take care of making sure
caps can't reach ref 0 before we are done doing our check.

CID 1212144

v4l2object: Remove uneeded cast for code clarity

v4l2pool: Fix leak of config structure in error case

CIDs 1212167 and  1212167

v4l2object: Fix use of unitilized pool pointer

CID #1212173

qtdemux: don't respond to a position query in BYTE format with a TIME position

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

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

CID 1212145.

v4l2object: Readback pool config if used within the baseclass

v4l2: Replace miss-use of crop meta in favour of proper offset

This moves away from copying information and store everything inside
the GstVideoInfo structure. The alignement exposed by v4l2 api
is now handled using proper offset.

v4l2object: Style fix

v4l2allocator: Reset imported buffer size with expected size

This ensure that the buffer pool won't always discard buffer with these
memory when they are released.

v4l2allocator: Reset flushed group

This ensure that a flushed group memory are the same size as when they
where originally allocated / imported.

v4l2bufferpool: Get number of allocated buffers from allocator

The value of num_allocated buffer would get confused when
buffer are being discarded.

v4l2allocator: Add a method to read number of allocated group

v4l2bufferpool: Improve debugging

v4l2bufferpool: Ensure we don't re-enqueue buffer during flush

v4l2transform: Initilialize debug category

v4l2allocator: Fix libv4l2 support

Need to include config.h, otherwise we endup directly using the
ioct/mmap/munmap calls and need to vall v4l2_munmap.

v4l2allocator: Set the flags on the object

We where not setting the probed flags on the allocator, which mean even if
CREATE_BUFS was supported on some driver, it would endup being ignored.

v4l2bufferpool: Re-enqueue buffer at stream start

v4l2allocator: There is not group on error

v4l2transform: Handle FLUSH_STOP event

v4l2bufferpool: Acquire cannot return a buffer from another pool

Return a buffer from an otherpool has unwanted side effects that lead to leaks and
prevents deactivating the pool. Instead, we change the _process() API so it can
replace the internal buffer with the buffer from the downstream pool. This implied
moving from _fill() to _create() method in the src.

v4l2bufferpool: Remove unreached acquire code

The acquire is done in _prepare now.

v4l2bufferpool: Sanetize buffer refount handling

Buffer refcounting is a bit hard, because of the duality between CAPTURE and
OUTPUT mode. In the long term, we should consider having two seperate pool
instead of this mess. At least state should be better kept this way.

v4l2transform: Add more traces

v4l2-allocator: Add S to REQBUFS/CREATE_BUFS enum

All enum that has REQBUFS and CREATE_BUFS where missing S, which was
confusing since they are supposed to match with associcated ioctl name. This
also fixes the yet unused CAN_REQUEST flag check.

v4l2transform: Enabled QoS

v4l2: Fixup USERPTR/DMABUF capture support

v4l2object: Improve selecton of min/max in decide allocation

v4l2bufferpool: Update config if meta is missing

Rather then hard failure, we should update the config with the meta option we
need and return false.

v4l2: Add DMABUF and USERPTR importation

v4l2allocator: Valid FD are bigger or equal to zero

v4l2object: Don't leak downstream pool in propose_allocation

parse_nth_allocation_pool() give a ref on the pool, we need to unref it
when done.

v4l2: Introduce DMABUF_IMPORT IO mode

v4l2: Add dmabuf export support

This can be enabled sing io-mode=dmabuf. This will enabled mmap base
drivers to export the buffers as dmabuf.

v4l2allocator: Guaranty queued state integrety

Because of the buf in videobuf2, dqbuf may leave the DONE flag being,
which would implied that the buffer is queued. As this has been broken
for 4 years, simply guaranty the state flags integrity when doing
qbuf/dqbuf.

See https://patchwork.linuxtv.org/patch/23641/

v4l2transform: Implement open/close

v4l2transform: Ensure output pool is configured