sunaudiosrc: remove unnecessary NULL checks before g_free()

wavparse: remove unnecessary NULL checks before g_free()

matroskamux: remove unnecessary NULL checks before g_free()

matroska/read-common: remove unnecessary NULL checks before g_free()

isomp4/atoms: remove unnecessary NULL checks before g_free()

rtp/theorapay: remove unnecessary NULL checks before g_free()

rtp/vorbispay: remove unnecessary NULL checks before g_free()

rtp/jpegpay: remove unnecessary NULL checks before g_free()

rtpgstpay: remove unnecessary NULL checks before g_free()

rtspsrc: remove unnecessary NULL checks before g_free()

flxdec: remove unnecessary NULL check before g_free()

effectv/optv: remove unnecessary NULL checks before g_free()

effectv/shagadelictv: remove unnecessary NULL checks before g_free()

effectv/ripple: remove unnecessary NULL checks before g_free()

effectv/radioac: remove unnecessary NULL checks before g_free()

effectv/streak: remove unnecessary NULL check before g_free()

shout2: remove unnecessary NULL checks before g_free()

vp9enc: remove unnecessary NULL check before g_free()

vp8enc: remove unnecessary NULL check before g_free()

souphttpsrc: remove unnecessary NULL checks before g_free()

v4l2object: add support of NV16, NV61 and NV24 formats

Mapped respectively to V4L2_PIX_FMT_NV16/V4L2_PIX_FMT_NV16M,
V4L2_PIX_FMT_NV61,V4L2_PIX_FMT_NV61M and V4L2_PIX_FMT_NV24 v4l2 formats.

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

splitmuxpartreader: Fix GCond leak

inactive_cond is not being cleared resulting in memory leak.

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

jpegdec: fix output state memory leak

When jpeg_finish_decompress is called, output state reference is being created.
But if there is any failures in finishing decompress, it jumps to setjmp,
and at that point state was not referenced. Resulting in leak of output state.
Hence adding another setjmp after output state is referenced.
Similarly adding another setjmp to unmap the frame in case error happens before
finish_decompress

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

qtdemux: respect qt segments in push-mode for empty starts

In push-mode it is hard to support qt segments overall but it is
possible to support when the file isn't heavily edited but just contain
a segment to indicate a gap at the beginning. This also allows properly
timestamping data that has negative DTS in push-mode.

It is relevant to support those for 2 scenarios:

1) fragmented streaming
2) HTTP playback of 'regular' mp4

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

pulse: Don't leak caps and structures in the device provider

rtpmanager: Document properties that are expressed in bits per second

This changed in 928cd110bcea5d143cab3ea747991851d52ecbad and
73c0c2920f9aca96982a4de0c20b3417aa148b81 but was not documented.

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

rtpmanager: Trivial gst-indent fixes

qtdemux: support for cenc auxiliary info parsing outside of moof box

When the cenc aux info index is out of moof boundaries, keep track of
it and parse the beginning of the mdat box, before the first sample.

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

matroskademux: Use codecutils helpers for creating Opus caps

Also fix up codec data with values from the container.

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

matroskademux: There is no multistream field for Opus anymore

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

matroska/webmmux: Support Opus in webmmux and VP9 in matroskamux

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

matroskademux: Parse and handle CodecDelay, SeekPreroll and DiscardPadding

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

matroskamux: Write CodecDelay, DiscardPadding and SeekPreroll for Opus

And also adjust timestamps and durations according to the codec delay, both
should include it for whatever reason.

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

matroskamux: Opus headers are not in-band

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

v4l2: Set O_CLOEXEC on the device fd

This is needed to make sure that child processes don't inherit the video
device fd which can cause problems with some drivers.

rtpmanager: switch G_GINT64_FORMAT for GST_STIME_ARGS

No need to use G_GINT64_FORMAT for potentially negative values of
GstClockTimeDiff. Since 1.6 these can be handled with GST_STIME_ARGS.
Plus it creates more readable values in the logs.

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

rtpmanager: use GST_STIME_ARGS for GstClockTimeDiff

No need to manually handle negative values of diff, GST_STIME_ARGS does
exactly this.

videomixer: use GST_STIME_ARGS for GstClockTimeDiff

No need to manually handle negative values of diff, GST_STIME_ARGS does
exactly this.

deinterlace: use GST_STIME_ARGS for GstClockTimeDiff

No need to manually handle negative values of diff, GST_STIME_ARGS is
available for this.

