souphttpsrc: Unset limit on the number of connection if soup session sharing is used

Soup allows only up to two connections per host in a session,
if we use default value. When session sharing is used, however,
more connections might be required in a session.
(e.g., multi-audio adaptive streaming case)

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

imagefreeze: fix use-after-free on seek event

Get seqnum before unreffing the seek event.

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

qtmux: robustify time tracking for sparse subtitle stream

qtmux: correctly track chunk size of subtitle stream

... thereby ensuring correct chunk offset tracking for all streams.

rtpstats: fix assertion 'denom > 0' failed

gst_util_uint64_scale_int takes a gint as denom parameter
whereas ctx->clock_rate is a guint32.

It happens when gst_rtp_packet_rate_ctx_reset set clock_rate
to -1.

So just define clock_rate as gint like it is done in rtpsource.h

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

v4l2: Block recursive calls to resurect_buffer

When resurrecting a buffer, the subsequent free call can result
in the group-released handler being called again, which causes
a recursive loop.  This patch blocks the signal handler during
the time that it executes, ensuring that the loop will not occur.

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

tests: souphttpsrc: Avoid deprecated ssl-ca-file property

SoupSession's ssl-ca-file property is deprecated. Use the recommended
tls-database property.

This is a bit more complex as it requires creating a GTlsFileDatabase
object for an absolute (!) path to the CA certificates file.

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

tests: souphttpsrc: Avoid deprecated server ssl properties

The ssl-cert-file and ssl-key-file properties are deprecated. Use the
soup_server_set_ssl_cert_file function to load the files.

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

tests: souphttpsrc: Make ssl_cert/key_file static

Just a bit of cleanup.

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

tests: souphttpsrc: Update test-cert.pem

Recent GnuTLS disregards the Common Name and only looks at the Subject
Alternative Name extension. Since our test-cert has no SAN extension,
validation fails.

Generate a new certificate with SAN. In addition to 127.0.0.1, for good
measure make it valid for localhost and ::1, too.

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

souphttpsrc: Allow any type of proxy

Currently we only allowed HTTP proxy. Don't filter for the scheme, just check
if it looks like an URI. Soup will warn if the URI is invalid or if
proxy protocol is not supported. This enables using SOCKS 4/5 which is
directly implemented into GIO.

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

v4l2: increase by one the number of allocated buffers

Increasing this number fix a buffer starvation problem I'm hitting
with a "v4l2src ! kmssink" pipeline.

kmssink requests 2 buffer as it keeps a reference on the last rendered
one. So we were allocating 3 buffers for the pipeline.
Once the first 2 buffers have been pushed we ended up with:
- one buffer queued in v4l2
- one being pushed
- one kept as last rendered

If this 3rd buffer is released after that v4l2 used the first one to
capture we end up with a buffer starvation problem as no buffer is currently
queued in v4l2 for capture.

Fixing this by adding one extra buffer to the pipeline so when one
buffer is being pushed downstream the other can already be queued to
capture the next frame.

We were already adding 3 buffers if downstream didn't reply to the
allocation query. I reduced this number to 2 to compensate the extra
buffer which is now always added.

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

rtspsrc: Create send/recv mutexes once, not on every connect()

Also fixes a crash caused by freeing an uninitialized mutex in an error
case.

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

v4l2allocator: Fix memory leak with dmabuf

This patch fixes a memory leak that is caused if the dmabuf file
descriptor dup fails.  Previously, _cleanup_failed_alloc() would
not unref the memory because mems_allocated had not yet been
incremented.

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

qtdemux: specify '_swr' atom as a container atom

... so it is parsed as an mp4 style metadata atom as written by muxer

qtmux: initialize mdhd language code as undefined

rtpjitterbuffer: Add a faststart-min-packets property

When set this property will allow the jitterbuffer to start delivering
packets as soon as N most recent packets have consecutive seqnum. A
faststart-min-packets of zero disables this feature. This heuristic is
also used in rtpsource which implements the probation mechanism and a
similar heuristic is used to handle long gaps.

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

meson: Allow using glib as a subproject

meson: build examples

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

