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.

Back to development

Release 1.3.2

Update .po files

Automatic update of common submodule

From 211fa5f to 1f5d3c3

video: And check comparison for real

video: Fix broken comparison in unit test

libs/video.c:540:50: error: comparison of constant 2 with boolean expression is always false
      [-Werror,-Wtautological-constant-out-of-range-compare]
            && !GST_VIDEO_INFO_N_PLANES (&vinfo) > 2) {
               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~

rtsp-transport: clarify port usage

Comment in the docs what the client_port and server_port fields are used
for in TCP mode (if the application wants to set those values).

dmabuf: share the mapping with shared copies of the memory

With lots of shared memory instances (e.g. created by a RTP payloader) the
overhead of duplicating the file descriptor and creating extra mappings is
significant. To avoid this, the parent memory maps the whole region and the
shared copies just reuse the same mapping.

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

rtspconnection: Add read source on write socket.

Add a read source on write socket when lost tunnel.
To be able to detect when clint closes get channel.

This is already done in gst_rtsp_source_dispatch_write but
only when the queue is empty.

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

playsink: Always take the playsink lock when adding or removing pad probes

Otherwise we might end up inside the callback without having stored
the probe id... then try to remove that probe (not!) from the callback
and wait forever for the pad to unblock.

alsasink: pass correct error to g_strerror

The error we get is a negated errno.

While there, fix a couple typos in messages.

gst-play: Free playlist_file string if only printing the version

audio_sink and video_sink leakage fixed

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

rtspconnection: Don't use argument for local storage

By re-using the uri argument for storing local data, we could end up in
a situation where we would free uri ... which would actually be the
string passed in argument.

Instead explicitely use a local variable. Fixes double-free issues.

CID #1212176

video-info: Also check the stride and offset are equal

gst_video_info_is_equal() was not checking if stride and offset
had changed.

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

videodecoder: Free data after removing it from the list

While it wouldn't have caused any failures (g_list_remove doesn't dereference
the provided pointer), it does make the code cleaner.

CID #1212174

mikey: Actually replace payload ...

This function is intented to replace the payload, let's actually do that
instead of putting back the same (freed) payload

CID #1212175

mikey: Free MikeyPayload in error cases

CID #1212135
CID #1212136
CID #1212137
CID #1212138

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

It does not makes sense and asserts

tag: xmp: fix leaks in error code paths

CID 1212133

rtspconnection: Reset control_stream.

Reset control_stream when gst_rtsp_connection_close.

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

videodecoder: 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. This ensure we have a configured
pool if possible.

uridecodebin: use downloadbuffer for download buffering

Use the new downloadbuffer element to implement the download buffering
feature

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

oggmux: push eos event when empty pad data

If gst_ogg_mux_queue_pads returns NULL it means we are at EOS, because we get a
NULL buffer and this function never sets bestpad.

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

configure: Use X11 detection macro from common

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

examples: playback-test: fix crashes when setting buffer-size

playbin's buffer-size property takes a gint, not a gint64,
so only pass the bits expected to the vararg function, or
the terminator might not be found, leading to crashes, esp.
with negative numbers.

Spotted by Ravi Kiran K N <ravi.kiran@samsung.com>

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

examples: fix indentation of playback-test

Revert "playback-test: Set buffer-size only for non-negative size"

This reverts commit 07a637e2847d56d0f2b0c0ac9095bf37dd324e26.

playback-test: Set buffer-size only for non-negative size

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

win32: Update defs file

commit 622007e7db7e3d32bf8e04e673e057897b646220 added the function
gst_discoverer_info_get_missing_elements_installer_details (). It needs to be
added to the defs file.

rtsp: Link to ws2_32 on Windows

Needed for getsockname and setsockopt

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

Make X11 detection more precise

Don't be content with just X11/Xlib.h, check for X11/XKBlib.h as well.
This prevents false positives (for example, from partial X11 headers
installed by tcl/tk).

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

tests: fix printf format compiler warning in playback test on win32

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

Add new unit test binary to .gitignore

discoverer: Add APIs to simply get installer details for missing plugins

Currently the API is far from optimal and the user has to work around
our badly defined API to simply install missing plugins.

API:
  new:
    gst_discoverer_info_get_missing_elements_installer_details

  deprecated:
    gst_discoverer_info_get_misc
    gst_discoverer_stream_info_get_misc

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

Back to development

textoverlay: Link unit test with the local version of the library, not an installed one

Release 1.3.1

Update .po files

po: Update translations

rtpbasepayload: Implement reconfigure event & renegotiation without subclass

Implement the reconfigure event, also do correct downstream caps negotiation
if the subclass doesn't implementy set_caps.

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

tests/check/libs/rtpbasepayload.c: Run gst-indent

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

Automatic update of common submodule

From bcb1518 to 211fa5f

rtpbasepayload: Save the PT after fixating

rtspdefs: remove outdated comments

rtpbuffer: avoid underflow in size calculation

videodecoder: do not parse caps for not using it

Saving some cpu

rtpbasepayload: restrict initial random sequence number to be <= 32767

In order to prevent SRTP roll over counter issues the initial sequence
number is restricted to <= 32767. This is recommended by RFC 4568 section 6.4.

sdp: Add some more gobject-introspection annotations for bindings

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

playbin: Don't block on non-serialized events

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

playsink: Don't block on non-serialized events

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

playsinkconvertbin: Don't block on non-serialized events

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

subtitleoverlay: Don't block on non-serialized events

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

rtcpbuffer: check claimed data size against available size

Coverity 1208773

rtspconnection: Empty queue when flush.

Empty the watchs queue when calling
gst_rtsp_watch_set_flushing with flushing variabel is TRUE.

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