caps: Don't leak features on error cases

If we fail to parse fields, we would end up leaking the features we
parsed just before

CID #1212152

basetransform: Correctly reset configuration

When pool can't we use, and we fall back to default pool, we need to
correctly reset that pool configuration.

nettimeprovider: Use non-freed variable

address is only used temporarily. Use the proper variable instead.

CID #1212189

tests: multiqueue: test to check queue overrun with pts=none

Checks if buffers with pts=none can break the queue time size limit
and allow more buffers than expected

downloadbuffer: Fix 32bit build

format '%lli' expects argument of type 'long long int', but argument 8 has type 'gsize'

pool-nego: Correctly reset the configuration

When pool cannot be used, correctly reset the configuration before
configuration a default pool.

pool-nego: Retry setting configuration with modified config

Buffer pool set_config() may return FALSE if requested configuration needed
small changes. Reget the config and try setting it again (validating the
changes first). This ensure we have a configured pool if possible.

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

bufferpool: Add an helper to validate config

When we call gst_buffer_pool_set_config() the pool may return FALSE and
slightly change the parameters. This helper is useful to do the minial required
validation before accepting the modified configuration.

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

bufferpool: Update the configure even if set_config() returned false

According to the documentation, when set_config() return false, it should be
possible to read the modified version of the config. This patch fixes the
implementation so it is now according to the documentation.

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

bufferpool: Add support for reconfiguring a pool

If a pool config is being configured again, check if the configuration have changed.
If not, skip that step. Finally, if the pool is active, try deactivating it.

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

value: Add support for GstAllocationParams comparision

This is useful to compare buffer pool configuaration.

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

value: Add support for GObject comparising in structures

This is useful to allow comparing pool configuration where a GstAllocator
is set.

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

plugin: fix case where gst_plugin_load_file() didn't set the error on failure

sparsefile: add some Since markers to docs

sparsefile: sprinkle G_BEGIN_DECLS / G_END_DECLS

for c++, and remove outdated comment, and add
new unit test to .gitignore.

downloadbuffer: small cleanups

downloadbuffer: update docs

win32: update def

Add new downloadbuffer element

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

pads: update docs for request pads

We would like to encourage the use of gst_element_request_pad()

check: use _exit() instead of exit() in fail_unless() so we exit immediately

exit() will call atexit handlers, which may try to
clean up things or wait for things to get cleaned up,
which we don't want or need. We just want to stop
and let the parent know about the failure as quickly
as possible in case fork() is used.

Fixes timeouts on assert failures in checks where
an exit handler waits for things to stop, but they
don't stop because they haven't been shut down,
and they haven't been shut down because there's no
simple way to do so on failures.

http://sourceforge.net/p/check/patches/50/

value: init flag mask more correctly

filesrc: g_memmove() is deprecated

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

Back to development

Release 1.3.1

Update .po files

po: Update translations

docs: add new device probing API to docs table of contents

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

doc: Add GstDevice* to gstreamer-sections.txt

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

Automatic update of common submodule

From bcb1518 to 211fa5f

basesink: Always render prepared buffer

Currently, if prepare() takes too much time, we skip the call to render().
The side effect of this, is that we endup starving the render(). The solution
in this patch is to always render frames that are on time before prepare() is
executed. This will maximize the number of frames we display and smoothly
degrade the rendering performance.

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

scripts/git-version.sh: add more modules

Add more git repositories to check (so git-version.sh is consistent with
gst-uninstalled) and display the date of the last commit, which is more valuable
information than the last commit's hash.

bin: Always first post the state-changed message for PAUSED->READY before posting any pending EOS message

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

basesink: Add test for checking that EOS always comes after the state change to PLAYING

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

bufferpool: fix log message of buffer pointer

scripts/git-version.sh: remove unused variable

scripts/git-version.sh: fix mistaken comments

multiqueue: avoid signaling overrun on the first segment

When the first segment has position != 0 and position > max-size-time
it will immediatelly cause the multiqueue to signal overrun.

This can happen easily with adaptive streams when switching bitrates
and starting a new group. The segment for this new group will have
a position that is much greater than 0 and will lead to this issue.
This is particularly harmful when the adaptive stream uses mpegts
that doesn't emit no-more-pads and it might happen that only one
of the stream pads was added when the multiqueue overruns and gets
the group ready for exposing. So the user will only get audio or
video.

