baseparse: fix minor memory leak

aacparse: Add function for the baseparse subclass to push buffers downstream

Also handle the case gracefully where the subclass decides to drop
the first buffers and has no caps set yet. It's still required to
have valid caps set when the first buffer should be passed downstream.

baseparse: Fix seek event leaking

aacparse: ADIF: do not send bogus timestamps, leave to downstream (decoder)

aacparse: fix sample rate extraction from codec data

In one case we extracted the sample rate index from the codec data
and saved it as sample rate rather than getting the real sample
rate from the table. Fix that, and also make sure we don't access
non-existant table entries by adding a small helper function that
guards against out-of-bounds access in case of invalid input data.

aacparse, amrparse: remove bogus gst_pad_fixate_caps() calls

baseparse: propagate return value of GstBaseParse::set_sink_caps()

gst_base_parse_sink_setcaps() presumably should fail if the subclass
returns FALSE from its ::set_sink_caps() function.

baseparse: don't try to GST_LOG an already-freed caps string

The proper way to log caps is via GST_PTR_FORMAT anyway.

aacparse: set channels and rate on output caps, and keep codec_data

Create output caps from input caps, so we maintain any fields we
might get on the input caps, such as codec_data or rate and channels.
Set channels and rate on the output caps if we don't have input caps
or they don't contain such fields. We do this partly because we can,
but also because some muxers need this information. Tagreadbin will
also be happy about this.

baseparse: fix debug category

baseparse: fix (regression in) newsegment handling

(aacparse, amrparse, flacparse).  Fixes #580133.

baseparse: Fix slightly broken buffer-in-segment check (aacparse, amrparse, flacparse)

baseparse: Fix push mode seeking (aacparse, amrparse)

Sending the flush-start event forward before taking the stream lock actually
works, in contrast to deadlocking in downstream preroll_wait (hunk 1).

After that we get the chain function being stuck in a busy loop. This is fixed
by updating the minimum frame size inside the synchronization loop because the
subclass asks for more data in this way (hunk 2).

Finally, this leads to a very probable crash because the subclass can find a
valid frame with a size greater than the currently available data in the
adapter. This makes the subsequent gst_adapter_take_buffer call return NULL,
which is not expected (hunk 3).

baseparse: Delay newsegment as long as possible.

If newsegment is sent (too) early, caps may not yet be fixed/set,
and downstream may not have been linked.

aacparse: Fix busyloop when seeking. Fixes #575388

The problem is that after a discont, set_min_frame_size(1024) is called when
detect_stream returns FALSE. However, detect_stream calls check_adts_frame
which sets the frame size on its own to something larger than 1024. This is the
same situation as in the beginning, so the base class ends up calling
check_valid_frame in an endless loop.

aacparse: Refactor check_valid_frame to expose broken code

Just moving code around and removing an unhelpful/misleading comment.

baseparse: revert last change and properly fix

Baseparse internaly breaks the semantics of a _chain function by calling it with
buffer==NULL. The reson I belived it was okay to remove it was that there is
also an unchecked access to buffer later in _chain. Actually that code is wrong,
as it most probably wants to set discont on the outgoing buffer.

baseparse: remove checks for buffer==NULL

Accordifn to docs for GstPadChainFunction buffer cannot be NULL. If we would
leave the check, we would also need more such check below.

aacparse: Fix license specified in plugin details.

Fix the return value of the default parse_frame function.

Fix the return value of the default parse_frame function in both
copies of GstBaseParse

Log aac details found in codec_data.

gst/aacparse/gstaacparse.c: Don't autoplug aacparse until it works.

Original commit message from CVS:
* gst/aacparse/gstaacparse.c: (plugin_init):
Don't autoplug aacparse until it works.

tests/check/: Add unit tests for new parsers.

Original commit message from CVS:
* tests/check/Makefile.am:
* tests/check/elements/aacparse.c:
* tests/check/elements/amrparse.c:
Add unit tests for new parsers.

gst/: Fix baseparse type name.

