Release 1.11.90

Update .po files

po: Update translations

aacparse: streamline and improve AudioSpecificConfig parsing

AudioSpecifigConfig is used in a variety of AAC streams but was
being parsed differently. Instead, make everyone use the same parsing.

* Remove unused 'bits' field (it was always set to 0 if present)
* Add proper GAConfig parsing (to know the  number of samples per frame
  if present).

Fixes wrong rate/channels configuration in streams coming from qtdemux

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

v4l2videodec: Fix 32bit only printf format

The previous patch was using %llu for 64bits printf, which is 32bit
specific. We also trace the latency in time human readable form now.

v4l2object: set streamparm for outputs that support it

Without a specified framerate from the sink, the decoder frame interval
should be set using the framerate of the encoded video stream.
Therefore, the v4l2object should be able to change the framerate on the
output if the V4L2 device accepts it.

This is also necessary for mem2mem encoders so that their bitrate
calculation code may work correctly and they may report the correct
frame duration on the capture queue.

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

v4l2videodec: only set latency if the frame duration is valid

If the duration of the v4l2object is GST_CLOCK_TIME_NONE, because the
sink did not specify a framerate in the caps and the driver accepts the
framerate, the decoder element uses GST_CLOCK_TIME_NONE to calculate and
set the element latency.

While this is a bug of the capture driver, the decoder element should
not use the invalid duration to calculate a latency, but print a warning
instead.

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

v4l2sink: Block in preroll_wait on unlock

The correct behaviour of anything stuck in the ->render() function
between ->unlock() and ->unlock_stop() is to call
gst_base_sink_wait_preroll() and only return an error if this returns an
error, otherwise, it must continue where it left off!

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

vp9dec: Add warnings for unsupported frame formats

At least output an element warning on the bus when we
encounter a frame format GStreamer doesn't currently support.

aacparse: Handle Parametric Stereo with HE-AAC(v2)

According to ISO/IEC:14496-2:2009 , in the case of HE-AACv2 (audioObjecType
29) parametric stereo is used (a single mono track is used and then
transformations are applied to it to provide a stereo output).

We therefore report two channels in the case where there is one reported
in the audioChannelConfiguration.

Fixes the various issues where a demuxer would report two channels, but
then the parser would say there's only one channel, and then the decoder
would output two channels.

qtmux: Simplify buffer refcounting in add_buffer() and remove unneeded NULL checks

qtmux: Select the best pad based on the cached last_buf if any

last_buf is the one we're going to write next, not buf. As such we
should check timestamps against that one if there is one to select the
earliest pad.

Also remember the currently selected pad in the very beginning when
storing the first last_buf.

This both solves some edge cases where not the correct next pad was
selected corresponding to the target interleave.

qtmux: Error out immediately if a timecode is to be written but downstream return not-OK

qtdemux: Update variables before early exit

This is an update of d78d5896272d78df41e696fac929e7dfb3bb3dfa

We still exit as early as possible in case of non-ok/non-unlinked combined
flow, but we first make sure that we update the internal position variables.

This ensures that if upstreams "ignores" the flow return (and carries on pushing),
we don't end up processing data with completely bogus variables/positions.

interleave: avoid using uninitialised ordering_map

If self->channel_positions == NULL (which seems unlikely),
self->default_channels_ordering_map will be used unintialised.

We avoid that by keeping track of the channel_mask, which is set when
the ordering map is initialised.

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

interleave: don't overflow channel map with >64 channels

When there are more than 64 channels, we don't want to exceed the
bounds of the ordering_map buffer, and in these cases we don't want to
rempa at all. Here we avoid doing that.

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

v4l2videodec: Set rank to GST_RANK_PRIMARY

Change-Id: If9c79874dc4af246f485b63c151ff732bc31abc9

meson: Use get_pkgconfig_variable instead of calling pkg-config ourself

It is avalaible in meson 0.36 which is now are requirement

pkgconfig: Do not ever build an installed .pc file

meson: test: Fix environment object usage

pkgconfig: Generate the pkg-config with meson too

qtdemux: In gap mode, consider the mdat offset when calculating the remaining mdat size

The mdat generally does not start at offset 0, we have to include the
size of the moof and whatever else was in front of the mdat.

atomsrecovery: Error out when fseek() fails instead of silently ignoring

CID 1403262

v4l2object: Also add videometa if there is padding to the right and bottom

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

rtpmux: fix output segment and buffer DTS to correspond to the flattened PTS

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

qtmux: Remove some unused variables

qtmux: Remove a couple of unneeded levels of indentation

qtdemux: distinguish TFDT with value 0 from no TFDT at all

TFDTs with time 0 are being ignored since commit 1fc3d42f. They're
mistaken with the case of not having TFDT, but those two cases
must be distinguished in some way.

This patch passes an extra boolean flag when the TFDT is present.
This is now the condition being evaluated, instead of checking for
0 time.

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

