avidemux: remove dead code

sub may not be NULL in this switch, there is a bail out just
before it if so.

Coverity 206098

flacparse: remove dead code

The block_size == 0 was shortcut earlier, and the variable is not
modified in the meantime.

Coverity 206097

videomixer: remove dead code

While it seems to keep a compile time selection, I traced it
to some code copied from videoconvert, where it was removed,
with the following comment:

    Also remove the high-quality I420 to BGRA fast-path as it needs
    the same fix, which causes an additional instruction, which causes
    orc to emit more than 96 variables, which then just crashes.
    This can only be fixed in orc by breaking ABI and allowing more
    variables.

Thus, I remove it here as well.

Coverity 206064

isomp4: fix incorrect masking for multiple tags

Coverity 206058

isomp4: fix wrong atom flags set when adding samples

Coverity 206057

audiofx: fix comparison of delta time to a threshold

Coverity 206055

wavparse: do not rely on call failure keeping return data unmodified

This is clearer this way too.

Coverity 206029

isomp4: catch fseek error

Coverity 206028

isomp4: report failures to caller

Coverity 206027

rtpjitterbuffer: refuse serialied query when buffering

When we are buffering, we can't block and wait for the serialized query
to complete because the jitterbuffer will not try to forward the query
while buffering. Instead, just refuse the query.

rtpjitterbuffer: don't free the serialized query

We should never free a serialized query in the queue, it is the upstream
caller that will free it.

aacparse: Fix memory leak in the test

videomixer: Create hashtable only when we actually use it

In error cases we previously returned without freeing it.

videomixer: Chain up to the parent class' dispose function

v4l2videodec: Initialise ioctl struct with zeroes before passing it to ioctl()

udpsrc: correct LOG msg for -1

Signed-off-by: Marc Leeman <marc.leeman@gmail.com>

interleave: Fix negotiation to work at all again

The caps query handling function for the sinkpads was called for
the srcpad, and the sinkpads had none. This commit moves it to the
right pad, but nonetheless the negotiation still looks wrong.

This makes the test pass again after the recent coverity fix
and also allows interleave to work again, but someone should
really review the negotiation code and fix it.

oss4: Maximum number of channels support is 8

Avoids doing potential overwrites in ch_layout (which only has 8
fields).

CID #1139826

osxvideosink: Set rank to MARGINAL

If available we prefer using glimagesink over osxvideosink. It supports
more formats and in general has more features than osxvideosink.

rtph264depay: only guess AU boundaries when aren't indicated by marker

The marker bit isn't mandatory and we had in place code to guess AU
boundaries by detecting a new picture start. This guessing code
didn't work with interlaced content that has proper marker bits
to indicate the AU boundaries. It was leaking the first field buffer
and producing a corrupted output.

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

pngdec: enable libpng interlaced picture handling

Makes libpng deinterlace Adam7 interlaced pictures
by default. It is the only interlaced format available
and if the picture isn't interlaced the code should behave
as before.

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

souphttpsrc: Only keep-alive the connection in stop() if we have finished all previous messages

After cancelling a request we need to create a new connection.

dvdec: Don't set bogus timestamp/duration

This will happen if we have an incoming stream with a non-TIME segment

Could be improved later to figure out proper pts/duration.

CID #1199702
CID #1199703

dvdec: Properly refuse incoming stream without framerate

The return value wasn't properly propagated back if the caps
didn't contain a framerate

souphttpsrc: Also retry on unexpected network failures

souphttpsrc: New property to specify the maximum number of retries before we give up

souphttpsrc: Change default timeout to 15 seconds

If nothing happens after 15 seconds, chances are good that
our connection will never will work. Stop after 15 seconds
instead of waiting until the system's default timeout, which
can be > 1 minute.

qtdemux: replace duplicated variable when parsing trex atom

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

souphttpsrc: Use GST_FLOW_FLUSHING when flushing, not GST_FLOW_EOS

... and reset it properly after flushing is done. Fixes playback
in many cases when buffering is used.

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

qtdemux: Properly return stream flags when parsing trex atom

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

osxvideosink: use the video frame API instead of the video meta API

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

osxvideosink: advertize video meta API support

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

interleave: Add missing break in switch statement

The caps query is handled entirely already before.

CID #1139757

tests: souphttpsrc: use SoupKnownStatusCode if needed

From libsoup docs:

Prior to 2.44 SoupStatus was called SoupKnownStatusCode,
but the individual values have always had the names they
have now.

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

avidemux: use frames, not bytes, for position query in VBR streams

Coverity 1139648