meson: fix with-package-name option

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

meson: tests: icles: simplify build file

meson: build tests/icles/

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

qtmux: correctly calculate overall first_ts to ensure stream sync

... by minding and compensating for the dts_adjustment that may have
been introduced in the PTS timeline.

matroskademux: track highest known cluster position and time

... to use as a fallback initial duration estimate and to provide for
interpolation when scanning for position.

matroskademux: improve and simplify searching for cluster and position

... avoiding inefficiency proportional to file size

matroskademux: increase chunk size when scanning for cluster

matroskademux: maintain variable state when searching for position

... so skipping to next cluster happens efficiently

meson: build raw1394 plugin

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

meson: build aalib plugin

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

meson: build caca plugin

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

docs: update for git master

sys: remove sunaudio plugin

Even though hooked up to the build system, it's clear that no one
has ever built or used this with GStreamer 1.x. It wants to link
against libgstinterfaces, which no longer exists. And uses 0.10-style
raw audio caps. And the last meaningful change was done in 2009.
Let's just remove it.

meson: build oss4 plugin

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

meson: build oss plugin

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

rtspsrc: Actually use the receive lock when receiving, not the send lock

flactag: Fix warning with the newly added GstStateChange values

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

rtspsrc: do not checksum the stream id

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

qtmux: add support for muxing PNG

Demuxer already supported it.

rtspsrc: Use a mutex for protecting against concurrent send/receives

We currently send data to the RTSP connection from multiple threads:
whenever a command is to be handled and whenever RTCP is generated. This
can cause data corruption or worse if both happen at the same time.

As such, protect gst_rtsp_connection_send() and gst_rtsp_connection_receive()
calls with a mutex. While this means that we hold a mutex during the IO
operation, this is not actually a problem as the IO operation can be
interrupted (gst_rtsp_connection_flush()) at any time and is blocking by
itself anyway.

qtmux: Un-merge the last two stsc entries after serializing

The last entry will most likely get new samples added to it in "robust"
muxing mode, changing the samples_per_chunk and thus making it wrong to
keep the last two entries merged. It will run into an assertion later
when adding a new sample to the chunk.

Thanks to gdiener@cardinalpeak.com for the analysis of the bug and
proposal for a solution.

wavparse: Actually clip to upstream size instead of size of the data chunk

There might be other chunks after the data chunk, so clipping the chunk
size with the data size can lead to a negative number and all following
calculations go wrong and cause crashes or worse.

This was introduced in 3ac119bbe2c360e28c087cf3852ea769d611b120.

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

splitmux: Drop allocation queries

They can cause us to deadlock, while we're waiting for a new frame and
upstream is waiting for the allocation query to be answered before
sending a frame

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

rtspsrc: uniquify stream ids

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

meson: Do not use path separator in test names

Avoiding warnings like:

    WARNING: Target "elements/audioamplify" has a path separator in its name.

Fix v4l2 example

qtdemux: remove not needed code

remove not needed code about res variable.

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

v4l2videoenc: Make sure min_buffers is valid

When upstream does no use the v4l2videoenc pool, we need to activate
that internal pool. Though, we relied the driver to provide a minimum
required buffer, which Qualcomm Venus driver don't currently provide.

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

rtph265depay: fix caps leak

rtph264depay: simplify buffer accumulation control flow

There is no difference between pushing out a buffer directly
with gst_rtp_base_depayload_push() and returning it from the
process function. The base class will just call _depayload_push()
on the returned buffer as well.

So instead of marshalling buffers through three layers and back,
just push them from one place in handle_nal() and always return
NULL from the process vfunc. This simplifies the code a little.

Also rename _push_fragmentation_unit() to _finish_fragmentation_unit()
for clarity. Push sounds like it means being pushed out, whereas
it might just be pushed into an adapter.

This change has the side-effect that multiple NALs in a single STAP
(such as SPS/PPS) may no longer be pushed out as a single buffer if
we output NALs in byte-stream format (i.e. not aggregate AUs), but
that shouldn't really make any difference to anyone.

rtpsession: print value of unknown RTCP Payload Type