qtmux: Reset current chunk after writing out timecode

If we have multiple tracks with timecodes, or it's not the first track
that has timecodes, or not the first buffer, we already started a chunk
for media data. We now need to "close" that chunk because we wrote data
for the timecode track and a new chunk has to be started for the
original track the next time it has data.

qtmux: Do timecode handling per track, not per muxer instance

There could be multiple video tracks with timecodes.

qtdemux: matroskademux: Ignore repeated seek events

Similar to what was done in adaptivedemux, ignore seek
events we've already handled - such as when they are received
on every srcpad of files with lots of streams.

dashdemux: Update mdatleft from overall mdatsize and offset when observing a gap

Otherwise mdatleft will have a value calculated from the initial
mdatsize minus the parts of the stream that we saw, which is not
including all the parts of the stream that might've been skipped.

souphttpsrc: check range response when update content_size

In case request_position is content_size,
the response content_length can be zero with status code 200.
(reproduced in Youtube http streaming)

Change-Id: If28565375e0e13dc84054fd2383268467faf68aa
(cherry picked from commit 0ae443abd9066053d0412c28fe9d9afddc8488c8)

docs: update two references to the removed 'mad' plugin

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

rtprtxqueue: add basic documentation and example pipelines

Mostly explaining the difference between rtprtxqueue and rtprtxsend.

v4l2: Fix meson plugin shared object name

It didn't match between AutoMake and Meson, and the Meson name
didn't math the plugin name (video4linux2).

rtprtxreceive: fix example pipelines and improve the documentation

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

flacparse: fix playback if sample number does not start at 0

This reverts commit 29b807685d3c962bbe8afe351c5dca97d59eb5e0, while
fixing the original breaking tests/check/pipelines/flacdec.

Revert "flacparse: fix playback if sample number does not start at 0"

This breaks gst-validate on the build server (though not locally),
and a unit test, and I can't run unit tests right now for some
unrelated reason.

This reverts commit 0747b56f8e7f4731d67f8d13a4bdc453dde0fdf7.

rtpsession: print the correct variable in debug statement

This debug statement is meant to print the time since the last (early)
RTCP transmission, not the last regular RTCP transmission (which also
happens to be set a few lines above to current_time, so the debug output
is just confusing)

rtprtxsend: convert LOG message to TRACE

This is printed too often (for every chained buffer!) and just clutters the logs.

rtpsource: fix warning message

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

flacparse: fix playback if sample number does not start at 0

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

rtpsource: get clock-rate from pt if needed to generate SR

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

souphttpsrc: Include GStreamer souphttpsrc version in default User-Agent string

rtph264depay: fix crash with empty sprops-parameters

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

atomsrecovery: also handle extra atoms after 'mdia' in a 'trak'

Take into account the atoms at the end of the 'trak' atom when
recovering it. So that its size (already computed and added in the trak
size) isn't making offsets wrong.

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

qtmux: avoid fallthrough to moovrecovery failure section

Return before that to preserve our successfull results, otherwise no
moov recovery information would be written

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

atomsrecovery: expect more atom types at the headers

Skip more atoms at the header until it finds the 'mdat' to continue the
moov recovery

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

pulse example: Remove

That example only tested the property probe interface, which has been removed.
The same kind of thing can now be done with the generic gst-device-monitor tool.

v4l2: Remove unused macro

v4l2: Remove unused definitions

qtmux: add CineForm support

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

qtmux: Only create new chunks if we have more than a single stream

There's no point in creating multiple chunks otherwise, it only wastes
some bytes for storing the chunk offsets.

qtdemux: add S16L support

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

splitmux test: Use passed first/last timestamps

Don't hard-code the expected timestamp range, use the
values the caller is passing in.

Add old plugin names to cruft list

This will help fixing uninstalled setup. Also fix missing path
correction in one of the plugin xml.

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

rtph264depay, rtph265depay: remove stray g_debug()

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

qtmux: init fourcc

Initialize the fourcc to 0 so that we can detect failure later.

tests: Add missing LDADD for libm in tests using math.h

Also, remove the math.h include for the one that just prentend to need
it.

Fix shout2 plugin doc generation

In the previous patch, we also renamed shout2send to shout2, so it does
not clash with it's feature. Though we forgot to rename it in the doc
reference. This patch also add a cruft detection on the xml that made me
miss this error.

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

Fix plugin filenames to match plugin names

- libgstpulse.so becomes libgstpulseaudio.so
- libgstsouphttpsrc.so becomes libgstsoup.so
- libgstoss4audio.so becomes libgstoss4.so

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

qtmux: Free EDTS instead of just clearing it and setting it to NULL

qtmux: Fix some memory leaks related to timecode tracks

splitmux: Add unit test for reverse playback

Ensure that reverse playback works and generates the range
of timestamps (0-3s) we expect, in monotonically descending order.

splitmuxsrc: Fix reverse playback

