gstinfo: fix infinite loop in gst_debug_unset_threshold_for_name()

Ensure iterator is advanced. The current list iteration code only
advances the iterator (walk) if a match is found, which results
in an infinite loop when more than one entry exists in the list.

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

scripts: create-uninstalled-setup: miscellaneous fixes

Error out if required build tools (flex, bison, pkg-config)
are not present, instead of printing a message and then
continuing.

Check out submodules when fetching the repositories, so
they're already there and ready later.

Remove some 0.10 cruft.

Add INSTALL to .gitignore

tests: error out if test environment is not actually set up properly

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

configure: can use AM_SILENT_RULES unconditionally now

https://autotools.io/automake/silent.html

configure: bump automake requirement to 1.14 and autoconf to 2.69

This is only required for builds from git, people can still
build tarballs if they only have older autotools.

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

Remove INSTALL file

autotools automatically generate this, and when using different versions
for autogen.sh there will always be changes to a file tracked by git.

gstreamer-uninstalled: Update path to the GstValidate scenarios

buffer: Check return value of meta transform function in gst_buffer_copy_into()

... by printing some debug output whenever copying a GstMeta fails.

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

event: fix header formatting

tests: protection: fix leak in unit test

gst.h: include the new gstprotection.h header

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

protection: add GstProtectionMeta to support protected content

In order to support some types of protected streams (such as those
protected using DASH Common Encryption) some per-buffer information
needs to be passed between elements.

This commit adds a GstMeta type called GstProtectionMeta that allows
protection specific information to be added to a GstBuffer. An example
of its usage is qtdemux providing information to each output sample
that enables a downstream element to decrypt it.

This commit adds a utility function to select a supported protection
system from the installed Decryption elements found in the registry.
The gst_protection_select_system function that takes an array of
identifiers and searches the registry for a element of klass Decryptor that
supports one or more of the supplied identifiers. If multiple elements
are found, the one with the highest rank is selected.

This commit adds a unit test for the gst_protection_select_system
function that adds a fake Decryptor element to the registry and then
checks that it can correctly be selected by the utility function.

This commit adds a unit test for GstProtectionMeta that creates
GstProtectionMeta and adds & removes it from a buffer and performs some
simple reference count checks.

API: gst_buffer_add_protection_meta()
API: gst_buffer_get_protection_meta()
API: gst_protection_select_system()
API: gst_protection_meta_api_get_type()
API: gst_protection_meta_get_info()

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

event: add new GST_EVENT_PROTECTION

In order for a decrypter element to decrypt media protected using a
specific protection system, it first needs all the protection system
specific  information necessary (E.g. information on how to acquire
the decryption keys) for that stream.

The GST_EVENT_PROTECTION defined in this commit enables this information
to be passed from elements that extract it (e.g. qtdemux, dashdemux) to
elements that use it (E.g. a decrypter element).

API: GST_EVENT_PROTECTION
API: gst_event_new_protection()
API: gst_event_parse_protection()

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

tee: fix use of possibly-freed pad in debug statement

The gst_object_unref() in the block above may be dropping
the last ref to the pad and free the pad. Set pad pointer
to NULL here, so that we don't accidentally use a
possibly-freed pad pointer in the debug log statements
further below, and also use the tee element as log object
since that's more appropriate anyway.

Fixes valgrind warnings and crashes in tee test_stress
unit test when debug logging is enabled.

tests: info: fix unit test when run with GST_DEBUG=*:9

Only save the messages we're interested in and expecting.
When run with *:9 we might get additional TRACE level
messages from other categories and then we don't end up
with the number of messages we expect.

tests: pad: fix buffer leak in new blocking_with_probe_type_idle test

tests: pad: fix invalid memory access in debug log message

The string we put in the buffer is not NUL-terminated, so
don't try to print that via %s in a debug log message.

helpers: on OSX, MKDIR_P is install-sh -c -d

So we need to call it before cding to the bin directory.

helpers: install -D isn't portable, use $(MKDIR_P) instead.

tests: pad: test that idle probe will block

This tests add an idle probe on an idle pad from a separate thread
so that the callback is called immediatelly. This callback will sit
still and then we try to push a buffer on this same pad. It verifies
that the idle probe blocks data passing

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

pad: block data flow when idle probe is running