Original commit message from CVS:
* gst/aacparse/gstbaseparse.c:
* gst/amrparse/gstbaseparse.c:
Fix baseparse type name.

Add two new baseparse based parsers (aac and amr) from Bug #518857.

Original commit message from CVS:
* configure.ac:
* gst/aacparse/Makefile.am:
* gst/aacparse/gstaacparse.c:
* gst/aacparse/gstaacparse.h:
* gst/aacparse/gstbaseparse.c:
* gst/aacparse/gstbaseparse.h:
* gst/amrparse/Makefile.am:
* gst/amrparse/gstamrparse.c:
* gst/amrparse/gstamrparse.h:
* gst/amrparse/gstbaseparse.c:
* gst/amrparse/gstbaseparse.h:
Add two new baseparse based parsers (aac and amr) from Bug #518857.

jitterbuffer: Make src_query MT-safe

It is possible that the element might be going down while the event arrives

jpegdec: Unref event if the parent element disappeared

jitterbuffer: Unref event if the parent element disappeared

jpegdec: Make upstream events MT-safe

jitterbuffer: Make upstream events MT-safe

rtp: Unref events if the parent element disappeared

rtpmanager: fix pad callbacks so they handle when parent goes away

1) We need to lock and get a strong ref to the parent, if still there.
2) If it has gone away, we need to handle that gracefully.

This is necessary in order to safely modify a running pipeline. Has been
observed when a streaming thread is doing a buffer_alloc() while an
application thread sends an event on a pad further downstream, and from
within a pad probe (holding STREAM_LOCK) carries out the pipeline plumbing
while the streaming thread has its buffer_alloc() in progress.

rtpsession: make iterate_internal_links MT-safe

Revert "Pulsesink: Allow chunks up to bufsize instead of segsize"

This reverts commit 1e2c1467ae042a3c6bb1a6bc0c07aeff13ec5edb.

The commit causes pulsesink to ignore the latency-time baseaudiosink property.

rtpspeexpay: Do not transmitt samples with GAP flag

If we get GAP samples, there is no need to transmitt it.
In some situations, microphone is muted, we can drop net traffick
usage to ~1 kbit/s. Without patch it will stay ~20 kbit/s

speexenc: Use speex intern silence detection

Speex has build in silence detection. If speex_encode_int returns 0,
than there is silence and sample do not need to be transmitted.
This work only if vbr=1 and dtx=1 optionas are enabled.
So if we get 0, we add GAP flag to the sample.

rtspsrc: handle * control correctly

Parse session control attributes when no media control attribute is
present. Threat * control attributes as an empty string, just like the
spec says.

Fixes #646800

matroskamux: Add support for A-Law and µ-Law

Fixes bug #646567.

jack: Fix build with jack 0.120.1

9544622674c0d0a3147a9b51145159b02eec68e9 checked
for 0.120.2 and later, but the deprecation was introduced in
0.120.1

docs: fix docuemntation warnings (and reindent)

videomixer: update orc dist files

Automatic update of common submodule

From 1ccbe09 to c3cafe1

pulsesink: Always call pa_stream_new_with_proplist()

pa_stream_new_with_proplist() can take a NULL proplist, so we don't need
to concern ourselves with whether it's NULL or not.

rtspsrc: perform post-flush state tricks downstream to upstream

... so downstream is set when upstream resumes data flow.

rtspsrc: distribute new base_time to manager children following flush seek

... by forcing a state changed to PLAYING, which should otherwise be a
no-op as elements should already be in that state.

In particular, jitterbuffer needs new base_time as soon as possible to perform
proper timing (e.g. eos timeout handling) and can't wait for the new base_time
that will be distributed when the whole pipeline returns to PLAYING.

See bug #646397.

Revert "jitterbuffer: reset element base_time upon flush"

This reverts commit f84b8a69cba9c538f5546869cb4ef454ad5efb9d.

Fixes bug #646397.

flv: Specify the only possible stream-format for h264 in the pad templates.

qtdemux: Check for invalid (empty) classification info entity strings

