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

videometa: fix texture_type memcpy size

Coverity 1139589, 1139588

sdpmessage: fix multi statement macros

Wasn't playing nice with an if statement below.

Coverity 1139767

audiocdsrc: guard aginst overflow

An audio CD may contain about a tenth of the samples 32 bit can
represent, so it doesn't seem likely this will be hit in practice.

Coverity 1139805

pbutils: descriptions: default to systemstream=false for partial video/mpeg caps

Assume systemstream=false for video/mpeg caps where that field
is missing.

audiobasesink: avoid possible sample count overflow

At 48 kHz, 2<<31 samples is reached before 13 hours so it
sounds plausible this would be hit.

Coverity 1139800, 1139801

theoraenc: fix comparison to unset timestamp

Also rejects negative timestamps that aren't GST_CLOCK_TIME_NONE.

Coverity 1139797

oggstream: fix a few left shifts operations on 32 bits cast to 64 bits

This should not cause any actual bug since Theora and Daala have
a maximum shift of 31, and a packet duration of 2^31 seems very
implausible. But it fixes:

Coverity 1139804, 1139803, 1139802

oggstream: remove NULL test after dereference

And add NULLness asserts at top of function. The only call
to this passes local variable pointers, so non NULL.

Coverity 206375

oggmux: test for failure to return tag

It should really not happen unless the tag list it corrupt,
but the API returns a failure code so we may as well use it.

Coverity 1139595

oggdemux: do not dereference NULL pad in warning message

Coverity 1197695

video-event: Update the running times in the force-keyunit events from the pad offsets

decodebin: In adaptive streaming mode, only have a fixed buffer limit for the non-buffering multiqueue

sdp: guard against address parse errors.

adder: rework the logic to check if eos has to be sent.

Checking the size available was incorrect, and the infos
for per-pad EOS are available.

Same logic as audiomixer.

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

audioringbuffer: parse channels field from compressed audio caps

Also parse channels as an optional field in the caps for compressed
audio formats.

subtitleoverlay: Consider all caps for overlays, not just the first.

Check all supported caps on the overlay video pad, not just the
first of (possibly) many.

tools: update gst-play-1.0 man page

videodecoder: do not deactivate the bufferpool, just unref

Videodecoder does late renegotiation, it will wait for the next
buffer before renegotiating its caps and bufferpool. It might happen
that downstream element switched from passthrough to non-passthrough
and sent a reconfigure upstream (that caused this renegotiation).
This downstream element will ask the video sink below for the bufferpool
with an allocation query and will get the same bufferpool that
videodecoder is holding, too.

When renegotiating, if videodecoder deactivates its bufferpool it
might be deactivating the bufferpool that some element downstream
is using and cause the pipeline to fail.

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

audiobasesink: clip start samples to match clipped start time

Clock slaving can clip start time to zero, giving us a shorted
duration than we originally got. To keep in sync, we must then
discard the samples falling before that zero timestamp.

This possibly fixes random distortion caused by constant PA
underflows which are never resynced.

mikey: Fix the KEMAC payload

The KEMAC payload actually needs to have subpayloads and the key should
go into the KEY_DATA subpayload. Add support for subpayloads and
implement the KEY_DATA payload.
Add some pointers to the conversion functions that allow us to add
encryption and decryption later.

playbin: Drop reference to any source element in NULL state

Drop the reference instead of waiting for either finalize(), or
for a new source when reused. Everyone else already forgot about
the old source.

rtspconnection: Added gst_rtsp_watch_set_flushing to list.

Added gst_rtsp_watch_set_flushing to list in file
libgstrtsp.def

videodecoder: Always drain the decoder after a discont group in reverse playback mode

videodecoder: Flush the decoder once per discont group, not once per keyframe

videodecoder: Handle reverse playback with multiple GOPs per discont group properly

baseparse will reverse each GOP for us already, so the segment events can
be after our keyframe. Make sure to get it and all other relevant sticky
events before starting to decode.

videodecoder: Log event types of events that are pushed downstream