When idle probe runs directly from the gst_pad_add_probe() function
we need to make sure that no data flow happens as idle probe
is a blocking probe. The idle probe will prevent that any
buffer, bufferlist or serialized events and queries are not
flowing while it is running.

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

docs: clarify that return value of gst_filename_to_uri() must be freed

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

bin: undo upward state changes on children when a child fails

When a bin changes states upwards, and a child fails to change,
any child that was already switched will not be reset to its
original state, leaving its state inconsistent with the bin,
which does not change state due to the failure.

If the state change was from NULL to READY, it means that deleting
this bin will cause those children to be deleted while not in
NULL state, which is a Bad Thing. For other upward changes, it
is less of a problem, as a subsequent switch back to NULL will
cause an actual downwards change on those inconsistent elements,
albeit from the "wrong" state.

We now reset state to the original one when a child fails.

Includes unit test.

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

helpers: use $(INSTALL) to ... install the helper.

As it will create the folders and set permissions appropriately,
better than doing it manually.

helpers: Fix Makefile.am to install the completion-helper correctly.

+ The program is installed at install-exec time, we thus need
  to move it in install-exec-hook, not install-data-hook.

Revert "basesrc: fix pool leak on allocation query error path"

This reverts commit 84fdf50b2f98951a32fa14802b62621f1105cd35.

It seems the bug was fixed independently, and the merge was
automagic, yielding two extra free calls.

tests: clock: fix test clock name

Don't call the slave test clock "Master".

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

elementfactory: add ENCRYPTOR class defines

to go with DECRYPTOR.

elementfactory: add DECRYPTOR class defines

An element that performs decryption does not naturally fit within any
of the existing element factory class types. It is useful to be able
to easily get a list of all elements that support decryption so that
a union can be computed between the protection systems that have a
supported decryptor and the allowed protection systems for a particular
stream.

This commit adds a new GST_ELEMENT_FACTORY_TYPE_DECRYPTOR and its
associated string identifier "Decryptor". It also adds
GST_ELEMENT_FACTORY_TYPE_DECRYPTOR to GST_ELEMENT_FACTORY_TYPE_DECODABLE
so that uridecodebin can auto-plug a decryption element.

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

typefindelement: Use gst_event_unref() instead of gst_mini_object_unref() + casting

typefind: fix leak in gst_type_find_element_src_event()

gst_type_find_element_src_event() is supposed to consume @event but wasn't
doing so when it was handling the event itself.

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

gstvalue: reset errno before g_ascii_strtoull call

"errno" already has meaningless value before g_ascii_strtoull call.
This causes invalid error check without reset.

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

basesrc: Only set DTS to segment.start on the first buffer if subclass did not provide PTS

Otherwise we're going to set a rather arbitrary DTS of segment.start (usually
0) for live sources, which confuses synchronization if the source started
capturing at a later time. And it's especially wrong for raw media, for which
we should not set any DTS at all.

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

typefind: Run the default have-type handler after all application handlers

Otherwise the CAPS event will already be forwarded downstream and
the application has no way to intervene anymore.

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

multiqueue: Don't automatically enter the buffering state when use-buffering is set.

There is no reason I can see to set mq->buffering = TRUE when
use_buffering is set; the code here also calls update_buffering(), which
will set mq->buffering = TRUE if this is warranted because of low buffer
levels.

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

inputselector: fix cached buffer leak in chain function

gst_selector_pad_chain() was popping cached buffers out of the queue without
freeing those. Make sure we don't steal the GstBuffer as the cached buffer ref
has been passed to the pad chain function.

This can be reproduced by running the
validate.file.playback.switch_subtitle_track_while_paused.test5_mkv scenario
with Valgrind.

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

Signed-off-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

tests: Use AM_TESTS_ENVIRONMENT

Needed by the new automake test runner

bufferlist: make sure list is writable before adding or removing buffers

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

bufferlist: minor docs addition for gst_buffer_list_get()

Return buffer remains valid as long as list is valid
and buffer is not removed from list.

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

basesrc: fix pool leak on allocation query error path

It could be triggered by:

gst-launch-1.0 videotestsrc num-buffers=20 ! videcrop bottom=214748364 ! videoconvert ! autovideosink

Spotted while testing:

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

baseparse: Forward SEGMENT_DONE events immediately

There might be no more data coming afterwards, and we just drained everything
that was left to be pushed anyway.

