dashdemux: corrected parsing of negative values into unsigned data

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

codecparser: h264: initialize parsing structures

Initialize to 0 these parse structures before filling them: GstH264SEIMessage,
GstH264NalUnit, GstH264PPS, GstH264SPS and GstH264SliceHdr.

When calling the functions which fill those structures, they may fail, leaving
unitialized those structures. This situation may lead to future problems, such
as a segmentation fault when freeing, for example.

This patch initializes to zero these structures, before filling them.

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

codecparser: h265: initialize parsing structures

Initialize to 0 these parse structures before filling them: GstH265SEIMessage,
GstH265NalUnit, GstH265VPS, GstH265PPS, GstH265SPS and GstH265SliceHdr.

When calling the functions which fill those structures, they may fail, leaving
unitialized those structures. This situation may lead to future problems, such
as a segmentation fault when freeing, for example.

This patch initializes to zero these structures, before filling them.

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

liveadder: Make latency property be a uint in millisecs

This restores roughly the same behaviour as the old liveadder element.
Except that the latency now also includes the output-buffer-duration.

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

mpdparser: forbid negative values for duration

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

dashdemux: parser rejects XMLs with negative period duration

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

mpdparser: check segment lists have either duration or timeline

And add error checking along the way.

Add duration where appropriate so unit tests still pass.

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

mpdparser: validate representation set identifier

It must have no whitespace, and must comply with RFC 1738 when
used to build a URL.

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

glslstage: initialize output params to 0 before calling gl functions

The client side API of the Chromium's GPU Process has asserts
in debug mode that check that output params are initialized to 0.

mxfmux: Remove some dead code that could never be called

Coverity CIDs #1328818, #1328819, #1328820.

opengl: build/provide glstereomix/split on gles only systems

geometrictransform: rename gemetric math functions to have their symbols namespaced

Otherwise those symbols can conflict with external libraries when
linking everything statically for mobile targets.

Use the gst_gm_ prefix, short for gst geometric math.

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

waylandsink: don't advertise the local wl_display context

This wl_display proxy is temporary only until waylandsink goes NULL,
at which point the connection to the display is disposed. Unfortunately,
if this is advertised as a GstContext, playbin will cache it and re-feed
it to the sink when it goes PLAYING again, but the wl_display pointer
will at that point be invalid and cause a crash.

Another solution to the problem would be to also cache the GstWlDisplay
object inside the GstContext, which would automatically ref-count
the display connection, but I see no reason in doing that at the moment,
as there are no known users of this GstContext outside waylandsink.
It's probably better to avoid chasing hidden refcounts.

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

glcolorconvert: allow outputting multiple textures for gles3 contexts

the USING_GLES2 includes all GLES3 contexts as well which does support
drawing to multiple buffers.  Instead make or decision solely based on
whether glDrawBuffers is available or not.

glviewconvert: expose the element on gles2 platforms

We can do everything with gles3 however gles2 restricts us not allowing
separated or frame-by-frame multiview modes due to multiple draw buffers.

tests: hlsdemux_m3u: add test for master playlist with missing variant list uri

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

hlsdemux: don't crash or leak memory on broken master playlist

If a (master) playlist contains a variant list entry without a
URI then during parsing of the next variant list entry we are
a) leaking the entry we're currently parsing (new_list), and
b) free'ing the pointer to the previous list entry (list) without
   updating the pointer.

Hence when then adding the URI for the latest parsed entry, incorrect
information is stored, as the information is used from 'list' which
is not valid memory anymore, also leading to crashes.

Fix this by correctly storing the new variant list entry pointer
as needed.

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

hlsdemux: change dispose function to finalize

hlsdemux: remove helper function only used once

hlsdemux: remove deprecated 'fragments-cache' property

It's been non-functional for two cycles now, let's
get rid of it.

hlsdemux: m3u8: move file lookup by sequence directly into code

Nicer to read, two lines of code less, and also the callback
function should've been a GCompareFunc that returns a gint
and not a boolean (it did work correctly, was just confusing).

hlsdemux: m3u8: clean-up: remove unused argument in function call

hlsdemux: m3u8: clean-up: reindent header for better readability

And remove duplicate define.

mxfmux: Remove empty set/get_property()

We can add them again if we ever add properties.

mxfmux: Port to GstAggregator

mxfmux: Set GC essence element UL version to 0x01

ffmpeg otherwise rejects it and the spec is not 100% clear about that.

