gl/color: remove if statement with no effect

Coverity 1199698

gl/color: pass large value by reference

Coverity 1199700

srtpdec: fix assertion checking ssrc from rtcp packets

rtcp_buffer_get_ssrc is called even with RTP buffers. this means we
might end up with an exception and not find any valid RTCP packet type
and thus hit GST_RTCP_TYPE_INVALID. we now take care of this.

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

jp2kdecimator: fix maybe-uninitialized compiler error

mpegtsmux: Remove unneeded unref

packet_buffer has already been cleared in all code paths leading
to this point.

CID #1199696

dvbsrc: Handle symbols added in minor version 7 and 8

Some symbols were re-defined in minor v8 (GO TEAM BACKWARDS COMPATIBLE!)

Some symbols were added in minor v7

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

gl/mem: cache the stride/unpack length for upload

gl/color: explicitly map the buffers we are copying into

Otherwise it may not be downloaded at all.

dvbsrc: Add comment with all DVB API v5 minor changes

There wasn't a definite list anyway, so I just dove in the kernel
and extract all that information.

gl/color: cache the temporary textures

gl/mem: provide defines for tokens not defined in GLES2/ancient platforms

gl: fixup incorrect number of arguments

gl: fix incorrect usage of g_return_if_fail

gl/mem: allocate the memory per plane

This patch provides the basic infrastructure required for this.
Upload and Download has been ported to this.

Has the nice effect of allowing GstGLMemory to be our
refcounted texture object for any texture type (not just RGBA).

Should not lose any features/video formats.

gl/eglimage: #if 0 unused code

gl: add colorconvert object that converts between color spaces/formats

Currently used by both upload and download objects separately.

faac: fix faac API error checking

Coverity 1139790

dashdemux: do not ignore failure to register typefinder

Coverity 1139629

dashdemux: fix time leak

Coverity 1147016

dashdemux: remove dead code and variable

A recent commit made stream not able to be NULL, ending up with
a flag being always set to FALSE when it matters.

Coverity 1146968, 1147012

curl: remove unneeded NULL check

We just checked for NULL and allocated if it was. And glib aborts
if it can't allocate, I'm told.

Coverity 1139841

curl: remove duplicate assignment

Coverity 1197708

hlsdemux: Also update the sequence position when updating non-live playlists

hlsdemux: Try reloading the playlist first if downloading a fragment fails

But only add this for non-live playlists. For live playlists we already
have another thread that is periodically updating playlists.

Reason for this is that sometimes downloading a fragment can fail because
the URIs have changed or expired since last time.

hlsdemux: When updating a non-live playlist make sure to find the current sequence by time

Sequence numbers in different playlists are not guaranteed to be the same for the
same position, e.g. fragments could have different durations in different playlists.

In theory we should do exactly the same for live playlists, but unfortunately we can't
because doing this kind of seeking requires the complete playlist since we started
playback. For live playlists the server is however dropping fragments in the beginning
over time and we have no absolute time references.

smoothstreaming: avoid using an uninitialized manifest

If we did not set the manifest yet, early out in the seeking query.

Coverity 1139737

h264parse: Set framerate even for interlaced videos

Co-Authored by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

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

mpegts: add atsc terrestrial virtual channel table

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

liveadder: fix memory leak querying sink caps

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

hlssink: catch failure to write playlist file

Coverity 1139613

kate: fix implicit promotion to signed int breaking on largeish inputs

Coverity 1139858, 1139857

ladspa: remove duplicate assignment

Coverity 1139706

mimdec: fix open error checking

Coverity 206364

schrodec: remove unnecessary check

The tag was dereferenced earier. From the libschroedinger code,
it's not obvious to see whether tag and frame would be NULL at
the same time. I think is likely that both will be non NULL
here, but that's not certain. Additional tests may be needed
to avoid dereferencing tag and/or frame, but what to do if
only one is NULL isn't obvious, as the _get_tag function does
transfer ownership so isn't undoable.

Coverity 1139850

smoothstreaming: remove unnecessary check

The function is static, the only call site passes a non NULL
pointer, and the pointer is dereferenced before anyway.

Coverity 1139839

srtp: remove dead assignment

Coverity 1197707

mpegts: fix sdt length check

It was originally test for 9 bytes (as the comment says) and was
rewritten buggily. So rewrite it a third way, which is now
hopefully consistent with the original and the comment, while
being more sense-making to humans.