docs: fix cross-reference to environment variables in GstInfo

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

memory: add check for writablity in resize

Add guard to gst_memory_resize() to make sure the
memory to be resized is actually writable.

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

tests: multiqueue: add test to make sure initial events go through without buffers

tests: queue: check that the initial events are sent on immediately

Add a check that makes sure stream-start, caps, and segment events
are passed on by queue without delay, i.e. even if no buffer is
sent.

pad: Print debug output from gst_pad_link_full() if preparing linking failed

Makes it easier to find linking failures in debug logs.

segment: small docs addition

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

docs: design: fix some 0.10-isms in GstSegment docs

1) segment.accum -> segment.base

2) Refer to GstSegment members as S.foo instead of
NS.foo, the event is now called a segment event
rather than newsegment event.

3) There's no more abs_rate field in GstSegment,
and there never was an abs_applied_rate field.

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

basesrc: do not leak buffer pool in error case

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

uri: Silence a compiler warning

This is a false positive for use initialized. The variable is set and
used enclosed in the safe if condition.

doc: Add gst_segment_to_running_time_full

basesrc: Fix documentation, buffer pools are unreffed and not freed

Update INSTALL to the automake 1.14 version

Automatic update of common submodule

From bc76a8b to c8fb372

win32: fix exports

segment: add gst_segment_is_equal

It beats memcmp due to the 'reserved' fields.

API: gst_segment_is_equal()

Found via, but probably not directly linked to,
https://bugzilla.gnome.org/show_bug.cgi?id=738216

win32: add new API to exports

pad: allow probes to remove the data item whilst returning PROBE_OK

Use case: we want to block the source pad of a leaky queue and
drop the buffer that causes the block. If we return PROBE_DROP
then the buffer gets dropped, but we get called again. If we
return PROBE_OK we can't easily drop the buffer. If we just
replace the item into the GstPadProbeInfo structure with NULL,
GStreamer will push a NULL buffer to the next element when we
unblock the pad probe. This patch ensures it doesn't do that.

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

element: Document when a clock is available from gst_element_get_clock()

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

pipeline: Add binding friendly gst_pipeline_get_pipeline_clock()

Also skip gst_pipeline_get_clock() and gst_pipeline_set_clock() from the
bindings as they are confused with gst_element_*_clock().

API: gst_pipeline_get_pipeline_clock()

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

basetransform: Add Since mark for new method

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

basetransform: Add a method to let subclasses cleanly update srcpad caps

API:
    gst_base_transform_update_src

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

docs: pwg: fix missing comma and 0.10-ism in code sample

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

memory: improve docs for _copy() and _share()

test: filesink: add tests for buffers with multiple memory blocks

Update test_seeking testcase to verify the render and render_list
virtual method handle buffers and buffer list containing multiple
memory blocks correctly.

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

element: Add a FIXME for 2.0 about request_new_pad VS request_pad naming

element: Fix request_new_pad introspection

Marking gst_element_request_pad as the caller of the ->request_new_pad
virtual method.

tests: filesink: add check for render_list virtual method

GstFileSink implements the render_list virtual method to render
a list of buffers. Update the test_seeking test case to also
check the render_list method implementation.

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

debugutils: nicer printing of caps features

Only print interesting caps features, don't
append (memory:SystemMemory) to all caps,
which makes them much more unwieldy and
harder to read. Also use internal function
to get caps features so that our printing
has no side effects on the caps.

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

debugutils: plot caps features

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

pad: Fix a typo in a docstring

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

introspection: Don't use g-ir-scanner cache at compile time

It pollutes user directories and we don't need to cache it

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

pad: fix outdated debug message

Buffer lists don't have groups any more in 1.0

basesrc: Flush-stop starts live task in paused

The flush-stop event should not restart the task for live sources unless
the element is playing. This was breaking seeks in pause with the rtpsrc.

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

tests: check location isn't truncated

Test covering the recent commit where location='".abc' won't get truncated
to '.ab' anymore

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

gstvalue: only unwrap string delimited with "