The solution is to fallback to the sink segment while the source pad
has no segment.

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

docs: enhancements to porting guide documentation

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

query: Add boolean to URI query to specify if a redirect is permanent or not

funnel: Check if the last pad was set

If no data is coming but only EOS is sent from all of the sinkpad, it is not
forwarding the EOS.

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

docs: add docs for various GstPad macros

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

Revert "docs: add docs for various GstPad macros"

This reverts commit d17438d5fd321daec4adbeb28a8fb5d5e07298dc.

This commit featured the wrong author, sorry.

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.

buffer: Only set TAG_MEMORY if the memory has been replaced

Currently we set TAG_MEMORY as soon a resize changes the size of one
of the memory. This has the side effect that buffer pool cannot know if
the memory have simply been resized, or if the memorys has been replaced.
This make it hard to actually implement _reset(). Instead, only set the
TAG_MEMORY if one or more memory has been replaced, and do a light
sanity check of the size.

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

docs: add docs for various GstPad macros

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

bin: When going to READY make sure to always deactivate pads

We might not have reached PAUSED yet because of an async error,
but nonetheless we want to make sure that the pads are always
deactivated in READY state.

bin: Don't left-shift into the sign bit, the result is undefined

value: Use an unsigned 64 bit integer as a mask

We shift the mask to the right later and shifting the result
of shifting over the sign bit is undefined.

basesrc: Make sure to always hold the LIVE_LOCK when going to the flushing label

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

funnel: Handle end of stream event on sink pad

Handle end of stream events on sink pad. Check all the sink pad
has received eos before forwarding to source pad.

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

gstvalue: Prevent division or modulo by zero

The step can end up being zero if the underlying value isn't a valid
range GValue.

In those cases, return FALSE.

We don't use g_return*_if_fail since it will already have been triggered
by the above-mentionned _get_step() functions.

CID #1037132

libs: g-ir-scanner: do not hardcode libtool path

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

memory/buffer: Initialise GstMapInfo to zeroes if mapping fails

This should allow for more meaningful errors. Dereferencing NULL
is more useful information than dereferencing a random address
happened to be on the stack.

preset: Unref return value of gst_child_proxy_get_child_by_index() instead of leaking it

preset: Automatic code style fixes

queue2: fix event/preroll deadlock differently

The qlock is released between popping a buffer from the queue
and pushing it. When this buffer causes the sink to wait in
preroll, this lets a query see that the queue is empty, and
push the query then wait for it to be serviced. However, this
will not be done till after peroll, and this will thus block.
If upstream was waiting on buffering to reach 100% before
switching to PLAYING, a deadlock would ensue.

This had been fixed recently by failing queries when the
queue2 was buffering, but this happens to break some other
case (playbin on a local http server and matroska), while
this patch works for both.

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

check: Fix exported symbol name

it's _template and not _templ

pad: Add missing space in debug output

check: Add new API to set up pads from non-static pad templates

collectpads: Fix memory leak in unit test

testclock: replace newly-added GstTestClockIDList structure with a simple GList

Keep it simple. Likely also makes things easier for bindings,
and efficiency clearly has not been a consideration given how
the existing code handled these lists.

docs: testclock: fix up Since markers

testclock: add back gst_test_clock_wait_for_pending_id_count()

.. but deprecate it. ABI stability and all that.
It's a dangerous and racy function to use.

testclock: remove unused variable

Fixes compiler warning.

testclock: add support for waiting and releasing multiple GstClockIDs

In order to be deterministic, multiple waiting GstClockIDs needs to be
released at the same time, or else one can get into the situation that
the one being released first can add itself back again before the next
one waiting is released.

Test added for new API and old tests rewritten to comply.

pad: don't access unowned and possibly already freed event

Don't print the name of the event when ownership is given away.

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

inspect: print structure values of properties

input-selector: Hold pad object lock when changing tags

Avoid spurious crashes when tags are retrieved just as
new ones arrive.

gstbuffer: Fix range iteration

We want to iterate over items idx to idx + length