Coverity 1139654

Revert "tsmux: catch alloc failure"

Turns out glib aborts on allocation failure, so this is pointless.
We'll just ignore Coverity warnings on such constructs.

This reverts commit d347809a82e24057302f948269392e76aaacea10.

mpegts: turns out glib aborts on allocation failure

So this is actually pointless. We'll just have to ignore
Coverity moaning on those.

Revert "mpegts: test for allocation failure"

This reverts commit 224cb81b8fcbf03df5e07f045ecb9ecf64189400.

mpegts: test for allocation failure

While it's unlikely to get there, it silences the coverity warning
on the error code path that we test for NULL before freeing, when
all branches there are from locations where pmt cannot be NULL,
and removing the NULL check makes the code more vulnerable to a
hypothetical future branch from somewhere where it can be.

Coverity 1139852

asfmux: remove unnecessary test

We've been dereferencing that object like there's no tomorrow
in the code just before that.

Coverity 206369

asfmux: remove stray gst_buffer_unmap on error path

A previous commit removed the map, and the unmap on the main
code path, but not the one on the error path.

Coverity 1139930

autoconvert: remove dead code

Those events were previously taken from a cached events list,
which was removed during the port to 0.11, but this bit stayed.

Coverity 1139684

camerabin2: remove unneeded check

A message may not be NULL in the message handling function
(and nothing there sets it to NULL).

Coverity 1139848

compare: special case empty regions with 1 SSIM to avoid dividing by 0

Coverity 1139689, 1139688

inter: remove dead code

Coverity 1139666

liveadder: remove dead code

From the 0.11 port

Coverity 1139676

resindvd: avoid crashing in pathological case

When we'd see an unknown stream type, then a SDDS stream.
Then we'd get to the end of the switch with a NULL temp stream
pointer, and dereference it.

Coverity 1139708

mpegtsdemux: catch prev-not-found when inserting in the group list

While this probably should never happen if callers are well behaved,
this avoids a crash if it does. With a warning about it. Unsure if
it'd be better to not add at all, but it should not happen...

Coverity 1139713

mpegtsmux: guard against _dispose being called mutiple times

_dispose calls _reset, so we need to make sure _reset handles
already NULLed fields.

Coverity 1139843

tsmux: prevent possible double free on error path

Spotted while looking at a Coverity issue in the area.

tsmux: catch alloc failure

While it will probably not trigger, it should silence a Coverity
warning about the fail code path testing for NULLness before
freeing, where the buffer was already dereferenced. It seems
safest to keep that test, in case future goto fail statements
happen to have a NULL buffer there.

Coverity 1139851

decklink: initialize priv to NULL

While the code that creates the object sets priv to some existing
pointer after new, this ensures any future new not doing this will
hit the various priv!=NULL asserts in the code.

Coverity 1139935

speed: make duration query able to convert bytes to time

It was only querying in time, but then trying to use dead bytes
to time conversion code.

Coverity 1139677

mfc: fix left crop change test

It was testing a noop

Coverity 1139643

opus: add missing va_end in variadic function

Coverity 1139944

shm: remove dead code

Coverity 1139683

mfc: reset "in use" flag on failure to initialize

mfc: test for allocation failure before dereferencing

Coverity 1139849

mpeg2enc: remove dead assignment

Coverity 1139829

gl: pass large structure by const pointer, not value

Avoids large pointless memcpy.

Coverity 206236, 206237

camerabin: add any feature to viewfinderbin static pad template

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

camerabin: report an error if we fail to link vfsrc and viewfinder queue

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

d3dvideosink: post proper error message when window disappears

flite: fix off by one in channel mask building

The first loop would shift 1 by 64, which is either undefined
or implementation defined, instead of clearing the top bit.

Coverity 1197691

hls: restore NULL test mistakenly removed

Thanks to tpm for point out I'm an idiot.

hls: only set DISCONT flag on a valid buffer

Recent refactoring causes this code to be called with either a NULL
fragment, or a non NULL fragment. In the former case, we don't have
a buffer. In the latter case, the original code dealing with DISCONT
assumed the buffer was valid. Testing for a NULL buffer here thus
does not seem to change the intent, and fixes:

Coverity 1195147

mxfdemux: guard against NULL material track

Just to be on the safe side.

hls: bring NULL test before dereference