audiochebband: Fix typo in example pipeline

Fix typo in example pipeline.

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

v4l2: fix double-unref in the v4l2 device provider

matroskamux: don't drop JPEG frames that only have PTS but no DTS set

For the MS/VfW codec ids, we want to write DTS timestamps instead
of PTS because that's what everyone else seems to do (and it's also
how it is in AVI). So for those input formats we use the buffer DTS
instead of the PTS. However, if there's no DTS set but only the PTS
then just take the PTS instead of dropping the input buffer. This
is useful especially for I-frame only codecs like JPEG and huffyuv,
but should also be fine as fallback in general.

Fixes regression with input JPEG frames that only have PTS set on them.

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

tests/check/splitmux: test that the release_pad vfunc of splitmuxsink actually releases pads

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

splitmuxsink: do not destroy the multiqueue & muxer when going to NULL

Instead, delay it until all request pads have been released. This is
because the release_pad() vfunc requires the multiqueue and muxer to
be there in order to release their request pads as well. If those
elements are destroyed earlier, release_pad() does not work, no
pads are released and some resources are leaked.

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

matroskademux: Read buffer timestamp *after* actually setting it

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

scaletempo: Fix handling of rate < 0

We have to reverse all samples in a buffer before processing them to properly
have continuous data from one buffer to another. As a result we will have a
negative applied rate and a rate of 1.0.

Also make sure that input buffers are correctly clipped to the segment,
otherwise our calculations are going to go wrong.

Also copy over the segment event's sequence number to the output segment while
we're at it.

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

deinterlace: break as soon as non-interlaced if found

It looks for a non-interlaced entry on the filter caps, break
as soon as one is found to avoid wasting cpu

deinterlace: implement accept-caps

Implement accept-caps handler to avoid doing a full caps query
downstream to handle it.

This commit implements accept-caps as a simplification of the _getcaps
function, so it exposes the same limitations that getcaps would.
For example, not accepting renegotiation to caps with capsfeatures when
it was last configured to a caps that it has to deinterlace.

tests: deinterlace: fix small typo in comment

check: Dist splitvideo0[012].ogg test files.

scaletempo: Add support for F64

docs: Minor fixes in various places

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

goom: remove compiler trick

After commit 2cb6cfed22166b262ae50cb58f3ff11dd8ba91f9 there is no need to
trick the compiler anymore about the usage of variable cpuFlavour.

Automatic update of common submodule

From b99800a to b319909

audiofx: remove unused variable

Remove unsued variable have_coeffs in audiofxbaseiirfilter

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

Use new GST_ENABLE_EXTRA_CHECKS #define

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

Automatic update of common submodule

From 9aed1d7 to b99800a

flvdemux: relax creation time parsing

Parse wrong timestamps like we used to write as well,
e.g. 10:9:42, and the hour might be without a leading
zero in any case.

flvdemux: fix indentation

flvdemux: extract both creation date and time

Before we only extracted the date part.

flvmux: fix writing of creation time

Don't write time as e.g. 11:9:42

rtpj2kpay: update fragment offset

It was always being set to 0, making the resulting stream broken
for the receiver

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

qtmux: Don't unconditionally use strnlen()

It's not available on older OSX and we can as well use memchr() here.

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

auparse: Fix event memory leak

Free the event after being handled to prevent memory leak.

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

qtmux: unify raw audio caps into a single caps structure

qtdemux: add support for FFV1 coded streams in mov

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

souphttpsrc: EOS immediately if we have an empty seek segment

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

souphttpsrc: Make non-inclusive segment boundaries inclusive

The problem is that the filesrc and souphttpsrc are behaving
differently regarding the calculation of the segment boundaries. The
filesrc is using a non-inclusive boundaries, while the souphttpsrc
uses inclusive. Currently the hlsdemux calculates the boundaries as
inclusive, so for this reason there is no problem with the souphttpsrc,
but there is an issue in the filesrc.

The GstSegment is non-inclusive, so the proposed solution is to use
non-inclusive boundaries in the hlsdemux in order to be consistent.
Make the change in the hlsdemux, will break the souphttpsrc, which
will expect inclusive boundaries, but the hlsdemux will offer
non-inclusive. This change makes sure that the non-inclusive
boundaries are converted to inclusive.

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

souphttpclientsink: Add the retry and retry-delay properties

These allow a failed request to be retried after the given number of seconds
instead of failing the pipeline. Take account of the Retry-After header if
present. Add retries parameter that controls the number of times an HTTP
request will be retried before failing.

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