Fix the check for whether the start time of the segment has
been reached when playing in reverse. Otherwise, playback
stops after reaching the start of any file part, instead of
continuing until all parts within the segment have played

qtdemux: Don't lose crypto info on a new moof

We parse the next moof in advance of having pushed
all samples from the previous one in some cases, and
we'll still need the crypto info from the previous
fragment so keep around any unused crypto info entries
when adding new ones

qtmux: Update modification times when sending the moov

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

sbcparse: Fix up values for allocation enumeration.

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

souphttpsrc: add seekable flag if it is seekable

Change-Id: If66d168ac7fcc5847eaa2473d3731ac66ec05d66

rtprtxreceive: fix potential leak of old, unassociated, association requests

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

avidemux: Don't increment -1 / unset indices

CID 1398545

qtdemux: Protect against NULL pointer dereference for streams without caps

CID 1363332

rtph263pay: Free mac on errors

CID 1212149

rtpvorbispay: Add missing break to for loop

check: Fix splitmux test CFLAGS

Needs to know where the gstapp headers are

qtdemux: Fix compilation with gcc 7

qtdemux.c: In function ‘qtdemux_parse_samples’:
qtdemux.c:8450:39: error: ‘*’ in boolean context, suggest ‘&&’ instead [-Werror=int-in-bool-context]
         if (stream->samples_per_frame * stream->bytes_per_frame) {
             ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~

mpegaudioparse: Fix compilation with gcc 7

gstmpegaudioparse.c: In function ‘gst_mpeg_audio_parse_reset’:
gstmpegaudioparse.c:209:3: error: ‘memset’ used with length equal to number of elements without multiplication by element size [-Werror=memset-elt-size]
   memset (mp3parse->xing_seek_table_inverse, 0, 256);
   ^~~~~~
gstmpegaudioparse.c: In function ‘gst_mpeg_audio_parse_handle_first_frame’:
gstmpegaudioparse.c:951:7: error: ‘memset’ used with length equal to number of elements without multiplication by element size [-Werror=memset-elt-size]
       memset (mp3parse->xing_seek_table_inverse, 0, 256);
       ^~~~~~

rtpvorbispay: When getting new headers, replace the old version of them

This prevents storing an infinite amount of e.g. comment headers if they
come without a new initialization header in front of them. There can
only be one header of each type.

rtp-payloading: Add new test for Vorbis renegotiation

Check if encoding, payloading, depayloading and decoding works if the
stream configuration (and thus the headers) change.

vorbispay: Only replace headers when receiving a new config header

If we also replace all headers when receiving any possibly following
comments header, we would throw away the config header before being able
to make use of it.

tests: splitmux: add unit test for content with sparse streams

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

splitmuxpartreader: ignore sparse streams when calculating the end offset of a part

A sparse stream's ending timestamp can be considerably smaller
than the ending timestamps of the other streams, which can lead
to skipping considerable time from the next part.

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

splitmuxpartreader: identify sparse streams

Update Edgard Lima's email

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

rtpjitterbuffer: Don't always reset PTS to 0 after a gap

In function rtp_jitter_buffer_calculate_pts: If gap in incoming RTP
timestamps is more than (3 * jbuf->clock_rate) we call
rtp_jitter_buffer_reset_skew which resets pts to 0. So components down
the pipeline (playes, mixers) just skip frames/samples until pts becomes
equal to pts before gap.

In version 1.10.2 and before this checking was bypassed for packets with
"estimated dts", and gaps were handled correctly.

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

meson: Update version

Back to development

Release 1.11.2

Update .po files

po: Update translations

souphttpsrc: Extract redirection uri on libsoup's restarted callback

Let libsoup handle redirection automatically.
And then, to figure out redirection uri, extract it on "restarted"
callback which will be fired before soup_session_send() is returned.

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

v4l2object: Update image size when extrapolating

Update the image size according the amount of data we are going to
read/write. This workaround bugs in driver where the sizeimage provided
by TRY/S_FMT represent the buffer length (maximum size) rather then the expected
bytesused (buffer size).

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

v4l2: fix typo in _acquire_format() error messages

Fixes:

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

tests: matroskamux, qtmux: don't add codec_data buffers to template caps

streamheader and codec_data buffers fields are only meant to be
in the negotiated caps, not the template caps.

Fixes false-positive leaks of those buffers detected by the leaks
tracer, as template caps are static, and we decided to not include
code in gstreamer core to handle this unusual case of template caps
having buffers in them.

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

rtpvorbispay: Update and send out headers when new headers are received

The payloader needs to reset and update the vorbis config data which is
pushed on the network if it receives new headers, or at least, it may
have to do so.

Without this, the stream configuration could change without the
payloader sending the new configuration to the other side.

splitmuxsink: Change files on incompatible caps

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

splitmuxsink: Reset ready_for_output on state change

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

splitmuxsink: Remove unused next_max_out_running_time

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

splitmuxsink: Remove unused muxed_out_time

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