matroska: blindly fix writing variable length negative values

Spotted while fixing something else in the area.

Nothing calls this with a negative value.

matroska: do not lose the top bits when writing a > 32 bit value

Coverity 1139806

videoflip: add missing break in switch

Coverity 1139755

matroska: do not try to call gst_pad_query_default on a NULL pad

gst_matroska_parse_query can be called explicitely with a NULL pad.
If we reach this point with a NULL pad, fail the query.

Coverity 1139715

matroska: do not return GST_FLOW_OK if we did not get a buffer

Coverity 1139714 (which will likely come back in another guise,
as the _read_init call can have a failing _map)

matroska: catch failure to map buffer

Avoids dereferencing NULL.

Coverity 1139712

avimux: refuse caps with invalid framerate

Coverity 1139701

qtmux: handle 0 size packets without dividing by 0

Coverity 1139691

qtdemux: guard against invalid frame size to avoid division by 0

Coverity 1139690

qtdemux: trivial typo fix

speexdec: remove dead code

fpp can never equal 0 here, or the loop would not execute at all.
Zero fpp was possible before as the loop condition was allowing
it specifically, but no more.

Coverity 1139681

oss4: remove dead mixer code

This was partly removed in the port to 0.11. If still needed,
it's still there in the history.

Coverity 1139687

oss4: fix a missing unlock and a return-only-when-assertions-enabled

Spotted on the side while looking at another issue.

v4l2: Correctly map RGB32 format

In v4l2 specification, RGB32 has the alpha, or pading, first, not last.
See http://linuxtv.org/downloads/v4l-dvb-apis/packed-rgb.html .

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

flacdec: remove dead code

For 8 bit width, we always have depth==gdepth==width==8.

Coverity 1139678

mpegaudioparse: remove dead code

A stricer check is already done earlier, and integer overflows
do not seem possible here.

Coverity 1139675

rtpvrawpay: guard against pathological "no space" condition

Even if one woul hope one pixel can fit in a MTU, ensure we do not
overwrite a buffer if this is not the case.

Spotted while looking at Coverity 1208786

rtpjpegdepay: sanity check for NULL qtable

Can happen (at least in crafted stream)

Coverity 1208778

wavparse: pass on tags from upstream if there are any

Don't just ignore upstream tags from e.g. an ID3 tag before
the .wav data, pass them on downstream.

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

rtpjitterbuffer: optimize timer update

When we are not doing retransmission, we just need to find the current
seqnum so we can stop when we found it.

rtpjitterbuffer: small optimizations

Small optimizations where we can.
Add some more debug.

rtpjitterbuffer: signal when next_seqnum changed

Signal the pushing thread when the next_seqnum changed and we might be
able to push a buffer now.

rtpjitterbuffer: only signal event when head changed

After adding a buffer, only signal the pushing thread when the head
buffer changed or else we cause a useless wakeup.

rtpjitterbuffer: rework packet insert

Rework the packet queue so that the most common action (insert a packet
at the tail of the queue) goes very fast.

Report if a packet was inserted at the head instead of the tail so that
we can know when to retry _pop or _peek.

rtpvraw: use plane pointers when needed

Pack/unpack planar formats to/from the first plane.

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

souphttpsrc: Remember if a redirect is permanent or not and store it in the query

goom: Remove french comment saying to prefix functions

All non-static function in this file are already prefixed with goom_.

goom: fix compilation on ios-arm7-10.9 and osx-x86_64

uint is not a standard type, and the rest of the code uses
Uint which is locally typedefed to unsigned int.

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

goom: fix undefined behaviour of left-shift

Don't left-shift into the sign bit, the result is undefined and potentially
an overflow could flip the sign.

qtdemux: check return from qt_demux_video_caps

Now qtdemux_video_caps() can return NULL. We need to check this return before
using it's value.

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

docs: remove outdated and pointless 'Last reviewed' lines from docs

They are very confusing for people, and more often than not
also just not very accurate. Seeing 'last reviewed: 2005' in
your docs is not very confidence-inspiring. Let's just remove
those comments.

qtdemux: initialize caps pointer to null

Make sure the caps pointer returns initialized when using it in
qtdemux_parse_tree ().

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

rtpjitterbuffer: Clear last_pt on flush-stop.

Otherwise, we don't recheck the buffer caps for clock-rate
properly on the next chain.

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