smpte: fix copy/paste error causing unmap on wrong buffer

Coverity 1139647

deinterlace: guard against finding no suitable pattern

The code handles a -1 pattern index, and it seems plausible
that a pattern might be found later, so it seems best to not
send an element error here.

Coverity 1139766

rtspsrc: update for new MIKEY API

rtspsrc: send sender SSRC in the MIKEY message

Allocate a new SSRC for our RTCP messages back to the server and set
this in the MIKEY message.

rtspsrc: make random number for the CSB

As recommended in the RFC

rtspsrc: don't put spaces in keymgmt header

rtspsrc: create and send the RTCP encryption key

Create and make a key for encrypting the RTCP packets back to the server
and wrap this in a MIKEY message that we send as a header in the SETUP
request.

rtspsrc: free the srtpdec element

rtspsrc: cleanup stream_free function

There is no reason to NULL all fields, we will free the stream anyway.

jitterbuffer: demote warning to debug

For TCP, it is normal that we don't have timestamps so don't WARN on
it.

v4l2: Fix support for caps without width, height, framerate or format

For format like mpegts, width and height is rarely in the negotiated caps. This
patch fixes failure when setting format, and prevent introducing width, height,
framerate and format to the caps when fixating.

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

avidemux: Always set PTS=DTS on raw video streams

avidemux: Always set pixel-aspect-ratio on raw video streams

That field is mandatory in caps and if it is not present in the
AVI container, it means square pixels thus 1/1.

matroska-mux: add mapping for Opus audio

Might want to consider adding channels/rate
requirement to template caps, but requires
fixing up of encoder and parser first.

matroska-demux: add mapping for Opus audio codec

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

v4l2src: Fix support for mpegts streams

It seems that GStreamer's mpegts elements (tsdemux, tsparse) require caps
`video/mpegts,systemstream=true`.  As far as I can see the significance
of systemstream is to indicate that this is a container format rather than
an elementary stream.  As this is the case (and I can't understand how it
could not be the case with mpegts) I add systemstream=true to v4l2src's
caps.

This allows v4l2src to be linked with tsdemux for playback from my
Hauppauge HD-PVR with the pipeline:

    v4l2src ! queue ! tsdemux ! video/x-h264 ! decodebin ! xvimagesink

In combination with the next commit this fixes using Hauppauge HD-PVR with
GStreamer 1.0+.

v4l2: attempt to fix infinite (for small version of infinite) loop

rtpmanager: copy sticky events when exposing pads in more places

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

v4l2: fix distcheck

