glimagesink: Check if context creation failed before trying to use it

Otherwise we will cause assertions everywhere by passing NULL to functions
and eventually crash when dereferencing a NULL pointer.

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

glcontext: Add more assertions to make sure that everything sets the GError during context creation if something fails

freeverb: Don't pass floats to abs() but instead cast them to ints first

gstfreeverb.c:781:29: error: using integer absolute value function 'abs' when
      argument is of floating point type [-Werror,-Wabsolute-value]
    if (abs (out_l2) > 0 || abs (out_r2) > 0)

rsvgdec: Unref handle and output state on error paths

CID 1214604

opensles: Work around race condition in Android < 4.2 that leads to deadlocks on shutdown

We need to sleep a bit before destroying the player object
because of a bug in Android in versions < 4.2.

OpenSLES is using AudioTrack for rendering the sound. AudioTrack
has a thread that pulls raw audio from the buffer queue and then
passes it forward to AudioFlinger (AudioTrack::processAudioBuffer()).
This thread is calling various callbacks on events, e.g. when
an underrun happens or to request data. OpenSLES sets this callback
on AudioTrack (audioTrack_callBack_pullFromBuffQueue() from
android_AudioPlayer.cpp). Among other things this is taking a lock
on the player interface.

Now if we destroy the player interface object, it will first of all
take the player interface lock (IObject_Destroy()). Then it destroys
the audio player instance (android_audioPlayer_destroy()) which then
calls stop() on the AudioTrack and deletes it. Now the destructor of
AudioTrack will wait until the rendering thread (AudioTrack::processAudioBuffer())
has finished.

If all this happens with bad timing it can happen that the rendering
thread is currently e.g. handling underrun but did not lock the player
interface object yet. Then destroying happens and takes the lock and waits
for the thread to finish. Then the thread tries to take the lock and waits
forever.

We wait a bit before destroying the player object to make sure that
the rendering thread finished whatever it was doing, and then stops
(note: we called gst_opensles_ringbuffer_stop() before this already).

mpegts: mpegtsdescriptor: fix gobject-introspection warnings

Adds missing : to gobject-introspection markers

mpegts: sections: prevent assertion when packet parsing fails

the descriptors entry can be left as NULL and freeing the structure
will fail (assertion happens)

motioncells: fix memleak

Check gst_buffer_map return and remember to unmap and free memory before
returning

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

bluez: fix property names

These were forgotten when they got renamed.

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

gl/download: update to be similar to the glupload semantics

gl/colorconvert: fix up alpha clobbering

Previously it would only work if the alpha value was in the last
component (RGBx, BGRx).  Now it works wherever the alpha value may
be (xRGB, xBGR, etc).

mxfdemux: post error message when erroring out on unsupported essence wrapping

Add missing backslash in Android.mk

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

hlssink: do not leak playlist object

In finalize, also release the playlist

glfilter: Fix building without EGL support

This fixes the OSX build and any builds with --disable-egl. That issue
was introduced in "glfilter: rewrite transform_caps to preserve caps fields".

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

glfilter: rewrite transform_caps to preserve caps fields

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

gl/colorconvert: use the texture scaling from the gl memory

The colorconvert values were not being used at all.

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

glmemory: Fix handling of stride with alignement larger then 8

Setting a scaled factor for X coordinate is not enough as the indexer
will still think stride is shorter and will not fully skip it. Instead,
update width, so the lines are as expected. Combined with the scale, it
will hide the cropped portion.

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

dashdemux: mpdparser: properly unref gstdatetime

Avoid leaking the GstDateTime instances when summing dates

CID #1212143

dashdemux: mpdparser: avoid double free of segment timeline node

The parsing function already frees the old value (if any), avoid a double
free by not freeing it before calling the function without setting the
pointer to NULL

Coverity ID: 1212178

dashdemux: mpdparser: avoid double free

The _parse_url function already frees the previous pointer, avoid
freeing it before without setting to null or we have a double free.

Coverity ID: 1212181
Coverity ID: 1212180
Coverity ID: 1212179

