deinterlace: Fix compiler warning

gstdeinterlace.c: In function 'gst_deinterlace_output_frame':
gstdeinterlace.c:1537:57: error: 'pattern.length' may be used uninitialized in this function [-Werror=maybe-uninitialized]

This actually is always initialized before it is used there, but
let's just silence gcc here.

rtpmux: fix buffer list drop check

While porting to 0.11, the check was mistakenly made constant,
instead of testing for the return value of process_buffer_locked.

Coverity 1139663

matroska: fix content encoding scope validity check

It's 3 bits, and http://matroska.org/technical/specs/index.html
says it can't be 0.

Coverity 1139660

matroskamux: fix PAR fraction sanity check

It was checking par_num twice, and never par_denum.

Coverity 1139634

multiidpsink: warn when setsockopt fails

This doesn't seem to be fatal, but it's good to let the user know
in the logs.

Coverity 1139630

interlace: catch failure to create audio info from caps

Coverity 1139627, 1139628

gstrtph264pay: Reset sps pps variable when state change.

Reset last_spspps and sps/pps arrays  when state transition
GST_STATE_CHANGE_PAUSED_TO_READY.

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

jitterbuffer: improve EOS handling

Make a new method to disable the jitterbuffer buffering.
Rework the update_estimated_eos() method. Calculate how much time
there is left to play. If we have less than the delay of the
jitterbuffer, we disabled buffering because we might never be able to
fill the complete jitterbuffer again.
If we receive an EOS event, disable buffering. We will drain the
buffer and eventually push the EOS event out.
When we reach the estimated NPT timeout and we didn't receive an EOS
event, make one and queue it so that it can be pushed.

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

rtpsession: send reconfigure when internal-ssrc changes

When the internal-ssrc property changes, we want to send a reconfigure
upstream to make payloaders use the new suggested ssrc.
Using the internal-ssrc property to change the SSRC of a stream is not a
good idea and doesn't work when there are multiple senders, we want to
set the SSRC directly on the payloaders. Therefore, deprecate this
property.

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

jitterbuffer: assume a full buffer when eos

Rework the logic to make buffering messages a little, make sure we
don't make the same message multiple times.
Consider the buffer full when EOS was received.

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

rtprtx: Don't forget to unmap rtp buffer in the test

rtprtxsend: Require clock-rate in the caps and handle no ssrc in the caps properly

rtprtx: Provide an ssrc in the test

And increase timeout to allow all tests to run in valgrind.

rtpsession: Fix memory leaks in test

rtpjitterbuffer: Fix hundreds of memory leaks in the test

rtpjitterbuffer: Unref clock id when waiting for the clock is interrupted

rtpcollision: Fix memory leaks in unit test

videomixer: name collectpads object based on videomixer name

Makes it easier to track things in debug logs when there
are multiple mixers and muxers.

videomixer: better logging of incoming events

The pad and parent names are already logged as part of logging
the object. Instead log the full event details.

videomixer: Fix memory leak in unit test

level: Use the correct number of samples to iterate over the input array

Fixes invalid memory accesses and accesses to uninitialised data.

icydemux: Unref dropped events

matroska: fix check for amount of data to read

History shows length==0 should set data to NULL and return,
so we do that too instead of trying to read nothing.

Coverity 206205

deinterlace: fix sign comparison

history_count is unsigned, so the whole comparison will be made
as unsigned, and fail to reject what it was meant to.

Coverity 206204

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