pbutils: discoverer: Always set the pipeline back to NULL after an error

Otherwize the pipeline would be in an wrong state and on the next
iteration any kind of error could happen

Everytime an error happens in a pipeline the application has to set the
pipeline back to NULL instead of READY.

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

decodebin: add missing 'time' word to debug message

It prints the buffers, bytes and time limits, but 'time' was missing
from the string.

playbin: Pass through NO_PREROLL state change returns

Fixes playback of live pipelines.

uridecodebin: Pass through NO_PREROLL state change returns

Fixes playback of live pipelines.

playbin: fix 'attempt to unlock mutex that was not locked' in error code path

Fixes playbin unit test with latest GLib.

videoencoder: Don't delay set_format

This prevent implementing allocation query, as the format need to be
known in order to determin the size and number of buffers needed.

Note: This may lead to few regressions that will need fixing

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

decodebin: Don't unref caps for which we don't own a reference... get one first

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

playbin: Go asynchronously from READY to PAUSED

We now add all our elements to uridecodebin *after*
GstBin::change_state(READY->PAUSED), so we need to post async-start
and async-done messages ourselves if we want to work async.

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

uridecodebin: Go asynchronously from READY to PAUSED

We now add all our elements to uridecodebin *after*
GstBin::change_state(READY->PAUSED), so we need to post async-start
and async-done messages ourselves if we want to work async.

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

discoverer: Pretty-print topology tags

Call the code used in properties for topology tags too.
Side-effect achieved: more tags printed, buffers (e.g. images) shortened.

discoverer: Fix code style a bit

if (...)
  one_line;
else if (...) {
  many_lines;
} else
  one_line;

looks a bit confusing.

discoverer: prettier image tag printing

Rather than dumping the serialized sample value, the code now
prints the number of bytes in the buffer, then the caps in a
human-readable format.

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

audiodecoder: Handle CAPS events immediately instead of delaying them

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

videodecoder: Handle CAPS events immediately instead of delaying them

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

playbin: Fix unit test for last change

It will successfully asynchronously go to PAUSED now and
later fail.

uridecodebin: Create new sources after chaining up to the parent class

Otherwise we start the new sources already before the parent class
got ready to start.

playbin: Create new sources after chaining up to the parent class

Otherwise we start the new sources already before the parent class
got ready to start.

playbin-complex: Change template name from %d to the more common %u

decodebin: Link Parser/Converter directly and already connect to pad-added and other signals before setting elements to PAUSED

otherwise we're going to
a) start Parser/Converter before they are linked to their capsfilter,
   breaking their negotiation of a proper stream format
b) start demuxers without having connected to their pad-added signals. We
   miss pads and in the worst case don't link any pads at all

decodebin: Send sticky events to the new element after setting it to PAUSED

... and if this fails for whatever reason we skip the element and instead
try with the next element. This allows us to handle elements that fail
when setting caps on them by just skipping to the next alternative element.

decodebin: Only link elements further after setting them to PAUSED

They might fail to go to PAUSED, and when connecting them further
we might already expose their srcpads on decodebin if we're unlucky.
This prevents us to handle failures going to PAUSED gracefully.

decodebin: Remove ERROR message filter after we set the element to PAUSED

This allows us to catch more errors gracefully and switch to an alternative
element instead.

decodebin: Only continue autoplugging once the pad has final caps

If the caps query returned us fixed caps this doesn't mean yet
that these caps are actually complete (fields might be missing).

It allows to do us some decisions, but the selection of the next
element should be delayed as only complete caps allow proper selection
of the next element.

decodebin: Consider the caps after the capsfilter after parsers for autoplugging

Otherwise we might try to continue autoplugging e.g. for a specific
stream-format although the parser could convert to something else, thus giving
us potentially less options for decoders.

pbutils: fix missing plugin description for missing elements

CID: 1226445

Back to development

Release 1.4.0

Update .po files

appsrc: Fix memory leak with callback notify not being called in dispose

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

po: Update translations

encoding-profile: Add example for using encoder presets with profiles

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

encoding-profile: Fix typos and old API in docs

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

xvimagesink: fix property description string

Spotted by Josep Torra.

po: update POTFILES

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

playsink: Fix filter property getter

The switch-case set was incomplete.

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

Release 1.3.91

Update .po files

libvisual: Remove < 0.4 support

And remove the version guards that went along with it

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

dmabuf: Ensure _get_fd() works even for shared memory

Fixes regression introduced by:

commit b60888fd4bcacd42bb4e27fa938272d6e72c5c32
Author: Michael Olbrich <m.olbrich@pengutronix.de>
Date:   Tue May 20 11:18:56 2014 +0200

    dmabuf: share the mapping with shared copies of the memory

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

