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

rtspconnection: Add more tests

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

videotestsrc: fix undefined behaviour of left-shift

With a small type for the color values being left-shifted, the result is
undefined and it could potentially overflow.

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

win32: fix export files again

Revert unintended parts of d8a0927930a87a2eb60d4c98cb3fea8aed911b27

Add mikey.h file

audiodecoder: Make caps writable before fixating

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

sdpmessage: Add array length annotation to gst_sdp_message_parse_buffer

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

rtpbuffer: fix memory leak when gst_rtp_buffer_map fails

Make sure rtp->data[3] is set before jumping to error path.

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

gst-play: add option to supply media files from playlist file

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

giobasesink: we mustn't change the format of a query response

Not even in the DEFAULT case. That's bad 0.10 behaviour, no caller
is ever going to check the format of the response.

playbin: add nick for soft colorbalance play flag to fix gst-inspect

Fix gst-inspect-1.0 playbin criticals when printing the
flags, which was caused by a missing nick name for one
of the flags.

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.

giobasesink: Implement handling of the SEEKING query

audiodecoder: Plug caps leaks

We were returning in various places without unreffing the caps, and
we were also leaking (overwriting) the caps we got from _get_current_caps()

Spotted by Haakon Sporsheim in #gstreamer

audioresample: Don't left-shift into the sign bit, instead use unsigned integers

tag: exif: avoid adding empty strings

Fixes assertion with some jpeg files

play: Improve pipeline states

First set the pipeline to the PAUSED state to check if we are dealing
with a live pipeline or not. Then move to the desired state.

If we don't do this, it is possible that we receive a BUFFERING message
before we know that the pipeline is live and we would set the pipeline
to PAUSED and deadlock.

play: Update buffering state for live pipelines

Update the buffering variable, even for live pipelines so that we don't
print \n for each buffering message.

videoframe: Initialise GstVideoFrame 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.

exiftag: catch buffer mapping failure

Might be what caused:

Coverity 1139734

audioresample: Fix memory leaks in test

audioresample: Fix up indention

audioresample: Fix out of bounds memory accesses

pango: Make static caps actually static to fix a memory leak

videotestsrc: Fix memory leak in test

encodebin: Fix memory leak in test

encoding-profile: Free preset name in finalize

oggmux: Clear Ogg streams before initing them

They might've been inited before, in which case we leak
memory when initing them again without clearing.

audioconvert: Fix leaks in unit test

videoencoder/decoder: Fix memory leaks in the tests

audiodecoder: Actually allocate enough memory for 64 bits, not just 32 bits

Also fix a memory leak.

audioencoder: Fix memory leaks in unit test

rtp: Fix GBytes memory leak in test

rtpbasedepay: add stats property

Add a stats property that holds a structure with all the current
values of the depayloader.

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

rtpbasepayload: update docs

rtpbasepayload: add current timestamp and seqnum offset to stats

Expose the current timestamp and seqnum offset in the stats

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

textrender: push segment event after caps event

Fixes warning "Sticky event misordering, got 'segment' before 'caps'".

oggstream: use G_GUINT64_CONSTANT instead of ll suffix

Thanks slomo for pointing out it's not standard.

xvimage: remove dead code

matching_attr can not be NULL here, we've tested that away a few
lines beforehand.

Coverity 1139655

videotestsrc: bail out on unsupported caps

This avoids using uninitialized data (and properly rejects caps).

Coverity 1139898

typefind: remove pointless checks for data being NULL

It was already checked in an early out, and as it's only
incremented for at most the size of the passed buffer, it
can only become NULL in an address wraparound.

While there, don't cast away const on a pointer.

Coverity 1139845

decodebin: consider "no demuxer" case to not have dynamic pads

This fixes a possible NULL dereference.

Coverity 1195146

encodebin: guard against gst_pad_get_peer returning NULL

If it does, the pad may be leaked if it's a request pad, though.

Coverity 1139799

encodebin: guard against pathological NULL dereference

Coverity 1139798

audioresample: reject 0 denominator when creating resampler

Coverity 1195140, 1195139, 1195138

video-overlay-composition: guard against NULL pointer dereference on error

If gst_video_overlay_rectangle_apply_global_alpha is called with
a rectangle with unsuitable alpha, expanding the alpha plane will
fail, and thus lead to dereferencing a NULL src pointer. It's not
certain this will happen in practice, as the function is static
and callers might ensure suitable alpha before calling, but there
is no apparent explicit such check.
Add prologue asserts for proper alpha to explicitely prevent this.

Coverity 1139707