Don't unwrap strings that start but don't finish with a double quote. If a
string is delimited by two quotes we unescape them and any special characters
in the middle (like \" or \\). If the first character or the last character
aren't a quote we assume it's part of an unescaped string.

Moved some deserialize_string unit tests because we don't try to unwrap strings
missing that second quote anymore.

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

parse: check before truncating strings

Don't truncate the last character of a string if it isn't necessary.

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

bus: Add guards against invalid arguments to set_flushing() and poll()

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

baseparse: only post 'no valid frames' error if buffers were received

Otherwise baseparse will consider empty streams to be an error while
an empty stream is a valid scenario. With this patch, errors would
only be emitted if the parser received data but wasn't able to
produce any output from it.

This change is only for push-mode operation as in pull mode an
empty file can be considered an error for the one driving the
pipeline

Includes a unit test for it

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

tee: Add allow-not-linked property

This property avoids not linked error when all the pads are unlinked
or when there are no source pads. This is useful in dynamic pipelines
where it can happen that for a short time there are no pads at all or
all downstream pads are not linked yet.

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

docs: Fix typos

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

pluginloader: Fix typos

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

output-selector: add drain handling

Release the latest buffer, if any, and then just let
the drain be pushed downstream

Revert "Fix distcheck"

This reverts commit 56dd2d89c4eac460cbc37e2a51c1dd9e792999e8.

Installing completions to a custom prefix is now fixed.

helpers: remove completion-helper on uninstall

+ And add it to CLEANFILES

completions: remove last unnamespaced symbols.

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

completions: remove deprecated shell syntax.

https://bugzilla.gnome.org/show_bug.cgi?id=744877#c21

completions: prefix shell functions with _gst

+ To make it more difficult for them to conflict in the
  global namespace.

https://bugzilla.gnome.org/show_bug.cgi?id=744877#c21

bash-completion: Respect the prefix

Don't try and install the bash helpers outside the defined prefix.

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

input-selector: Rename _activate_sinkpad to _get_active_sinkpad

Removes the now unused 'pad' parameter and renames the function
to something more appropriate.

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

input-selector: Remove pad's 'active' field

This is now never read.

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

input-selector: Use segment-presence for running_time check

When determining whether the running_time of a pad can be
calculated, check if the segment is in TIME format instead
of using the 'active' field.

Since the latter is set through *any* activity, it's not a
reliable indicator of segment presence.

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

input-selector: Remove 'blocked' flag

With the disappearance of the 'block' signal, this
flag cannot be set to TRUE.

gst_input_selector_wait disappears as it never waits
and just returns self->flushing.

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

input-selector: Remove obsolete 'block' signal

This signal blocks the input-selector with no means of unblocking
other than a state change back to READY. It seems this signal was
part of an old way of synchronously switching the selector,
together with the already-removed 'switch' signal.

Removing the signal is safe, as attempting to use it could only
end in deadlocks. Attempting to emit an unknown signal just causes
g_criticals.

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

input-selector: Fix waiting on EOS

This apparently got broken by bc1ec4e. Since self->blocked is always
FALSE, gst_input_selector_wait never actually waits.

Using (!self->eos || self->blocked) && ... as the loop condition would
be incorrect as well, because then the other call to the function in
_chain would block until EOS, so the functions cannot be merged trivially.

Since blocking is obsolete, gst_input_selector_wait will get removed anyway.
As such, just inline the loop.

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

tests: input-selector: new tests for EOS handling

3 new tests:

1) Tests that a stream that is empty (just an EOS event)
   on inactive pad doesn't get through and tamper
   with the active pad that still has data

2) Tests that a stream that is shorter than the active one
   (pushes EOS earlier) doesn't has its EOS pushed

3) Tests that switching to an inactive stream that has received
   EOS will make input-selector push EOS

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

tests: selector: remove weird semicolons at the end of test functions

Even though it works, it is not needed and seems more natural
to not have semicolons at the end of function declarations

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

queue2: Process SEEKING query

Add QUERY_SEEKING handling to queue2, so RTMP live streams become
seekable when a queue2 in download or ringbuffer mode is inserted:

rtmpsrc ! queue2 ! flvdemux

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

check: Fix uninitialized variable compiler warning with gcc

check_run.c: In function 'sig_handler':
check_run.c:127:13: warning: 'child_sig' may be used uninitialized in this function [-Wmaybe-uninitialized]
             killpg(group_pid, child_sig);
             ^
check_run.c:130:31: warning: 'idx' may be used uninitialized in this function [-Wmaybe-uninitialized]
             sigaction(sig_nr, &old_action[idx], NULL);
                               ^