This adds printing the actual value of any unknown RTCP PT
to the already existing WARNING log message.

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

v4l2videoenc: Don't leak VideoCodecState

CID #1409852

dvdemux: Remove un-needed variable check

if pad wasn't present by now everything would have broken before

CID #1409854

po: update POTFILES

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

v4l2videoenc: Remove unused function

v4l2: Don't redefine __bitwise if already set

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

v4l2: Add Video Encoder support

This implements H264 encoding support using generic V4L2 interface. It is
reported to work with Samsung MFC driver, IXM.6 CODA driver and
Qualcomm mainline Venus driver. Other platform should be supported as
none of this work is platform specific.

The implementation consist of a GstV4l2VideoEnc base class, which
implements the core streaming functionality. This base class is implemented
by GstV4l2H264Enc class that implements the caps negotiation specific to
H264 profiles and level. This implementation supports hardware with multiple
H264 encoder. Though, to make it simplier to use, the first discovered H264
encoder will be named v4l2h264enc. Other encoder found during discovery will
have a unique name like v4l2video0h264enc.

This work is the combined work of multiple developpers in the last 3
years. Thanks to all of the contributors:

  Ayaka <ayaka@soulik.info>
  Frédéric Sureau <frederic.sureau@vodalys.com>
  Jean-Michel Hautbois <jean-michel.hautbois@veo-labs.com>
  Nicolas Dufresne <nicolas.dufresne@collabora.com>
  Pablo Anton <pablo.anton@vodalys-labs.com>

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

v4l2videodec: Remove unused forward declaration

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

v4l2pool: Fix wrong error message

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

v4l2: increase pre-allocated encoded buffer size

As of today, the MFC encoder often need to exceed that 1 MB
size for encoded buffer we fixed earlier for decoding.

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

rtpopusdepay: minor perf improvements

Use the ::process_rtp_packet() vfunc to avoid mapping the
RTP buffer twice.

gst_rtp_buffer_get_payload_buffer() returns a new sub-buffer
which will always be writable, so no need to make it writable.

rtp: opus: use existing utility funcs for copying/dropping metas

We had our own copies of those while the code was in -bad, but now
we can use the existing utility functions instead of re-implementing
them.

rtp: cache meta tag quarks and add more utility functions for metas

Every g_quark_from_static_string() is a hash table lookup serialised
on the global quark lock in GLib. Let's just look up the two quarks
we need once and cache them locally for future use. While we're at it,
add new utility functions for the two most commonly used tags
(audio + video). Make first argument a gpointer so we don't have to
cast and make the code ugly. These are used for logging purposes
only anyway.

aacparse : Fix, Caps were not set while reusing aacparse

While reusing aacparse caps were not set.This fix enables aacparse to reuse in same pipeline.

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

meson: don't need config.h.meson any longer

shout2send: use non-blocking I/O and a configurable network operations timeout

This allows timing out on network errors much earlier
(currently it takes ~15min to timeout) and we can still
unlock and change state in the meantime.

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

meson: make C++ compiler optional

It's only needed for the taglib plugin which is optional.

multifile: remove some cruft

osxaudio: fixes playback of mono streams with no channel-mask field in caps

Fixes a negotiation error seen when trying to playback of a .MOV file with
a mono AAC audio stream decoded by avcdec_aac that doesn't set channel-mask
field but sink was requiring channel-mask=0x3.

dvdemux: Push tag event to both pads

Tags are pushed to "videosrcpad"/"audiosrcpad" in
gst_dvdemux_add_pad() method, however they will be NULL
in this method, hence tags are not pushed.
Instead, send tag event to "pad" created gst_dvdemux_add_pad().

Signal no-more-pads when both pads are created

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

meson: add options to set package name and origin

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

multifilesink: fix property name in example pipeline

Since the move from CVS the property name of the documentation example
has been filename instead of location. Users trying the gst-launch
command as is will get:
no property name "filename" in element

Fixing it.

osxvideo: fix macOS 10.12 deprecation warnings

Add #defines to allow older versions of macOS to use the new constant names.

isomp4: Safely ignore [skip] atoms