qtdemux: fix caps leak

If the QtDemuxStream are re-used they may already have caps which used
to be leaked.

Reproduced using the
validate.dash.playback.seek_forward.dash_exMPD_BIP_TC1 validate
scenario.

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

qtdemux: Fix taglist memory leak

Free the stream and its sub items instead of just the stream

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

qtmux: Allow negotiating to S8 as a raw format but stop making it best choice

Negotiation to audio/x-raw,format=S8 was not possible because S8 does
not have a bit order so we ended up doing `if (!entry.fourcc) goto refuse_caps;`

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

qtmux: Add prores support

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

tests: add GST_PLUGINS_BASE_LIBS for flvdemux check

So it pulls in the right libgsttag-1.0.

goom/goom2k1: remove obsolete left over files

They now use the new GstAudioVisualizer base class
from gst-plugins-base/gst-libs/gst/pbutils

Also fixed undefined reference to gst_audio_visualizer_get_type
Added GST_PLUGINS_BASE_LIBS to Makefile.am and re-order LIBADD.

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

mpegaudioparse: Fix buffer memory leak during failures

mapped buffer is not being unmapped during failures

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

souphttpclientsink: Check if soup message is created

If soup message is not created then the same should not be passed
on, which is resulting in segfault. Hence throwing a warning message
and returning

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

souphttpclientsink: Check if location being set is valid

Adding a check in set_property to find if the location uri is valid
and printing warning if not valid.

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

souphttpclientsink: Fix memory leaks during failures

freeing streamheader_buffers and sent_buffers during failure cases.

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

souphttpclientsink: Replace redundant free_buffer_list function

Removing free_buffer_list and replacing it with already available function
g_list_free_full

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

check: Don't forget base CFLAGS for flvdemux check

elements/flvdemux.c:25:25: fatal error: gst/tag/tag.h: No such file or directory

matroskamux: Create a TIME segment when creating streamable output

Related to https://bugzilla.gnome.org/show_bug.cgi?id=754435 which
does the same for flvmux.

flvdemux: output speex vorbiscomment as a GstTagList

This is what speexdec expects.

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

flvmux: GST_BUFFER_OFFSETs should be GST_BUFFER_OFFSET_NONE

Or else flvdemux don't understand it

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

flvmux: use time segment and copy timestamps when streamable

Add a basic test using speex data to verify timestamping.

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

flvdemux: speex is also always 16KHz

This is just a cosmetic change for the logs, since the right caps
for Speex is being set elsewhere.

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

rtpmanager: Add 'source-stats' to stats and notify

Add statitics from each rtp source to the rtp session property.
'source-stats' is a GValueArray where each element is a GstStructure of
stats for one rtp source.

The availability of new stats is signaled via g_object_notify.

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

rtpsession: Implement sending of reduced size RTCP packets

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

audiofx: Remove unused variable

Remove unused variable 'degree' in audiodynamic

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

qtdemux: Fix memory leak for corrupted file

Free brands before overriding them.

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

gdkpixbufdec: Fix pixbuf_loader leak during failures

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

rtpbin: Add missing break

rtpmanager: Take into account packet rate for max-dropout and max-misorder calculations

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

rtpmanager: add "max-dropout-time" and "max-misorder-time" props

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

qtmux: Fix date memory leak

When getting date from taglist, the memory should be freed after
using it.

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

qtmux: Fix sample memory leak

When getting sample from taglist, the memory should be freed after
using it.

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

cutter: Fix buffer leak

Buffer is added to the internal cache, and pushed only when accumulated
buffer duration crosses 200 ms. So when the chain ends, the buffer accumulated
is not freed. Freeing the cache when the state changes from PAUSED to READY.

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

rtpmux: Use default upstream event handling

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

rtpmux: As 0xFFFFFFFF is a valid ssrc, check if it has been set

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

gstrtpmux: allow the ssrc-property to decide ssrc on outgoing buffers

By not doing this, the muxer is not effectively a rtpmuxer, rather a
funnel, since it should be a single stream that exists the muxer.

If not specified, take the first ssrc seen on a sinkpad, allowing upstream
to decide ssrc in "passthrough" with only one sinkpad.

Also, let downstream ssrc overrule internal configured one

We hence has the following order for determining the ssrc used by
rtpmux:

0. Suggestion from GstRTPCollision event
1. Downstream caps
2. ssrc-Property
3. (First) upstream caps containing ssrc
4. Randomly generated

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