po: Update translations

sdp: add gstmikey.h to sdp.h

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

riff: Print invalid fourcc in error message in hex

Previously this was printed as characters which caused later processing
of the error message to sometimes warn about non-UTF-8 characters.

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

videodecoder: parse any source data that is still available.

Fix gst_video_decoder_parse_available() to really parse any pending
source data that is still available in the adapter. This is a memory
optimization to avoid expansion of video packed added to the adapter,
but also a fix to EOS condition when the subclass parse() function
ultimately only needed to call into gvd_have_frame() and no additional
source bytes were consumed, i.e. gvd_add_to_frame() is not called.

This situation can occur when decoding H.264 streams in byte-stream/nal
mode for instance. A decoder always requires the next NAL unit to be
parsed so that to determine picture boundaries. When a new picture is
found, no byte is consumed (i.e. gvd_add_to_frame() is not called)
but gvd_have_frame() is called (i.e. priv->current_frame is gone).

Also make sure to avoid infinite loops caused by incorrect subclass
parse() implementations. This can occur when no byte gets consumed
and no appropriate indication (GST_VIDEO_DECODER_FLOW_NEED_DATA) is
returned.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

tests: codec-select: fix compilation

mikey: add more Since markers for new methods

mikey: make message and payload mini-objects

Make the MIKEY message and payload objects miniobjects so that they have
a GType and are refcounted.
We can reuse the dispose method to clear our payload objects.
Add some annotations.
Implement a copy function for the MIKEY message.
Fix the unit test.

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

tests: codec-select: Plug element name memory leak

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

docs: Assorted documentation and introspection fixes for new 1.4 API

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

rtspconnection: also allow POST before GET

Don't only allow GET and then POST request to setup tunneling over HTTP
but also allow POST and then GET.

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

libvisual: Rename get_type() function to prevent conflicts with static linking

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

Release 1.3.90

Update .po files

playsinkconvertbin: fix caps leak

Let go the reference to the converter caps after using it

tools: add gst-device-monitor-1.0 utility

Just shows devices with basic info and exits. Or will
wait for more devices to show up or be removed with
the --follow option. It's also possible to pass filters
as command line arguments in the form DEVICE_CLASSES
or DEVICE_CLASSES:CAPS.

libs: There is no G_TYPE_CHECK_INTERFACE_TYPE and G_TYPE_CHECK_INTERFACE_CAST

Remove the macros that used them, nobody could've used them anyway.

pbutils: handle more H.264 profiles and levels.

Recognize H.264 Level 5.2, as exposed by modern 2160p30+ streams,
i.e. commonly known as 4K. Also add initial support for handling
Annex.G (SVC) profiles.

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

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

typefind: Bump iso mp4 typefinder to PRIMARY. Add mp4 extension hint.

Fixes a problem with at least one file being detected incorrectly as
DTS because there's DTS packets early enough in the file.

tests: fix vararg handling in rtpbasedepayload unit test

Makes it pass on 32-bit systems.

tests: fix vararg handling in rtpbasepayload unit test

Makes it pass on 32-bit systems.

playsinkconvertbin: Filter out ANY capsfeatures from the converter caps

We can't convert to ANY capsfeatures, they are only there so that we
can passthrough whatever downstream can support... but we definitely
don't want to return them to upstream.

Back to development

Release 1.3.3

Update .po files

po: Update translations

audiodecoder: Don't be too picky about the output frame counter

With most decoder libraries, and especially when accessing codecs via
OpenMAX or similar APIs, we don't have the ability to properly related
the output buffers to a number of input samples. And could e.g. get
a fractional number of input buffers decoded at a time.

Previously this would in the end lead to an error message and stopped
playback. Change it to a warning message instead and try to handle it
gracefully. In theory the subclass can now get timestamp tracking
wrong if it completely misuses the API, but if on average it behaves
correct (and gst-omx and others do) it will continue to work properly.

Also add a test for the new behaviour.

We don't change it in the encoder yet as that requires more internal logic
changes AFAIU and I'm not aware of a case where this was a problem so far.

tcpserversrc: close the server socket after accepting a connection

g_socket_accept() is only called once for a server socket. So
keeping the socket open ist just confusing possible clients.

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

tcpclientsrc: return FLUSHING when select() is canceled

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

tcpserversrc: return FLOW_FLUSHING instead of an error when accept/select is canceled

Canceling the accept/select happens when the source is shut down. This is
not an error and the GST_FLOW_ERROR causes problems when only part of the
pipeline is shut down.

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

mikey: Fix Wall to NTP conversion

We are scaling from a unit in microseconds to a unit in ((1 << 32) per seconds).