Otherwise the classification string can be empty and gst_tag_list_add() will
complain or have a \0 in the first four bytes, which is wrong too.

qtdemux: Year 0 is not a valid year for GDate and the proleptic gregorian calendar

flacenc: Add support for writing METADATA_BLOCK_PICTURE blocks for GST_TAG_IMAGE and GST_TAG_PREVIEW_IMAGE

videomixer[2]: Use orc_memset() instead of memset()

videomixer: Add transparent background option for alpha channel formats

videomixer2: Add transparent background option for alpha channel formats

This option allows the videomixer2 element to output a valid alpha
channel when the inputs contain a valid alpha channel. This allows
mixing to occur in multiple stages serially.

The following pipeline shows an example of such a pipeline:

gst-launch videotestsrc background-color=0x000000 pattern=ball ! video/x-raw-yuv,format=\(fourcc\)AYUV ! videomixer2 background=transparent name=mix1 ! videomixer2 name=mix2 ! ffmpegcolorspace ! autovideosink  videotestsrc ! video/x-raw-yuv,format=\(fourcc\)AYUV ! mix2.

The first videotestsrc in this pipeline creates a moving ball on a
transparent background. It is then passed to the first videomixer2.
Previously, this videomixer2 would have forced the alpha channel to
1.0 and given a background of checker, black, or white to the
stream. With this patch, however, you can now specify the background
as transparent, and the alpha channel of the input will be
preserved. This allows for further mixing downstream, as is shown in
the above pipeline where the a second videomixer2 is used to mix in a
background of an smpte videotestsrc. So the result is a ball hovering
over the smpte test source. This could, of course, have been
accomplished with a single mixer element, but staged mixing is useful
when it is not convenient to mix all video at once (e.g. a pipeline
where a foreground and background bin exist and are mixed at the final
output, but the foreground bin needs an internal mixer to create
transitions between clips).

Fixes bug #639994.

pulsesink: also uncork during EOS waiting (and after EOS is rendered)

Pulsesink was recently changed to defer uncorking until there is data
to write. This condition will however never occur when EOS in being
rendered (since that marks the end of data). Changing to PAUSED state
while EOS is being waited on results in a hang: pausing corks the
stream, which will never be undone since there is no more data when
going back to PLAYING. If pulsesink is the clock provider, deadlock
ensues since time doesn't continue in corked state and the clock id
for EOS wait never fires.

Fixes #645961.

rtpbin: Don't try to request the same request pad twice

flacdec: fix issues with large metadata blocks when streaming unframed flac

Parse metadata blocks when handling unparsed flac in push mode. This
works around a bunch of issues with the flac decoder when handling
metadata blocks that are larger than the max. flac framesize, which
coverart blocks often are. We need to have all the data for these
blocks available when we pass data to libflac.

http://gstreamer-devel.966125.n4.nabble.com/Flac-files-that-will-playback-but-not-stream-td3338198.html#a3395276

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

flvdemux: Do not build an index if upstream is not seekable

An index is not useful if upstream cannot handle seeks and building it
for infinite files, for instance FLV streams, results in a memory leak.

v4l2: new v4l2radio element to control analog radio devices

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

Automatic update of common submodule

From 193b717 to 1ccbe09

Automatic update of common submodule

From b77e2bf to 193b717

cairo: fix the name of the *-marshall.list file to unbreak make distcheck

Automatic update of common submodule

From d8814b6 to b77e2bf

Automatic update of common submodule

From 6aaa286 to d8814b6

spectrum: refactor processing loop for block based operation

Previously the chain function was working sample frame based. In each cycle it
was checking if it is time to run a fft or if it is time to send a message.
Now we changed the data transform functions to work on a block of data and
calculate the max length until either {end-of-data, do-fft, do-msg}. This allows
us also to avoid the duplicated code for the single and multi-channel case (as
the transformers have the same signature now).

jack: unbreak the build for jack2 users

Jack2 (versions 1.X.X) does only have that API in svn. Limmit the use of the new
API for jack1 versions.

Automatic update of common submodule

From 6aec6b9 to 6aaa286