srtp: Fix critical warning trying to retrieve SSRC from a non fb message

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

mxf: Reorder CFLAGS and LIBS

aggregator: Add create_new_pad() vfunc to allow subclasses to override the default behaviour

Not all aggregator subclasses will have a single pad template called sink_%u
and might do something special depending on what the application requests.

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

mxfmux: Set KAG size to 1

We're not aligning our output in any way, and 0 is invalid.

mxfmux: Don't let the header partition's prev_partition point forward to the footer

mxftypes: Fix generation of operational pattern UL

mxfdemux-structure: Sync fakesink state with the bin to make the application work at all

mxfmpeg: h264 is always byte-stream inside MXF

mxf: Add unit tests for H264/PCM

mxfmux: Don't leak SEEK events when dropping them

mxfvc3: The wrapping is the 15th byte of the essence container UL, not the 16th

In other mappings it is the 16th though.

mxfdemux: If seeking to the remaining parts of the file fails on EOS, consider the stream done

Without this we would run this while loop forever, always seeking again for
the same stream.

mxf: Properly enable unit tests again

mxf: Re-enable mxfmux element

mxfjpeg2000: Fix caps

There is no fields field anymore.

mxfmux: Don't fail SEGMENT events

mxfmux: Push stream-start and caps event before segment event

And don't push the caps event during instance initialization already.

mxfaes-bwf: Format is unsigned if it's 1 byte per channel

... not 1 byte per block.

opencv: handdetect: no need to get a writable buffer

No need to get a writable buffer inside cv_trans_func of opencv video
filter. GstBaseTransform takes care of this.

Automatic update of common submodule

From b99800a to b319909

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

glshaderelement: free the vertex and fragment strings on finalize

Fixes a memory leak when using the vertex/fragment properties.

androidmedia: fix build errors

printf %lld instead of G_GINT64_FORMAT.
Double ret variable declaration.

glshaderelement: remove unneeded reference to <gst/gl/gstglshadervariables.h>

gl: fix usage of string.h and stdio.h functions without including the headers

e.g:
gstglcontext_egl.c:613:7: error: implicit declaration of function 'strcmp'
[-Werror=implicit-function-declaration]
   if (strcmp (G_MODULE_SUFFIX, "so") == 0)

glimagesink: create a context in NULL_READY

So that it's possible for decoders et al. to request the OpenGL context
in their READY_PAUSED transition with decodebin/playbin.

amcviddec: use gstcontext to retreive the OpenGL context

androidmedia: Only allow GL output if the decoder has unknown color formats

If GST_AMC_IGNORE_UNKNOWN_COLOR_FORMATS is set to yes, non-GL output
is still allowed.

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

androidmedia: Add support for GL output in amcvideodec

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

androidmedia: Do not flush codec if it is not started

androidmedia: Allow object to be NULL in gst_amc_jni_*unref functions

androidmedia: Fix debug statement (%d for a gsize argument)

androidmedia: Improve color format debug messages

androidmedia: Improve debug messages

androidmedia: Do not warn we do not support COLOR_FormatSurface

gl: remove unneeded shader variable parsing code

glshaderelement: implement setting arbitrary uniforms

Currently float and int are supported by default. vec2, vec3, vec4
and mat4 are supported if graphene is used.  Of course if one wants
to set custom uniforms they can also be set using the create-shader
signal.

opencv: handdetect: free best_r with delete

best_r is a cv::Rect, so it should be freed with delete and not with
g_free()

opencv: motioncells: remove unnecessary check

We know that the gchar arrays contain at least one string. Furthermore,
g_strfreev() checks if the array is NULL and simply returns if it is.
Hence, there is no need to check if the array is empty before using
g_strfreev().

CID 1327412-1327415

hls/m3u8: Update current position in all cases

In order to ensure the sequence_position will always be consistently updated,
store the current file duration.

This way, when we advance, we can always increment the position based on what
was previously outputted.

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

dshowvideosink: Fix 64bit compatibility issue

SetWindowLong works only for 32 bit systems, thus windows events (move/resize)
where not interpreted on 64 bit systems

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

gl: be consistent in gobject boilerpate

GST_GL_IS_* vs GST_IS_GL_*

git grep -l 'GST_GL_IS_' | xargs sed -i 's/GST_GL_IS_/GST_IS_GL_/g'

add glliveshader program to .gitignore

pnmdec: drop frame in case of _handle() failure