Make sure ext/*.h are dist'ed

ximagesrc: only extrapolate alpha mask for 32-bit depth

Instead of passing bogus alpha mask values when there's no alpha.

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

ximagesrc: Add ARGB/BGRA support

jpegpay: consider header len when calculating payload len

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

jpegdec: All frames are sync points

pngdec: All frames are sync points

matroskademux: segment closing not needed in 1.x

... as sender should keep track of segment base accumulation.
Rather, it may have some adverse effects as a spurious segment event,
e.g. in collectpads.

matroskademux: early sending pending codec-data for all streams

... at least before syncing across all streams might cause some gap
activity on any of those streams, notably sparse streams.

See also #712134

matroskamux: handle both sticky and non-sticky custom event

rtspsrc: only expose streams on dataflow

Only probe on buffers, we don't want to expose the streams on events.

rtspsrc: copy sticky events to ghostpad

When we expose internal pads as ghostpads, first copy the sticky events
so that we have the caps and segment etc.

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

rtspsrc: srtp handling

rtspsrc: set SSRC on caps if known

rtspsrc: put caps on udpsrc instead of using the signals

Try to avoid using the request-pt-map to get caps but set them directly
on the udpsrc element. That way, the caps get nicely transformed as they
pass through the different elements in the rtpbin, including the AUX and
decoder/encoder elements.

rtspsrc: use profile to set rtcp caps

Use the negotiated profile to set x-rtcp or x-srtcp caps

rtspsrc: set udpsrc to READY

READY is enough to allocate ports now

udpsrc: improve caps handling

Protect caps with the lock.
Don't push the caps event from the set_property function but mark the
pad for reconfiguration so that it will renegotiate and push the new
caps event in the streaming thread.

udpsrc: open/close socket in NULL<->READY state

We should open the socket when going to NULL<->READY and not in the
start/stop vemthod, which is called in READY<->PAUSED. This makes it
possible to allocate a socket without going to PAUSED (and starting the
negotiation).

rtspsrc: free caps in ptmap array

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

rtspsrc: handle NULL rtpmap and parse error better

configure: Don't check for gudev if video4linux2 is not present

configure: Don't fail if gudev is not present

PKG_CHECK_MODULES has the bad habit of failing the build if it doesn't
get what it wants, prevent that.

v4l2: Implement GstDeviceMonitor subclass

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

pulse: Add device monitors

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

v4l2: Remove GstPropertyProbe leftovers

videomixer: Port to new collectpads API

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

v4l2: Add types compatiblity for other OS

Adds type compatiblity with other OS like BSD. This uses types mapping macro to
avoid conflict with existing defined types. We resuse glib types as these are
already available on supported platforms. This is GCC only because of the
le32 type that uses bitwise attribute.

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

pulse: fix format info to caps conversion for mulaw

pulse: Make gst_pulse_format_info_to_caps() shared

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

v4l2: Fix typo V4L_DIR intead of V4L2_DIR

v4l2-build: Set HAVE_GST_V4L2 if headers are present

The name of HAVE_ need to match the USE_. Now set HAVE_GST_V4L2 if
videodev2.h is found.

v4l2: Actually build the plugin

The checks were removed inadvertedly in previous patch and not replaced.
Re-introduce the configure checks and some of the checks in order to enable
this plugin again. We only check if videodev2.h exist on the platform to
avoid building on Windows or OSX, though we build against our own copy. This
was breaking the build on built-bot.

translation: PO file changes caused by POTFILE.in update

v4l2: Remove XV support

XV support for v4l2 never became upstream and ended up being
commented out with an undef for a long time now.

v4l2: Use a copy of videodev2.h header

With years the amount of ifdef have grown up and we are not even sure if the
old code path compiles. Each time we need to update the v4l2 framework to add
the new feature, we break compilation on older kernel. With exception of two
controls in the video orientation control, this patch get rid of all ifdef by
including the latest version of videodev2.h inside GStreamer.

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

souphttpsrc: Add properties for selecting SSL/TLS certificate checking

And by default properly check certificates against the system's CA
certificates. Everything else is not a good default at all.

matroskademux: fix assert on fps lower than 1

Fixes assert caused by gst_duration_to_fraction calling
gst_util_uint64_scale_int with a denominator of 0 when fps is less
than 1.

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

videomixer2: store video info with buffers to keep it in sync

Instead the queued buffer might have an old caps while the pad
is already storing the information for a new caps. Mixing those
while handling buffers will often lead to issues

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

v4l2: Fix typo contol -> control

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

v4l2: Normalise control names in the same way as v4l2-ctl

V4L2 kernel drivers allow configuration of the hardware settings via a
mechanism called controls.  These can be referred to by name such as
"Brightness" and "White Balance Temperature".  The user-space command line
client for setting these controls (v4l2-ctl) normalises these names such
that they only contain lower case alphanumeric characters and the
underscore '_'.  e.g:

    Kernel                     v4l2-ctl
    ----------------------------------------------------
    Brightness                 brightness
    White Balance Temperature  white_balance_temperature
    Focus (absolute)           focus_absolute

GStreamer seems to want to follow this pattern but failed for controls with
more than one consecutive non-alphanum character.  e.g. GStreamer would
produce "focus__absolute_" rather than "focus_absolute".

This commit fixes that issue.  Backwards compatibility is preserved by
normalising all control names before comparison.

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

souphttpsrc: Make sure to not return EOS immediately if we finished a range request

Only return EOS the next time create() is called, if at all. basesrc
should already take care of not calling it again.

Also always return immediately if the previous flow return was
not OK. This indicates an error somewhere.

rtp: Remove caps restrictions from RTP depayloader sink caps

Remove caps restrictions that correspond to the default and are not
required in SDP. With the new usage of having pads require a subset
of the caps, they will make the negotiation fail.

rtpspeexdepay: Remove caps restrictions for depayloader

The "encoding-params" is optional in the SDP, because we now require
a subset of the caps, it would fail caps negotiatioin if it wasn't present.
So removed it from the template caps.

souphttpsrc: Don't forget to quit mainloop after we cancelled when we got data after the stop position

souphttpsrc: If we had a stop position, allow for the server to finish our connection instead of just cancelling

Otherwise keep-alive does not make much sense and also the server will have
confusing things in the logs.

rtspsrc: skip streams with same control url

Keep track of what streams we did the SETUP for. We only need to
configure caps, wait for pads and push events on setup streams. We can
remove the disabled state of the stream and simplify some checks.
After we setup a stream, skip the other streams that have the same
control url. Use a skipped flag to mark streams that should be skipped.

rtspsrc: remove obsolete code