We therefore scale the microseconds values by:

         value of a second in the target unit (1 << 32)
  --------------------------------------------------------------
  value of a second in the origin format (1 000 000 microsecond)

oggdemux: allow unset seek stop time in push mode

docs: add streamsynchronizer to documentation

docs: add playsink element to documentation

docs: add navigation interface to docs

appsrc: add send_event handler for flushing

Adds a send_event handling for allowing appsrc to flush its internal
data, allowing users to flush the pipeline without setting it to null.

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

videoscale: vs_image: strides are a gsize

The strides that are set from the GstVideoInfo structs are
a gsize. Using an int can cause overflows when dealing with large
enough images

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

video: avoid overflows when doing int operations for size

size is a gsize, so cast the operands to it to avoid overflows
and setting wrong value to the video size.

Includes tests.

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

theoraenc: Remove unneeded check

running timestamps are guaranteed to be positive and valid since the
GstVideoEncoder base class will clip incoming buffers

CID #1139797

vorbisenc: add missing va_end in variadic function

Coverity 1139944

tests: fix uninitialized variable use in video decoder test

uridecodebin: Also catch CODEC_NOT_FOUND errors and delay them until all decodebins are done

uridecodebin: Ignore missing-plugin messages unless all decodebins post one

When playing RTSP streams there will be one decodebin per stream. If some of
them fail because of a missing plugin we should not fail completely but play
the supported streams at least.

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

decodebin: Do async-done on expose errors too

dmabuf: fix checking mmap flags

A simple '&' is not sufficiant. With mmapping_flags == PROT_READ and
prot == PROT_READ|PROT_WRITE the check produces the wrong result.
Change the check to make sure that prot is a subset of mmapping_flags.

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

alsasink: make gst-ident happy

alsasink: fix occasional crash intersecting invalid values

When a pipeline using alsasink and push mode upstream fails
to preroll, the following state will be the case:
- A loop upstream will be PAUSED, pushing a first buffer
- alsasink will be READY, pending PAUSED, because async

On error, the pipeline will switch to NULL. alsasink is in
READY, so goes to NULL immediately. It zeroes its cached
caps. Meanwhile, the upstream loop can cause a caps query,
conccurent with the state change. This will use those cached
caps. If the zeroing happens between the NULL test and the
dereferencing, GStreamer will critical down in the GstValue
code.

Since it appears that such a gap between states (PAUSED
and pushing upstream, and NULL downstream) is expected, we
need to protect the read/write access to the cached caps.

This fixes the critical.

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

videodecoder: Keep still meaningfull pending events on FLUSH_STOP

Only EOS and segment should be deleted in that case.

+ Add a testcase

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

audiodecoder: Keep still meaningfull pending events on FLUSH_STOP

Only EOS and segment should be deleted in that case.

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

videoencoder: Keep still meaningfull pending events on FLUSH_STOP

Only EOS and segment should be deleted in that case.

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

streamsplitter: Keep still meaningfull pending events on FLUSH_STOP

Only EOS and segment should be deleted in that case.

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

audioencoder: Keep still meaningfull pending events on FLUSH_STOP

Only EOS and segment should be deleted in that case.

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

oggstream: consider all opus packets as "keyframes"

This lets oggdemux determine they are not delta units, and removes
spurious per packet warnings about being unable to determine the
packet's keyframeness.

mikey: Free MikeyPayload in error cases

CID #1212136

decodebin: aggregate buffering messages

Aggregate buffering messages to only post the lower value
to avoid setting pipeline to playing while any multiqueue
is still buffering.

There are 3 scenarios where the entries should be removed from
the list:

1) When decodebin is set to READY
2) When an element posts a 100% buffering (already implemented)
3) When a multiqueue is removed from decodebin.

For item 3 we don't need to handle it because this should only
happen when either 1 is hapenning or when it is playing a
chained file, for which number 2 should have happened for the
previous stream to finish

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

audio: Add a missing precondition to gst_audio_format_from_string()

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

tests: videodecoder: audiodecoder: add tests for eos after segment

Tests that pushing a buffer after the segment returns EOS

videodecoder: actually return the push result in backwards playback

It was always returning _OK regardless of what downstream returned

videodecoder: return EOS when segment is over

if a buffer is clipped by being completely out of segment, check if this
buffer is after the end of the segment and return EOS upstream

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

audiodecoder: return EOS when segment is over

if a buffer is clipped by being completely out of segment, check if this
buffer is after the end of the segment and return EOS upstream

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

oggdemux: use new gstutils helper GstFlowCombiner

Fixes the handling of GST_FLOW_EOS by using the helper object
from gstutils that does the correct combination of flow returns.

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

tools: play: use cubic volume factor when adjusting volume

This is more natural and better-suited for a playback application.