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

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