schrodec: Use pool configuration

It was leaking the config, but also not using it...

CID #1212155

rsvg: Don't leak GstVideoCodecState

And avoid useless call to _get_output_state (set_output_state() already
returns it).

CID #1212160
CID #1212161

mpegtbase: Small rewrite

Setting done was useless anyway

dash: Don't leak GstActiveStream

CID #1212142

shm: Don't leak GstShmPipe

By reordering the leaking code path to before the allocation

CID #1212153

vmndec: Don't leak output state

_set_output_state returns a reference to the codec state.

CID #1212170

sdpdemux: Don't leak adapter data

CID #1212158

mpegpsmux; Fix stream leak

CID #1212164

jp2kdecimator: Don't leak various slices and arrays

CID #1212139
CID #1212140
CID #1212141

asfmux: Don't leak bytereader

By making it go through the error path that takes care of cleanups

CID #1212168

glupload: Ensure we still have a texture after upload_set_format()

gst_gl_upload_set_format() resets the upload, hence the texture.
So we need to ensure we have a texture after this call when
uploading.

glupload: Correctly update the video info from video meta

Using gst_video_info_set_format() isn't complete when updating
a video info from video meta.

glupload: Correctly update video info in upload fallback

When the upload accelerated method fails, we fallback to an upload,
but the video info was not correctly updated.

mssdemux: avoid deadlocking when erroring

It might get stuck when posting the message while
a reconfigure event is being pushed upstream

mssdemux: fix caps leak

mssdemux: handle error messages from internal source

set up a bin message handler to get notified of error messages from
the internal source element

mssdemux: Always accept the latency configure event

Upstream or internal elements are irrelevant and we should
always succeed.

mssdemux: reenable segment tracking for streams

Store the latest pushed timestamp so that mssdemux streams know
where they are in time

mssdemux: deprecate unused max-queue-size-buffers

There is no internal storage of buffers anymore

mssdemux: measure bitrate similarly to hlsdemux

new bitrate = (old bitrate + (last fragment bitrate * 3)) / 4

mssdemux: remove uridownloader in favor of internal source

Refactor mssdemux to remove uridownloader to use an internal
source element which reduces startup latency and provides smaller
buffers for better buffering management downstream

dashdemux: initialize the mutex and cond for fragment download

gl/upload: set out_tex after reset, not before

Fix a regression introduced recently with the lazy init.
It was happening when calling gst_video_gl_texture_upload_meta_upload
from an aplication. So not using gst_gl_upload_perform_with_buffer.

gl/upload: update the video info on mapping a video frame

The buffer should contain the most specific data on how the data is
formatted.  We should use this information.

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

gl/convert: clobber alpha channel when needed

Needed in the conversion from padded RGB formats to formats with an
alpha channel.  e.g. BGRx -> ABGR, etc

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

gl/upload: avoid performing color conversion when there is no need

One such example is when the buffer contains GstGLMemory in the
RGBA format

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

dvbsrc: Don't free string we're still using

CID 1213780

dashdemux: remove unused stream uridownloader

The uridownloader for the stream fragments isn't used anymore.

assrender: Do not try to add a feature to a caps features ANY

It does not makes sense and asserts

Based on 7f9aac386fd5d5921c80100889a5f212c0148a88 from gst-plugins-base.

dvbsuboverlay: Do not try to add a feature to a caps features ANY

It does not makes sense and asserts.

Based on 7f9aac386fd5d5921c80100889a5f212c0148a88 from gst-plugins-base.

gl/colorconvert: choose the right alpha component for AYUV -> RGBA

gl/upload: update the respective state when we cache textures/data

dvbsrc: drop C++ style comments

This patch builds on previous work done by
Fabrizio (Misto) Milo <mistobaan@gmail.com>

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

dvbsrc: fix up error reporting

This patch builds on previous work done by
Fabrizio (Misto) Milo <mistobaan@gmail.com>

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

gl/upload: cache the textures that we are using

gl/upload: avoid recreating the GLMemory struct for output textures