Coverity 1195168

gl: test for frame NULLness before dereferencing it

Coverity 1195172, 1195171

dvbsuboverlay: Avoid infinite loops on short data

dvb: fix compilation

gl: fix leaks

As the relevant variables are initialized to 0/NULL, we can loop
over the full range and make sure we free partial allocations
when an error happens partway through initialization.

dvb: reject too long socket paths

Coverity 206004

mxfdemux: implement simple KEY_UNIT seeking

If a KEY_UNIT seek was requested, adjust segment
start to position of the key frame.

resindvd: guard against overflow in audio subtitle streams

Turns out there was the same issue as with subtitles.

There is space for a single audio stream, but up to 255
may be used based on a uint8_t value in a struct, which may
or may not be read from the (untrusted) data.
A comment in ifo_types.h says this value is either 0 or 1, so
we can ensure this here without drawbacks.

Coverity 1139585

resindvd: guard against overflow in menu subtitle streams

There is space for a single subtitle stream, but up to 255
may be used based on a uint8_t value in a struct, which may
or may not be read from the (untrusted) data.
A comment in ifo_types.h says this value is either 0 or 1, so
we can ensure this here without drawbacks.

Coverity 1139586

mxfdemux: fix seeking, send data starting from a key unit

Fixes multiple seeking issues. When doing ACCURATE or normal
non-KEYUNIT seeks, mxfdemux would just send data from the
edit unit that covered the seek position, whether that's
a keyframe or not. Decoders would only output things from
the next keyframe then, which means there's a gap between
the start of the segment and the first decoded data in
some cases. In combination with gst-editing-services this
might result in a frozen picture for the duration of that
gap at the beginning (if videorate fixes up the first
buffer's start timestamp to cover the entire gap), or
a black frame (if no videorate is used and videomixer
fills the gap). Also fixes A/V sync issue when requesting
a KEYUNIT seek.

dtsdec: fix buffer overflows

Channels can be up to 7. Also add a guard in case more can be
returned in the future.

Coverity 1139820, 1139821

directsoundsrc: Fix critical due to missing debug category initialization

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

festival: fix fd leak

Coverity 1139831

speed: Clarify passthrough in switch statement

We do want the caps event to be forwarded downstream

CID #1139754

soundtouch: Add missing break in switch

Whoops, modifying the pitch when modifying the output rate ...

CID #1139749

rsvg: Add missing break in switch statement

This wasn't fatal before (worst case it would clear an already empty
adapter).

CID #1139758

mpeg2enc: Add missing break in switch

QUERY_CAPS is fully handled, we don't need to passthrough to the default
handler.

CID #1139756

bz2: Initialize variables

There is a small chance that we might end up in the done step without
having any output available.

Furthermore, when going through not_ready, we need to ensure gst_buffer_unmap
has a properly initialized GstMapInfo.

CID #1139923
CID #1139924
CID #1139919
CID #1139920

check: Fix mpegtsmux caps

Incoming streams need to be parsed. Doesn't fix the test completely though

mpegtsmux: Initialize helper mpegts library

We now use that library, we need to initialize it so that debug
categories (amongst other things) get properly initialized

mxf: fix stop date parsing using the wrong input

Coverity 1139645

jp2kdecimator: fix copy/paste errors in expressions

Correct formulae found in ITU T.800, B.12.1.[345]

Coverity 1139649, 1139650, 1139651

tests: fix shm test deadlock

The test was not bringing the source back to NULL, so its thread
could start creating a buffer while the pads were being shutdown.

glimagesink: Create GL context and set up window from the streaming thread

gst_gl_context_create() might need to dispatch some operations to the
application's main thread, and calling this in the change_state function
can cause deadlocks.

configure: Fix check for iOS for the OpenGL support

There is also an i386 version of iOS, which is for the simulator.
Better use our already existing HAVE_IOS check instead of relying
on the host triplet.

wildmidi: fix possible uninitialized variables

mpdparse: only add location when node existed

Actually return FALSE when the location node was non-existing
Only add the location node when it existed

mpegtsmux: Require parsed/framed input for most of the supported formats

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

d3dvideosink: Only pass a dest rectangle if set, otherwise pass NULL

Call with an uninitialized rectangle will cause errors.

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

tests: fix gl unit tests for recent API changes

gl: pass video info's by reference

gl: fix array initialization