spectrum: fix the error accumulation and frames_todo handling

Even though we wrap around the accumulated second, we still need to add the
error in the same cycle. Increase the todo in the same conditional as afterwards
the accumulated error will be below one second.

spectrum: fix broken code resulting for a wrong splitup of changes

spectrum: simplify the have_interval calculation

Move some of the conditions to the places where the dependent variables change.

spectrum: use local var for input_data function

Avoid dereferencing the input_data from the instance from within an inner loop.

speexdec: Get and use streamheader from the caps if possible

This allows playback of streams where the streamheader buffers
were dropped from the stream for some reason.

flvmux: use running time for synchronization

Fixes #432612.

matroskamux: use running time for synchronization

Fixes #432612.

avimux: use running time for synchronization

See bug #432612.

configure.ac: redundant uses of AC_MSG_RESULT()

cleaned the redundant uses of AC_MSG_RESULT() in configure.ac

autogen: wingo signed comment

jackaudiosink: Fix typo from 9544622674c0d0a3147a9b51145159b02eec68e9

matroska: Mark tag mapping tables as static const

matroskamux: Use ARTIST instead of AUTHOR for GST_TAG_ARTIST

matroskademux: Use ARTIST Matroska tag instead of AUTHOR for GST_TAG_ARTIST

AUTHOR only existed in an old version of the spec and ARTIST is
the new replacement for this. We are still reading both to still
be compatible with old files.

Fixes bug #644875.

tests: enable more formats in videofilter unit test, check more resolutions

videoflip: Fix buffer overflow bug for odd resolutions and Y422 colorspaces

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

speexdec: silence warning message when appropriate

If we did not know how many frames to expect, then we get an unexpected
end of stream when trying to decode more frames that are there, if there
are leftover bits to pad to the next byte

videoflip: Add support for YUY2, UVYV and YVYU colorspaces

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

tests: in videofilter unit test also check with 'odd' widths and heights

And only use one test suite.

speexdec: Always process the number of frames per packet as specified in the header

Looking at the remaining bits in the bitstream after decoding a
single frame can't be used as loop condition. The remaining
bits might not give a complete frame and the speex decoder will
then output nothing but access uninitialized memory, which leads
to valgrind warnings.

Fixes bug #644669.

matroskamux: return TRUE from sink pad event function for tag events, which are handled

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

pulsesink: Better fix for deadlock on failed connect

This reverts the previous fix that would cause a double-unlock when the
stream connect failed.

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

pulsesink: Fix deadlock if connecting to PA fails

Commit dd4ec22e introduced a deadlock in the failure path while trying
to connect to PulseAudio. This makes sure we drop the lock on the
resource mutex to avoid this.

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

tests: order state-test blacklist and add jack elements

Jack audio src/sink elements recently got moved from bad and should be excluded
from the test (like the other device specific source and sinks).

Fixes #644288

dvdemux: Chain up to the parent class' ::send_event for non-seek events

dvdemux: Fix refcount issues with the seek event

Fixes bug #642963.

docs: fix pulsesink gtk-doc markup

jack: fix build against jack 0.120.2

jack_port_get_total_latency() has been deprecated in favor of
jack_port_get_latency_range().

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

spectrum: more comments and tune and logging

spectrum: avoid unneccesary extra fft runs

Before it was possible that we run an extra fft when the time for sending a new
message is due. Only do this if we have not run the fft for the interval at all.

spectrum: only scale the vectors that we are processing

Phase is not produced by default, so lets not scale it unconditionally to save a
few cycles.

spectrum: put number of channels to instance variable

When freeing data the format might have changed. Thus we need to remember for
which format we allocated memory.

spectrum: update doc review stamp

spectrum: use function pointers for data readers

Don't check the format for each sample frame to read. We can make that decission
in _setup already. This is still not ideal as we call the function per frame.
Ideally we determine how many samples we can copy and have a loop in the input
reader. As an alternative we might also consider to use the fft variants for the
various formats and not convert to float for all cases - we would still need to
mix or deinterleave though.