rtmpsrc: error out if we get EOS immediately without any data

It's not really right to just go EOS as if nothing was wrong.

tsdemux: remove confusing debug log line

autoconvert: factories don't need the lock

An atomic is enough, they can only be set once.

Revert "autoconvert: Handle caps query on internal srcpad"

This is not needed anymore if the gst_pad_link checks nothing.

This reverts commit a303375a3f5c12e97ca8bf50b5a93ce8652faf23.

autoconvert: No need to check anything when linking internal pads

autoconvert: Don't ignore elements accepted ANY, it is valid

autoconvert: Remove broken usage of the iterator

We're a subclass, so we can just take the object lock and iterate the list
directly.

autoconvert: Process upstream getcaps

Process it like downstream getcaps

autoconvert: Return the GList of factories, to a pointer to it.

autoconvert: subelement can only be change with stream lock held

It was already the case, now be more explicit about that, and remove useless
lock/ref/unlock/unref dances.

autoconvert: Keep a ref to the pads too

Simplifies the code

autoconvert: Remove uneeded if()

NULL is valid GList

autoconvert: Always push reconfigure upstream too

A reconfigure could trigger a change of contents instead of just reconfiguring the internal element

autoconvert: Remove unused internal parameter

gl: Fix some of the error handling

glupload: Update GstVideoInfo after the buffer is mapped

gst_video_frame_map() will store an updated video info base
on the video meta. In order to have the right stride and offset
we should update that video info accordingly.

gl: Fix memory leak

CID #1212171

hls: Remove invalid free

data does not have to be freed at all here, it's a pointer to
an arbitrary position inside the current line. Also don't reuse
the data variable for anything else, that will cause crashes
in playlists that have the I-frame playlist URI followed by
other attributes.

CID 1212127

mpegts: Fix leak

The cell extension was never added to the parent cell

CID #1212162

gl/colorconvert: preserve alpha in AYUV <-> RGBA conversions

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

gl/colorconvert: implement lazy init

gl/upload: set initted to TRUE when _init_upload succeeded

gldisplay: Use GST_STR_NULL() for printing possibly NULL strings and print all variables we use

Fixes compiler warning about set but not used platform_choice variable
among other things.

gleffects: use gl_vtable

gl/upload: fix compilation on OS X

gl/upload: fail propoerly if init fails

gl: make all GObjects inherit from GstObject

Aids memory leak debugging with GST_TRACE=mem-live

gl/upload: implement lazy init

Rename init_format to set_format

gl/upload: add get_format method

Simply retreives the format set by init_format

gl/filter: attempt to passthrough the pool if the caps are the same

Reduces the number of glbufferpool instances required for >=2
consecutive GL elements in a pipeline.

gl/filter: always initialize the upload object

gl: advertise GstGLMemory usage in the caps as capsfeatures

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

gl/examples: fix compilation for clutter examples

ClutterX11TexturePixmap is deprecated in Clutter 1.6 and we ask for
at least 1.8

gl/display: avoid endless recursion for cocoa

dashdemux: deprecate max-buffering-time property

It is not used as dashdemux doesn't have a internal buffer
for fragments anymore

dashdemux: removing old unused code

The 'catch up mode' isn't needed anymore since dashdemux
is downloading and pushing from the same thread

dashdemux: remember to set starting fragment to new fragments

dashdemux: handle a few more error cases

related to missing plugins, bad URI and a lockup on dealing with
errors from the source.

dashdemux: Only set timestamp to the first buffer of fragments

Setting it to the other parts of the fragment is slightly wrong
and misleading as it isn't accurate enough.

dashdemux: handle error messages from the sources

Set up a message handling function to be able to catch errors
from the source element and signal the cond to allow the download
loop to retry the download.

dashdemux: check for errors between downloads

To abort earlier in case of failures

dashdemux: Always accept the latency configure event

Upstream or internal elements are irrelevant and we should
always succeed.

dashdemux: measure download rate just like hlsdemux

new bitrate = (old bitrate + (last fragment bitrate * 3)) / 4