mpdparser: Use 64 bit integer format strings when passing a 64 bit integer

Otherwise we'll only get half of its bits printed on 32 bit architectures.

For this, promote the %d-style format strings to something that accepts
64 bit integers with G_GINT64_MODIFIER.

mpdparse: Validate format strings before using them

Using format strings from an untrusted source without validation is
calling for problems, and at least allows to remotely crash your application.
If not worse.

gl/dispmanx: Fix build

gl/example: fix build error when compiling cocoa-videooverlay

libtool: error: ignoring unknown tag OBJC
and
clang: error: argument unused during compilation: '-pthread'

gl/window/dispmanx: fix compiler warning

gl/window: use the default main loop implementation for all backends

fixes glimagsink being unable to display.

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

glwindow_cocoa: use parent default implementation

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

mpdparser: Add the startNumber as offset to the segment index when using a template

dtlsdec: Fix critical warning "got data flow before stream-start event"

Forward sticky events on requested src pad.

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

vc1parse: remove useless value assignments

In all these cases ret is set but overwritten before the return of the function

examples: gst-camerabin2-test: protect from division by zero

Highly unlikely to have 0 captures, but protect from crashes in the future by
doing none of the math if there is no data.

configure: lower curl requirement to 7.35.0

Build and unit tests were tested against 7.35 too now,
which is the version in Ubuntu 14.04.

mpegtsbase: do not access variable after unref

Data might not exist anymore

tests: audiointerleave: test not setting positions

Disable "channel-positions-from-input", but without actually giving
a position table, so every position should be NONE

tests: Fix indentation in audiointerleave test

zbar: don't ignore zbar_scan_image() errors

glsyncmeta: Set new sync point when copying

Set a new sync point when copying instead of
transferring the (non-refcounted) GL sync object
from the source meta.

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

glsyncmeta: Add some debug output

check: Use GST_CHECK_MAIN () macro everywhere

Makes source code smaller, and ensures we go through common initialization
path (like the one that sets up XML unit test output ...)

simplevideomark: fix display of videomark partially or fully outside video

In case of the videomark being partially or fully outside, an error was being
thrown saying the mark width is more than video width. And when the width,
offset properties are set to maximum it resulted in crash. Instead of throwing
an error, add logic to detect the mark in case of partial visibility or don't
show the mark when it is outside.

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

tsdemux: enable Chinese AVS video stream

Chinese broadcaster encapsulate AVS video codec into MPEG2-TS. They
use the stream_id 0x42 to identify AVS video streams. It should be noted
that this id is currently within the ISO reserved range, hence it's
utilisation is unofficial.

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

glmemory: provide compatibility definition for GLES2 for GL_RGBA8

gl: consolidate internal_rgba_format into glmemory

Expose some useful value format conversion functions available in
GstGLMemory.

audiointerleave: Always have "channels" be the actual pad count

Don't force it anywhere

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

audiointerleave: Use the channel count from the set caps

This is the same number that was used to allocate the buffer

aggregator: Document that the latency is in ns

amc: Only lower ranks of OMX.Exynos. audio codecs, the video codecs are actually working

amc: Give marginal rank to codecs that start with OMX.Exynos.

OMX.Exynos. codecs are existing on some devices like the
Galaxy S5 mini, and cause random crashes (of the device,
not the app!) and generally misbehave. That specific device
has other codecs that work with a different name, but let's
just give them marginal rank in case there are devices that
have no other codecs and these are actually the only working
ones

androidmedia: Give lower ranks to codecs not starting with OMX.

On some devices there are codecs that don't start with OMX., while
there are also some that do. And on some of these devices the ones
that don't start with OMX. just crash during initialization while
the others work. To make things even more complicated other devices
have codecs with the same name that work and no alternatives.
So just give a lower rank to these non-OMX codecs and hope that
there's an alternative with a higher rank.

Also stagefright gives codecs starting with OMX. a higher rank too and
considers other codecs that don't start with OMX. as software codecs.

tests: zbar: check for frame field on attach-frame=true

Also check for the sample to have a buffer and caps
associated with it.

Related to:

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

vtdec: Require width and height field for H264

This decoder does not work if width and height field are not set
in the sinkpad caps. Let's make this explicit by adding them to
the template caps.

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

glcontext_gpu_process: close the window when done

glwindow_gpu_process: avoid empty struct

glwindow: move main loop/context creation back to init/finalize

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

examples: Fix gl usage without wayland support

Not all platforms have wayland support. Handle that gracefully at
compile time

rtmpsink: Initialize GstMapInfo