Allows baseclass to handle it from there

Related to:

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

pnmdec: propagate input state after parsing

Store and copy input state fields when setting the
output state of the decoder. Avoids problems like
the framerate set by an upstream element being ignored

Related to:

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

pnmdec: completely reset parsing state at flush

Makes sure the mngr struct reflects a clean state
for the next frame, avoiding failures like:

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

glshaderelement: implement on-demand create-shader signalling

One may not have an GstGLContext available or current in the thread where one
would need to update the shader.  Support this by signalling create-shader
whenever the one-shot 'update-shader' is set to TRUE.

gl/calayer: don't leak a gl shader object

We will always overwrite the ca_sink->redisplay_shader without freeing the
object.

gtk: separate out the widget/window destroy callbacks

Fixes assertion due to the sink_finalize() being run before the widget destroy
callback.

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

gl/examples: add a live shader demo using the new GstGLSLStage

Implemented with videotestsrc ! glshader ! glupload ! gtkglsink

Errors on an invalid shader compilation are ignored however any error
provided by the glsl compiler is printed to stdout.

srtpdec: skip padding when mapping RTP packets

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

amcvideodec: Properly forward the return value of gst_video_decoder_negotiate()

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

amcvideodec: Implement support for COLOR_QCOM_FormatYVU420SemiPlanar32mMultiView for decoding

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

glslstage: add brackets around pointer dereference referencing an array

While technically, i is always 0 and *vertex_sources[i++] is equivalant
to (*vertex_sources)[i++].  Be future-proof in the case of code
moves/changes/etc.

CID 1327406

gl/caopengllayersink: port to new GstGLShader API

fixes build error:
"undefined symbols for architecture:
gst_gl_shader_compile_with_default_vf_and_check"

gl/tests: port glcontext test to opengl

Now uses vao's and vbo's when possible like the rest of the gstgl library.

glshader: port element to GstGLSLStage

- Provide a shader property to set the full shader pipeline
- Provide vertex and fragment properties for just providing simple
  shader sources.

glshader: port to using GstGLSLStage objects for string management

A GstGLShader is now simply a collection of stages that are
compiled and linked together into a program.  The uniform/attribute
interface has remained the same.

glsl: add a shader stage object GstGLSLStage

Represents a stage (vertex, geometry, fragment, etc) in the shader
pipeline.

gl/shader/variables: include generic gl.h header

Otherwise we may miss some forward declarations

Fixes build error: undefined reference to GstGLSLStage

gl: add some GLSL utility functions

Specifically parsing/setting GLSL versions and the shader related
function table.

dtls: Fix name conflict with openssl on win32

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

hls: if media sequence is not specified, use 0

Allows playlists that are missing the mediasequence information to
be correctly parsed. If the playlist was updated without reseting
the mediasequence it would constantly increase over subsequent updates,
leading to issues during playback.

waylandsink: fix fallback buffer allocation

Copy-paste mistake; the wl_buffer was attached on the wrong buffer...

waylandsink: rename GstWlDisplay::formats array to shm_formats

These formats are specific to the wl_shm interface. We are going
to add dmabuf formats later as well.

waylandsink: replace the custom GstWlShmMemory with GstFdMemory

po: Update POTFILES.in for OpenCV C++ file rename

hls: Allow debugging of the playlist

Helps debugging issues

hlsdemux: Avoid negative sequence numbers

For live streams, we want to make sure there's a certain distance
between the sequence to play and the last (earliest) fragment.

The problem is that it assumes there are at least 3 fragments in
the playlist, which might not always be the case (like in the case
of a server restarting and gradually adding fragments).

In order to avoid ending up with negative sequence numbers (which
will just loop forever), limit the new target sequence number to
the highest of:
* either the first sequence number of the playlist (fallback)
* or 3 fragments from the last one (standard behaviour)

opencv: switch remaining to C++

Switch remaining C files to C++ for consistency with other OpeCV elements,
and support of the new 2.4.11 API.

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

mpdparser: Set default last_byte_pos to -1

The value is optional in the range, and if it is absent it means we should
download until the end of stream. Not until position 0.

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

adaptivedemux: HTTP ranges are inclusive, GStreamer segment.stop is exclusive

Translate the values accordingly.

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

opencv: motioncells: Switch to C++

Switch gstmotioncells to C++ for consistency with other OpenCV elements,
and support of the new 2.4.11 API.

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