We use the len variable as the corrected number of memory to iterate
and then properly go over all items.

Fixes the issue where specifying any idx different from 0 had no effect

Spotted by clang static analyzer

structure: error out when trying to fixate a fraction near an invalid target

event: Update running time in QoS based on the pad offsets

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

pad: Apply pad offsets on all events, not just segment events

event: Add running-time-offset field to all events

Events passing through #GstPads that have a running time
offset set via gst_pad_set_offset() will get their offset
adjusted according to the pad's offset.

If the event contains any information that related to the
running time, this information will need to be updated
before usage with this offset.

utils: avoid dividing by zero when multiplying y/z by 0/x

The gcd of 0/x is 0, and this is then used as a denominator.

multiqueue: And actually run the other tests again

multiqueue: Wake up the queues if limits are changing in a way that would unblock the queue

multiqueue: Add test for checking if pads are waked up when limits are changed

baseparse: Fix memory leak

Queued frames were not released after being pushed, this
caused a leak of the GstBaseParseFrame structure.

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

queue2: use g_strerror() instead of strerror()

Need UTF-8 encoding.

Revert "check: only call setpgid on valid child PIDs"

This reverts commit b9313afc75b68d986e473b76b55543456857912b.

This should be fixed in upstream libcheck instead. We want
to keep diff of our local copy to upstream libcheck
to a minimum.

queue2: warn if we can't remove our temporary file

It's not fatal though, so do not error out.

Coverity 1037121

check: only call setpgid on valid child PIDs

Coverity 206186

filesrc: no need for a translated message for impossible error cases

The message is too technical anyway, the default message works
just fine here as well.

filesrc: catch failure to seek back to zero after seek test

This should never happen theoretically, but since a transient
failure would get us to silently read wrong data, it's worth
erroring out. And it silence this:

Coverity 206034

parse: Don't dist the bison and flex generated headers

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

debugutils: Handle caps field values being NULL

GST_DEBUG_BIN_TO_DOT_FILE() would cause a segfault whenever it encountered an
element's caps that had a field value being NULL. Such fields are successfully
handled e.g. by GST_*_OBJECT(), and with this patch so does
GST_DEBUG_BIN_TO_DOT_FILE(). Even if string fields with a NULL value are
not supposed to be valid in caps, such caps can be created.

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

baseparse: Remove always-true-checks

a gsize is guaranteed to be positive on all systems since it's an
unsigned value.

CID #1037147

adapter: Remove always-true-checks

a gsize is guaranteed to be positive on all systems since it's an
unsigned value.

CID #1037145
CID #1037146

gstvalue: Prevent division or modulo by zero

The step can end up being zero if the underlying value isn't a valid
range GValue.

In those cases, return FALSE.

We don't use g_return*_if_fail since it will already have been triggered
by the above-mentionned _get_step() functions.

Spotted by Coverity.

gstvalue: Fix comparison of int/int64 range

Checking step three times seems unnecessary.
A similar bug was fixed for double range in
commit 3ea6b04c10b10fde9d62190068f274b940edef07

inputselector: fix build with debug cached buffers enabled

gstinputselector.c:818:5: error: format not a string literal
and no format arguments [-Werror=format-security]

baseparse: Make sure to set the DISCONT flag on the first buffer of each GOP in reverse playback mode

basesrc: removing duplicated inner if

The inner if replicates the same code of the outer and is useless
as flag_segment will always be true.

Found by coverity.

gst-launch.1: Playbin2 is dead, long live playbin

Looks like that was the last remaining mention in core ...

gst-inspect: Add missing \n in output

scripts: gst-uninstalled: gst-plugins-gl libs got merged into -bad

pad: Include event type in debug output when delaying a sticky event because of not-linked

baseparse: Fix splitting and reversing of GOPs in reverse playback mode

We iterate the current discont group backwards and push each GOP forwards,
starting from the last one. However if the first buffer in the current
discont group is a keyframe, we will keep it around until next time,
which is far from ideal. Just push it.

launch: place the deep-notify on the right pipeline

If the toplevel bin is not not a pipeline, we place the bin in a
pipeline. Also make sure that we connect to the deep-notify of this new
pipeline because we will g_signal_handler_disconnect() from it later.