Avoids doing a call to unmap with it uninitialized

CID #1302834

audiovisualizer: clean dereferences of private structure

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

vtenc: fix keyframe request race condition

It is incorrect to modify the frame properties after passing them, since
VTCompressionSessionEncodeFrame takes reference and we have no control
over when it's being used.

In fact, the code can be simplified. We just preallocate the frame
properties for keyframe requests, and pass NULL otherwise.

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

audiovisualizer: make private all variable subclasses don't need

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

zbar: add frame sample to barcode message

New attach-frame property enables barcode frame
dumping when set to true.

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

gl: Fix compiler warning

gstglwindow_wayland_egl.c:246:41: error: unused variable 'surface_listener' [-Werror,-Wunused-const-variable]
static const struct wl_surface_listener surface_listener = {
                                        ^

androidmedia: Conditionally use get_{input,output}_buffer() Android 21 APIs

Also properly set limit/position on byte buffer, some codecs prefer to have
correct values there.

glwindow: fix compiler error

gstglwindow.c:1118:544: error: 'return' with no value, in function returning non-void [-Werror]
   g_return_if_fail (GST_GL_IS_WINDOW (window));

gl/examples: update gtk examples for glupload

gl/examples: update gtk examples for wayland

gl/wayland: implement setting the render rectangle

Places our subsurface at the rectangle provided position

glwindow: handle gst_video_overlay_set_render_rectangle

gl/wayland: don't block the event loop after poll

Use the dispatch_pending set of functions which just run the currently
queued up events instead of potentially waiting for an event to occur.

tests/gl/gtk: implement setting a wayland display/surface

gl/wayland: implement basic video overlay support via subsurfaces

Currently does not position the subsurface relative to the parent surface at all

gl/wayland: allow a NULL wl_event_queue

perform operations on the default wl_display event queue in that case

gl/utils: implement wayland display GstContext

requried for sharing GL contexts or subsurface support

gl/wayland: run each window on a separate queue

Based on patch by Julien Isorce <julien.isorce@collabora.co.uk>

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

gl: Don't leak temp strings in _RGB_pixel_order()

Fix a memory leak of temporary strings when computing
swizzling of RGB formats.

glfilter: Add transform_internal_caps() vfunc

Add a vfunc that is called by glfilter before it sets
caps features and intersects with the peer caps, and
move removing the size from caps into its default
implementation. Allows sub-classes to do more
sophisticated management of the size fields in case they
don't support arbitrary resizing or have distinct
preferences.

glfilter: Don't remove size fields from caps

Instead of removing size fields, set them to the full range
like videoscale does, so the caps are clearly unfixed in
all cases.

videoaggregator: Add class property to disable caps scaling

Add preserve_update_caps_result boolean on the class to allow
sub-classes to disable videoaggregator removing sizes and framerate
from the update_caps() return result.

videoaggregator: Catch errors, and allow sub-class to return NULL from get_output_buffer()

A return value of GST_FLOW_OK with a NULL buffer from get_output_buffer()
means the sub-class doesn't want to produce an output buffer, so
skip it.

If gst_videoaggregator_do_aggregate() generates an error, make sure
to propagate it - don't just ignore and discard the error by
over-writing it with the gst_pad_push() result.

rtmpsink: Do not crash when receiving buffers after GST_FLOW_ERROR

If the RTMP URI is invalid, the rtmpsink will return GST_FLOW_ERROR.
If it still receives buffers after that, it shouldn't crash.

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

dashdemux: avoid assertion when splitting buffer

Pass size=-1 as is if that is the case instead of subtracting
the offset. Otherwise we have an invalid size passed for that
buffer.

dtlssrtpdec: Release requested pads

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

dtlsdec: Remove unnecessary ref/unref operations

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

dtlsdec: Fix release request pad function

Don't unref the pad reference we don't own and just remove the pad.

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

hlsdemux: Search more when advancing fragment

In live situations, it is not uncommon for the current fragment to end
up out of the (updated) play range (lowest/highest sequence). But the next
fragment to play *is* present in the play range.

When advancing, if we can't find the current GstM3U8MediaFile, don't abort
straight away. Instead, look if a GstM3U8MediaFile with the next sequence value
is present, and if so switch to it.

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

gl: add GstGLContextGPUProcess backend

It builds its GL vtable from a proc address provided
by the application.

gl: add and use gst_gl_internal_format_rgba

Previously when compiling GstGL with both GL and GLES2,
GL_RGBA8 was picked from GL/gl.h. But a clash may happen at
runtime when one is selecting GLES2.

gst_gl_internal_format_rgba allows to check at runtime
if it should use GL_RGBA or GL_RGBA8.

glwindow_x11: use parent default implementation

glcontext_glx: rename variable from window to context

glwindow: initialize debug in class_init

Useful if one uses gst_gl_window_x11_new directly
instead of gst_gl_window_new.

glcontext: initialize debug in class_init

Useful if one uses gst_gl_context_egl_new directly
instead of gst_gl_context_new.

glcontext: add gst_gl_context_set_display helper

glcontext: add missing _CAST after G_TYPE_CHECK_CLASS

glwindow: add missing _CAST after G_TYPE_CHECK_CLASS

glwindow: provides some default implementations to factorize with all backends

aggregator: Push EOS on error return.

Before shutting down the srcpad task due to a
downstream error, push an EOS to give downstream
a chance to shut down somewhat cleanly.

mpegtsmux: don't overwrite value

Value of res is reset to FALSE in each iteration of the while loop. We want to
conserve TRUE if any pad event succeeded until we arrive to done.

Also, buf is set to the value of *outbuf twice. Removing the first assignment
since the second one is outside of a conditional.

opencv: 'for' loop initial declaration

'for' loop initial declarations are not allowed in C89, moving the declarations
to before the 'for' loops.

gleffects: 'for' loop initial declaration

'for' loop initial declarations are not allowed in C89, moving the declarations
to before the 'for' loops.

resindvd: remove unused value

The data variable is incremented but never read again. Remove the unused value
assignment.

hlsdemux: remove more unused attributes

Those are not used or only read

hlsdemux: remove unused attribute

It is never read for anything useful

gloverlay: fix a leak

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

gloverlay: properly handle errors while loading file

Post an error on the bus if anything bad happens while reading
and parsing the image file.

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

gloverlay: remove unused type_file field

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

gl: propagate return value from filter and filter_texture

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

dvbsuboverlay: remove unused assignment

buf is incremented just before returning, this new value is never used.
Removing unused assignment.

adaptivedemux: 'for' loop initial declaration

'for' loop initial declarations are only allowed in C89, moving the declaration
to before the 'for' loop.

exclusion: exception when set factor to 0

When factor property is set to 0, transform just returns.
Adjust the minimum value to 1.

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

gleffects: properly initialize the shaders across contexts implementing multiple API's

gleffects_laplacian: fix shader compilation in gl3/gles2

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

curlsink: handle socket callback for active FTP connections as well

Since version 7.28.0, libcurl allows application to set
socket options for active FTP connections.

Bump libcurl requirement to version tested.

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

dashdemux: Fix segfault introduced by segment iteration refactoring (2105a310).

The functions to get the next fragment, next fragment timestamp and to advance
to the next fragment need to work differently when stream->segments is NULL.
Use logic similar to that introduced by commit 2105a310 to perform these
functions.

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

dashdemux: use signed integet to represent segment index

We need to be able to represent -1 to check when we are EOS
on reverse playback and unsigned integer won't allow that.

CID #1298193
CID #1298194
CID #1298195

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

dashdemux: fix outdated example launch line

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

gl/wayland: remove dead event source code

gl/wayland: add GstGLDisplayWayland

Simple implementation split from GstGLWindowWayland

Can now have multiple glimagesink elements all displaying output
linked via GL or otherwise (barring GL platform limitations).

The intel driver is racy and can crash setting up the two glimagesink contexts.

e.g.
videotestsrc ! tee name=t ! queue ! glupload ! glimagesinkelement
  t. ! queue ! gleffects_blur ! glimagesinkelement

videotestsrc ! glupload ! glfiltercube ! tee name=t ! queue ! glimagesinkelement
  t. ! queue ! gleffects_blur ! glimagesinkelement

gltransformation: fix DrawElements call for element array buffers

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

gl/calayer: provide the exact GL api version for the wrapped context

Otherwise we could end up being mistaken for the diference between a
gl3 and a gl2 context resulting in a failure getting the list of
extensions from the wrapped context due to the difference between
glGetString and glGetStringi for the GL_EXTENSIONS token.

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

dashdemux: remove unnecessary semicolon

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

gl: win32: do not call SetParent in release_parent_win_id()

When called from gst_gl_window_win32_close(), internal window
could not exist, and if it does it's going to be destroyed just
after that anyway. Also it causes window_proc() to be called
and crash because it gets a NULL context.

When called from gst_gl_window_win32_set_window_handle() we are
going to set another parent anyway, and it's probably better to
reparent directly instead of passing by a NULL parent which could
cause the internal window to popup briefly.

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