Instead of warning about them

souphttpsrc: Make session sharing thread-safe on our side

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

Fix up package name and origin in some plugins

gst: Clear floating flag in constructor of all GstObject subclasses that are not owned by any parent

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

1394: Sink the clock reference in the constructor

This is now needed as GstClock does not do that internally anymore,
because that broke bindings.

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

pulse: Accept MPEG 1 layer 3 version 2.5

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

Remove plugin specific static build option

Static and dynamic plugins now have the same interface. The standard
--enable-static/--enable-shared toggle are sufficient.

splitmuxsink: Add alignment-threshold argument

If a non-reference stream is behind the reference stream by an amount of
time smaller than the alignment threshold (in nsec), it counts as being
after it.

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

qtmux: Do not check timecode data for mp4 container

Timecode trak is only supported for mov right now, not for mp4. That
code would otherwise create an invalid trak if the muxed video contained
timecode metadata.

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

qtmux: When accepting renegotiation, just return TRUE and change nothing

We only accept new caps if they are basically the same. We don't want to
reset anything as if the caps are new, otherwise various state could get
out of sync with the current run.

qtmux: In prefill mode, only pad buffers with > 0 sized memories as needed

Adding a 0-byte memory has not much effect.

Also add some debug output.

qtmux: Lateness is in QT timescale, diff in GstClockTime

Print the right one in debug output to get meaningful numbers.

qtmux: Error out if a gap edit list has to be written in prefill mode

We don't have any space reserved for this in the moov and the
pre-finalized moov would have broken A/V synchronization. Error out here
now

qtmux: Calculate with reserved moov size instead of last moov size

We have some padding added after the initial moov, so a bigger updated
moov can be handled to some degree and is expected. Previously we just
ignored the padding and errored out in cases when the padding would've
just been enough.

qtmux: Error out directly if sending filler data results in a flow error

CID 1405994

qtmux: In prefill mode, handle the case when only the first chunk was ever used

souphttpsrc: Use a in-memory cookie jar by default in sessions we created

This ensures that cookies are stored and used as set by the server, and
shared with other souphttpsrc that use the same SoupSession.

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

souphttpsrc: Implement soup session sharing

souphttpsrc now shares its SoupSession with other elements in the
pipeline via GstContext if possible (session-wide settings are all the
defaults), or if the context was forced by the application.

This allows multiple souphttpsrcs to reuse connections, cookies, etc.

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

qtmux: Add new prefill recording mode

This sets up a moov with the correct sample positions beforehand and
only works with constant framerate, I-frame only streams.

Currently only support for ProRes and raw audio is implemented but
adding new codecs is just a matter of defining appropriate maximum frame
sizes.

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

qtmux: Error out on discontinuities/gaps when muxing raw audio

When muxing raw audio, we have no way of storing timestamps but are just
storing a continuous stream of audio samples. If the difference between
the expected and the real timestamp becomes to big, we should error out
instead of silently creating files with wrong A/V sync.

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

vpxdec: Set fb->priv to NULL after freeing just in case

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

directsoundsink: Use GstClock API instead of Sleep() for waiting

It's more accurate and allows cancellation.

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

vpx: fix build against older libvpx versions

Such as 1.3.0 as on raspbian.

directsoundsink: Fix corner case causing large CPU usage

We were unnecessarily looping/goto-ing repeatedly when we had exactly
the amount of data as the free space, and also when the free space was
too small. This, as it turns out, is a very common scenario with
Directsound on Windows.

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

We have to do polling here because the event notification API that
Directsound exposes cannot be used with live playback since all events
must be registered in advance with the capture buffer, you cannot
add/remove them once playback has begun. Directsoundsrc had the same
problem.

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

directsoundsink: Clean up some debug logging

Don't need to print the function name, gstreamer does it for you.

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

matroskademux: improve index memory usage

Re-arrange order of index entry struct members to avoid padding
bytes in the middle of the struct, thus potentially reducing the
overall size of the struct and reducing memory used by the index.
On Linux x86_64 the size goes down from 32 bytes to 24 bytes for
each index entry.

Back to development

Release 1.12.0