platform/upstream/gstreamer.git
5 years agortpsession: test: Plug memory leak
Linus Svensson [Tue, 6 Nov 2018 14:05:54 +0000 (15:05 +0100)]
rtpsession: test: Plug memory leak

5 years agortpfunnel: Stop using G_DECLARE_FINAL_TYPE
Mathieu Duponchelle [Mon, 12 Nov 2018 23:37:11 +0000 (00:37 +0100)]
rtpfunnel: Stop using G_DECLARE_FINAL_TYPE

Fixes #516

5 years agoAdd Gitlab CI configuration
Jordan Petridis [Mon, 12 Nov 2018 11:42:29 +0000 (13:42 +0200)]
Add Gitlab CI configuration

This commit adds a .gitlab-ci.yml file, which uses a feature
to fetch the config from a centralized repository. The intent is
to have all the gstreamer modules use the same configuration.

The configuration is currently hosted at the gst-ci repository
under the gitlab/ci_template.yml path.

Part of https://gitlab.freedesktop.org/gstreamer/gstreamer-project/issues/29

6 years agov4l2object: Read driver selected interlace mode
Nicolas Dufresne [Fri, 19 Oct 2018 02:23:31 +0000 (22:23 -0400)]
v4l2object: Read driver selected interlace mode

If there was no interlace-mode field in the caps. Read back the value
selected by the driver. This way, if the driver does not support
progressive, then it will automatically negotiate the returned mode
unless this mode is not supported by GStreamer.

This method was already used for colorimetry. Just like colorimetry, the
interlace mode is not longer probed by v4l2src dues to performance
issues.

Fixes #511

6 years agomatroska: implement preliminary support for the bitrate query
Matthew Waters [Thu, 17 May 2018 11:58:25 +0000 (21:58 +1000)]
matroska: implement preliminary support for the bitrate query

Return the size / total duration as a ballpark estimate.

https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/issues/60

6 years agoisomp4: add preliminary support for the bitrate query
Matthew Waters [Thu, 17 May 2018 11:53:56 +0000 (21:53 +1000)]
isomp4: add preliminary support for the bitrate query

Return the upstream size over the duration as a first estimate.

https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/issues/60

6 years agortpbin: Sink jitterbuffer/storage before passing as parameters to signals
Sebastian Dröge [Tue, 6 Nov 2018 21:02:21 +0000 (23:02 +0200)]
rtpbin: Sink jitterbuffer/storage before passing as parameters to signals

Otherwise signal handlers from bindings will take ownership of them as
they are still floating, and we won't own a reference inside rtpbin
anymore.

Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/issues/515

6 years agoflvmux: Test that timestamps are always increasing
Havard Graff [Sat, 27 Oct 2018 17:00:52 +0000 (18:00 +0100)]
flvmux: Test that timestamps are always increasing

Decreasing timestamps break rtmpsink.

With contributions from Olivier Crête.

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

6 years agoflvmux: Force timestamps to always be increasing
Olivier Crête [Sat, 27 Oct 2018 18:27:12 +0000 (19:27 +0100)]
flvmux: Force timestamps to always be increasing

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

6 years agoUpdate common submodule location
Matthew Waters [Mon, 5 Nov 2018 05:36:26 +0000 (05:36 +0000)]
Update common submodule location

Remove the git directory

6 years agoClone the code from gitlab
Haihao Xiang [Mon, 5 Nov 2018 04:16:46 +0000 (12:16 +0800)]
Clone the code from gitlab

This fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/issues/513

6 years agoqtdemux: Ignore corrupted CTTS box
Seungha Yang [Thu, 1 Nov 2018 11:37:12 +0000 (20:37 +0900)]
qtdemux: Ignore corrupted CTTS box

If ctts (CompositionOffsetBox) has larger sample_offset
(offset between PTS and DTS) than (2 * duration) of the stream,
assume the ctts box to be corrupted and ignore the box.

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

6 years agoscaletempo: Implement SEGMENT query
Sebastian Dröge [Tue, 23 Oct 2018 08:45:36 +0000 (09:45 +0100)]
scaletempo: Implement SEGMENT query

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

6 years agowavparse: Implement SEGMENT query
Sebastian Dröge [Tue, 23 Oct 2018 08:42:21 +0000 (09:42 +0100)]
wavparse: Implement SEGMENT query

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

6 years agodtmfsrc: Declare output as interleaved
Olivier Crête [Sun, 28 Oct 2018 17:12:59 +0000 (17:12 +0000)]
dtmfsrc: Declare output as interleaved

This element doesn't support planar audio yet.

6 years agomeson: Add some missing test dependencies
Nirbheek Chauhan [Sun, 28 Oct 2018 14:09:21 +0000 (14:09 +0000)]
meson: Add some missing test dependencies

Without these dependencies, the enumtype may not be generated when the
test is built, which will cause a compile failure.

6 years agomeson: Cleanup old FIXMEs that relied on meson changes
Nirbheek Chauhan [Sun, 28 Oct 2018 14:07:54 +0000 (14:07 +0000)]
meson: Cleanup old FIXMEs that relied on meson changes

6 years agortpsession: Allow changing the SDES at runtime
Olivier Crête [Tue, 16 Oct 2018 21:28:00 +0000 (17:28 -0400)]
rtpsession: Allow changing the SDES at runtime

Make it possible to modify the SDES in a packet at runtime.

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

6 years agoqtmux: round to nearest when computing mehd and tkhd duration
Alicia Boya García [Thu, 1 Mar 2018 16:25:07 +0000 (17:25 +0100)]
qtmux: round to nearest when computing mehd and tkhd duration

This fixes a bug where in some files mehd.fragment_duration is one unit
less than the actual duration of the fragmented movie, as explained below:

mehd.fragment_duration is computed by scaling the end timestamp of
the last frame of the movie in (in nanoseconds) by the movie timescale.

In some situations, the end timestamp is innacurate due to lossy conversion to
fixed point required by GstBuffer upstream.

Take for instance a movie with 3 frames at exactly 3 fps.

$ gst-launch-1.0 -v videotestsrc num-buffers=3 \
  ! video/x-raw, framerate="(fraction)3/1" \
  ! x264enc \
  ! fakesink silent=false

dts: 999:59:59.333333334,  pts: 1000:00:00.000000000, duration: 0:00:00.333333333
dts: 999:59:59.666666667,  pts: 1000:00:00.666666666, duration: 0:00:00.333333334
dts: 1000:00:00.000000000, pts: 1000:00:00.333333333, duration: 0:00:00.333333333

The end timestamp is calculated by qtmux in this way:

end timestamp = last frame DTS + last frame DUR - first frame DTS =
  = 1000:00:00.000000000 + 0:00:00.333333333 - 999:59:59.333333334 =
  = 0:00:00.999999999

qtmux needs to round this timestamp to the declared movie timescale, which can
ameliorate this distortion, but it's important that round-neareast is used;
otherwise it would backfire badly.

Take for example a movie with a timescale of 30 units/s.

0.999999999 s * 30 units/s = 29.999999970 units

A round-floor (as it was done before this patch) would set fragment_duration to
29 units, amplifying the original distorsion from 1 nanosecond up to 33
milliseconds less than the correct value. The greatest distortion would occur
in the case where timescale = framerate, where an entire frame duration would
be subtracted.

Also, rounding is added to tkhd duration computation too, which
potentially has the same problem.

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

6 years agoudpsrc: print information about bind_error socket error
Marc Leeman [Wed, 16 May 2018 12:15:13 +0000 (14:15 +0200)]
udpsrc: print information about bind_error socket error

In some cases, a bind error occurs during operation. Printing
the information about the problem is critical for finding the
conflict

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

6 years agomatroska-demux: Fix caps memleak
Johan Bjäreholt [Wed, 17 Oct 2018 10:58:08 +0000 (12:58 +0200)]
matroska-demux: Fix caps memleak

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

6 years agov4l2bufferpool: fix typo resurect to resurrect
Wonchul Lee [Thu, 11 Oct 2018 00:24:53 +0000 (09:24 +0900)]
v4l2bufferpool: fix typo resurect to resurrect

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

6 years agov4l2videoenc: Add HEVC support
Amit Pandya [Thu, 18 Oct 2018 06:59:00 +0000 (12:29 +0530)]
v4l2videoenc: Add HEVC support

Add HEVC encoder support.

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

6 years agovl42allocator: Don't dup exported dmabufs
Nicolas Dufresne [Fri, 19 Oct 2018 21:37:28 +0000 (17:37 -0400)]
vl42allocator: Don't dup exported dmabufs

We can now use the new GstFAllocator to ask the allocator not to close
the wrapped FD. This way the dup is no longer needed.

6 years agov4l2allocator: Don't dup imported DMABuf FD
Nicolas Dufresne [Fri, 19 Oct 2018 21:14:15 +0000 (17:14 -0400)]
v4l2allocator: Don't dup imported DMABuf FD

There is no specific needs to duplicate the FD. Unlike the exportation,
we don't depend on code that will call close. This will make debugging
easyer since the traced FD will match the exporter.

6 years agomatroska-ids: Fix uninitialized memory in contexts
Johan Bjäreholt [Tue, 23 Oct 2018 11:04:34 +0000 (13:04 +0200)]
matroska-ids: Fix uninitialized memory in contexts

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

6 years agoqtmux: Add property for providing a threshold after which we create an edit list...
Sebastian Dröge [Fri, 19 Oct 2018 14:02:11 +0000 (17:02 +0300)]
qtmux: Add property for providing a threshold after which we create an edit list for gaps at the start

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

6 years agoqtmux: Correctly set tkhd width/height to the display size
Sebastian Dröge [Mon, 22 Oct 2018 11:21:54 +0000 (12:21 +0100)]
qtmux: Correctly set tkhd width/height to the display size

It was previously set to the display aspect ratio, e.g. 4x3, 16x9, etc.
but should be set to the display size.

This is a regression from e655d47dfce1652630fe8ff5fb6be56370087004
(1.5.1) and was correct before that.

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

6 years agotests: flvmux: Fix pushing invalid audio caps in tests
Yeongjin Jeong [Sun, 21 Oct 2018 02:15:15 +0000 (11:15 +0900)]
tests: flvmux: Fix pushing invalid audio caps in tests

Previous commit created caps with incorrect aac codec data
that did not match the audio channel.

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

6 years agoqtdemux: Fix build with GLib versions < 2.54
Seungha Yang [Fri, 19 Oct 2018 15:10:04 +0000 (00:10 +0900)]
qtdemux: Fix build with GLib versions < 2.54

g_ptr_array_find_with_equal_func was introduced in glib 2.54
which is a higher version than our minimum required one.

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

6 years agortpsession: fix up GHashTable-behavior dependent tests
Havard Graff [Wed, 17 Oct 2018 11:52:20 +0000 (13:52 +0200)]
rtpsession: fix up GHashTable-behavior dependent tests

GHashTable iteration order changed in recent GLib,
and tests were relying on that.

https://mail.gnome.org/archives/desktop-devel-list/2018-October/msg00016.html

6 years agoqtdemux: Don't switch active streams and old streams ...
Seungha Yang [Sun, 7 Oct 2018 11:07:39 +0000 (20:07 +0900)]
qtdemux: Don't switch active streams and old streams ...

... before the old streams is not exposed yet for MSS stream.

In case of DASH, newly configured streams will be exposed
whenever demux got moov without delay.
Meanwhile, since there is no moov box in MSS stream,
the caps will act like moov. Then, there is delay for exposing new pads
until demux got the first moof.

So, following scenario is possible only for MSS but not for DASH,
STREAM-START -> CAPS -> (configure stream but NOT EXPOSED YET)
-> STREAM-START-> CAPS (configure stream again).

In above scenario, we can reuse old stream without any stream reconfigure.

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

6 years agoqtdemux: Use GPtrArray to store QtDemuxStream structure
Seungha Yang [Sun, 7 Oct 2018 07:43:34 +0000 (16:43 +0900)]
qtdemux: Use GPtrArray to store QtDemuxStream structure

GPtrArray has less overhead than linked list and the length also
can be auto updated by using it.

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

6 years agoqtdemux: Make QtDemuxStream refcounted structure
Seungha Yang [Sun, 7 Oct 2018 07:50:45 +0000 (16:50 +0900)]
qtdemux: Make QtDemuxStream refcounted structure

This a prework for porting GPtrArray.
Refcounting will help the use of g_ptr_array_new_with_free_func()
with QtDemuxStream structure

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

6 years agoqtdemux: Make function foreach method friendly
Seungha Yang [Sat, 6 Oct 2018 11:19:40 +0000 (20:19 +0900)]
qtdemux: Make function foreach method friendly

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

6 years agoqtdemux: Only set width/height in caps if they're non-0
Olivier Crête [Thu, 26 Jul 2018 19:25:06 +0000 (15:25 -0400)]
qtdemux: Only set width/height in caps if they're non-0

If they are not valid, then let a downstream parser complete them.

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

6 years agoavidemux: fix misleading debug line
Wim Taymans [Thu, 16 Aug 2018 10:07:30 +0000 (12:07 +0200)]
avidemux: fix misleading debug line

6 years agoqtdemux: Avoid warning when reporting about decryptors
Philippe Normand [Fri, 22 Jun 2018 15:00:11 +0000 (16:00 +0100)]
qtdemux: Avoid warning when reporting about decryptors

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

6 years agomeson: Replace empty configuration_data() with copy keyword
Tim-Philipp Müller [Wed, 17 Oct 2018 13:15:33 +0000 (14:15 +0100)]
meson: Replace empty configuration_data() with copy keyword

Use 'copy' keyword to avoid meson warning message.
Note that 'copy' keyword in configure_file() is available
since meson 0.47.0

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

6 years agosplitmuxsink: Do not hardcode frames_of_daily_jam
Vivia Nikolaidou [Tue, 16 Oct 2018 12:42:12 +0000 (15:42 +0300)]
splitmuxsink: Do not hardcode frames_of_daily_jam

Apart from the obvious drawbacks of hardcoding, the drawback here was
that, if we subtracted 2 frames (instead of 2.6) from the target running
time, we'd request the next keyframe a bit too far into the future,
which would make our files split at the wrong position.

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

6 years agoqtmux: Allow up to 1% of frame rate for lateness
Vivia Nikolaidou [Tue, 2 Oct 2018 16:32:47 +0000 (19:32 +0300)]
qtmux: Allow up to 1% of frame rate for lateness

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

6 years agortpfunnel: fix shutdown
Mathieu Duponchelle [Tue, 18 Sep 2018 11:15:06 +0000 (13:15 +0200)]
rtpfunnel: fix shutdown

By disposing of the ssrc_to_pad map in finalize instead of
dispose.

6 years agoInitial commit of GstRtpFunnel
Havard Graff [Wed, 18 Oct 2017 09:14:36 +0000 (11:14 +0200)]
Initial commit of GstRtpFunnel

For funneling together rtp-streams into a single session.
Use-cases include multiplexing and bundle.

6 years agotests: flvdemux: Add new test for channel detect using aac codec-data
Yeongjin Jeong [Fri, 12 Oct 2018 13:33:15 +0000 (22:33 +0900)]
tests: flvdemux: Add new test for channel detect using aac codec-data

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

6 years agoflvdemux: Use aac codec-data to adjust channels if needed
Yeongjin Jeong [Thu, 11 Oct 2018 07:36:17 +0000 (16:36 +0900)]
flvdemux: Use aac codec-data to adjust channels if needed

Flv does not support various channels in AAC stream format, for example
flvdemux detect an audio channels of 2(stereo) when the AAC really is 1(mono).

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

6 years agotests: flvmux: Add new test for caps change after starting to write headers
Yeongjin Jeong [Thu, 11 Oct 2018 05:31:20 +0000 (14:31 +0900)]
tests: flvmux: Add new test for caps change after starting to write headers

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

6 years agoflvmux: Don't refuse caps changes after starting to write headers in streamable mode.
Yeongjin Jeong [Fri, 5 Oct 2018 08:16:26 +0000 (17:16 +0900)]
flvmux: Don't refuse caps changes after starting to write headers in streamable mode.

Flv does support changing the stream type and stream properties
after the headers were started to be written, and for example H264
codec_data changes can be supported.

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

6 years agosplitmuxsink: Fix if condition in drop-frame timecode wrap-around
Vivia Nikolaidou [Thu, 11 Oct 2018 10:55:01 +0000 (13:55 +0300)]
splitmuxsink: Fix if condition in drop-frame timecode wrap-around

Was previously: if ( x | y && a == b). Changed it into if ((x & y) && (a
== b)).

6 years agosplitmuxsink: Subtract daily jam offset when day wraps around
Vivia Nikolaidou [Tue, 9 Oct 2018 13:39:11 +0000 (16:39 +0300)]
splitmuxsink: Subtract daily jam offset when day wraps around

For drop-frame framerates, when the expected next max timecode wraps
around at the end of the day, we have to subtract the offset of the
daily jam, otherwise we end up with a duration that's a few frames too
long.

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

6 years agortpmux: respect downstream "timestamp-offset" in caps.
Havard Graff [Mon, 25 Sep 2017 12:30:13 +0000 (14:30 +0200)]
rtpmux: respect downstream "timestamp-offset" in caps.

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

6 years agortpmux: cleanup ssrc-handling code a bit
Havard Graff [Tue, 7 Jun 2016 12:38:19 +0000 (14:38 +0200)]
rtpmux: cleanup ssrc-handling code a bit

And add some better logging.

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

6 years agortpmux: protect against NULL caps
Havard Graff [Wed, 4 May 2016 09:48:04 +0000 (11:48 +0200)]
rtpmux: protect against NULL caps

Due to state-changes deactivating the pad from another thread,
this can happen.

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

6 years agortpmux: property should overrule both upstream and downstream
Havard Graff [Wed, 22 Jul 2015 07:47:22 +0000 (09:47 +0200)]
rtpmux: property should overrule both upstream and downstream

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

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

6 years agomeson: use new 'python' module instead of deprecated
Tim-Philipp Müller [Mon, 8 Oct 2018 19:45:08 +0000 (20:45 +0100)]
meson: use new 'python' module instead of deprecated

https://github.com/mesonbuild/meson/pull/4169

6 years agomeson: only build gtk gl examples if gst-gl was found
Tim-Philipp Müller [Mon, 8 Oct 2018 19:35:15 +0000 (20:35 +0100)]
meson: only build gtk gl examples if gst-gl was found

And fix typo in glliveshader example binary name.

6 years agosplitmuxsink: accept pads named 'sink' on the muxer, handle static pads as well
Peter Körner [Wed, 3 Oct 2018 14:17:22 +0000 (16:17 +0200)]
splitmuxsink: accept pads named 'sink' on the muxer, handle static pads as well

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

6 years agocairooverlay: Don't map input buffers if we just attach the overlay as meta
Sebastian Dröge [Tue, 25 Sep 2018 14:44:15 +0000 (17:44 +0300)]
cairooverlay: Don't map input buffers if we just attach the overlay as meta

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

6 years agocairooverlay: Add overlay as meta to the buffers if we can
Sebastian Dröge [Tue, 25 Sep 2018 14:02:26 +0000 (17:02 +0300)]
cairooverlay: Add overlay as meta to the buffers if we can

This requires that downstream supports it and
draw-on-transparent-surface is enabled.

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

6 years agocairooverlay: Pre-multiply and un-premultiply alpha in case of ARGB32
Sebastian Dröge [Tue, 25 Sep 2018 12:34:40 +0000 (15:34 +0300)]
cairooverlay: Pre-multiply and un-premultiply alpha in case of ARGB32

Cairo expects pre-multiplied alpha, we work on un-premultiplied alpha.

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

6 years agocairooverlay: Add property for drawing on a transparent surface and then blending
Sebastian Dröge [Tue, 25 Sep 2018 12:31:20 +0000 (15:31 +0300)]
cairooverlay: Add property for drawing on a transparent surface and then blending

This allows us to use the GstVideoOverlayComposition API and correctly
handle pre-multiplied alpha, while also only doing the alpha conversion
once instead of twice for the whole frame.

At a later point we can attach the meta to the buffer instead of
blending ourselves if downstream supports that.

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

6 years agomatroskdemux: do not use MapInfo.data after unmapping
Thibault Saunier [Wed, 3 Oct 2018 15:34:49 +0000 (17:34 +0200)]
matroskdemux: do not use MapInfo.data after unmapping

And minor gst-indenting

6 years agomatroska: Add the WebM encrypted content support in matroskademux
Yacine Bandou [Sun, 30 Sep 2018 17:28:07 +0000 (19:28 +0200)]
matroska: Add the WebM encrypted content support in matroskademux

This commit:

1. Reads the WebM and Matroska ContentEncryption subelements.

2. Creates a GST_PROTECTION event for each ContentEncryption, which
   will be sent before pushing the first source buffer.
   The DRM system id field in this event is set to GST_PROTECTION_UNSPECIFIED_SYSTEM_ID,
   because it isn't specified neither by Matroska nor by the WebM spec.

3. Reads the protection information of encrypted Block/SimpleBlock and
   extracts the IV and the partitioning format (subsamples).

4. Creates the metadata protection for each encrypted Block/SimpleBlock,
   with those informations: KeyID (extracted from ContentEncryption element),
   IV and partitioning format.

5. Adds a new caps for WebM encrypted content named "application/x-webm-enc",
   with the following new fields:

   "encryption-algorithm": The encryption algorithm used.
                           values: "None", "DES", "3DES", "Twofish", "Blowfish", "AES".

   "encoding-scope": The field that describes which Elements have been modified.
                     Values: "frame", "codec-data", "next-content".

   "cipher-mode": The cipher mode used in the encryption.
                  Values: "None", "CTR".

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

6 years agosplitmuxsink: Added a split-at-running-time action signal
John Nikolaides [Wed, 26 Sep 2018 14:43:05 +0000 (17:43 +0300)]
splitmuxsink: Added a split-at-running-time action signal

The video file can now be split at an arbitrary time, given by the user
as an argument to the action signal.

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

6 years agortpmp4gdepay: detect broken senders who send AAC with ADTS frames
Tim-Philipp Müller [Fri, 21 Sep 2018 18:47:44 +0000 (19:47 +0100)]
rtpmp4gdepay: detect broken senders who send AAC with ADTS frames

Strip ADTS headers if we detect any, apparently some Sony cameras
send AAC with ADTS headers. We could also change the stream-format
in the output caps, but that would be unexpected to pipeline builders
and would not exactly be backwards compatible.

6 years agortpmp4gdepay: factor out pushing of output buffer
Tim-Philipp Müller [Fri, 21 Sep 2018 17:17:25 +0000 (18:17 +0100)]
rtpmp4gdepay: factor out pushing of output buffer

6 years agoimagefreeze: Allow ANY capsfeatures
Sebastian Dröge [Wed, 26 Sep 2018 10:29:42 +0000 (13:29 +0300)]
imagefreeze: Allow ANY capsfeatures

6 years agodocs: update for git master
Tim-Philipp Müller [Tue, 25 Sep 2018 23:06:09 +0000 (00:06 +0100)]
docs: update for git master

6 years agoqtdemux: PIFF track encryption box support
Philippe Normand [Fri, 22 Jun 2018 11:05:17 +0000 (12:05 +0100)]
qtdemux: PIFF track encryption box support

The PIFF track encryption box is a UUID box containing the default encryption
values that should be used for PIFF sample encryption.

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

6 years agoosxaudio: add support for parsing more channel layouts ...
Nicola Murino [Mon, 24 Sep 2018 09:45:46 +0000 (11:45 +0200)]
osxaudio: add support for parsing more channel layouts ...

... and fallback to gst_audio_info_set_format for not yet supported layouts.

Fix audio playback on iOS 12.
Based on patch from Byron Schiel <byron@canary.is>

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

6 years agoqtdemux: turn impossible condition into an assert
Alicia Boya García [Sat, 22 Sep 2018 15:22:46 +0000 (17:22 +0200)]
qtdemux: turn impossible condition into an assert

qtdemux_update_streams() is only ever called after checking
`qtdemux->streams_aware` is TRUE. There is no need to check for that
condition again.

`qtdemux->streams_aware` is only modified when the demuxer is
hard-resetted, which is mutually exclusive with demuxing, so it cannot
be modified during the call.

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

6 years agomatroskademux: Emit no-more-pads after parsing Tracks
Alicia Boya García [Fri, 21 Sep 2018 20:24:02 +0000 (22:24 +0200)]
matroskademux: Emit no-more-pads after parsing Tracks

Currently matroskademux does not emit no-more-pads until the first
Cluster is parsed, even though the Tracks have already been parsed and
from that point on there can be no more tracks.

This is important in MSE because the browser needs to know when the MSE
initialization segment has been completely parsed so that it can expose
the tracks to the user. Some applications depend on this been done
before they feed frames to the demuxer.

As a consequence, historically WebKit has relied on hacks such as
listening to the `pad-added` event, which made impossible to support
multiple tracks in the same file. Let's fix that.

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

6 years agomatroskademux: Parse successive Tracks elements
Alicia Boya García [Fri, 21 Sep 2018 18:38:02 +0000 (20:38 +0200)]
matroskademux: Parse successive Tracks elements

This patch allows matroskademux to parse a second Tracks element,
erroring out if the tracks are not compatible (different number, type or
codec) and emitting new caps and tag events should they have changed.

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

6 years agomatroskademux: Refactor track parsing out from adding tracks
Alicia Boya García [Fri, 21 Sep 2018 14:23:57 +0000 (16:23 +0200)]
matroskademux: Refactor track parsing out from adding tracks

This splits gst_matroska_demux_add_stream() into:

* gst_matroska_demux_parse_stream(): will read the Matroska bytestream
  and fill a GstMatroskaTrackContext.

* gst_matroska_demux_parse_tracks(): will check there are no repeated
  tracks.

* gst_matroska_demux_add_stream(): creates and sets up the pad for the
  track.

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

6 years agomatroskademux: Allow Matroska headers to be read more than once
Alicia Boya García [Thu, 30 Nov 2017 19:44:23 +0000 (20:44 +0100)]
matroskademux: Allow Matroska headers to be read more than once

This is necessary for MSE, where a new MSE initialization segment may be
appended at any point. These MSE initialization segments consist of an
entire WebM file until the first Cluster element (not included). [1]

Note that track definitions are ignored on successive headers, they must
match, but this is not checked by matroskademux (look for
`(!demux->tracks_parsed)` in the code).

Source pads are not altered when the new headers are read.

This patch has been splitted from the original patch from eocanha in [2].

[1] https://www.w3.org/TR/mse-byte-stream-format-webm/
[2] https://bug334082.bugzilla-attachments.gnome.org/attachment.cgi?id=362212

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

6 years agosplitmuxsink: Implement split-after
Mathieu Duponchelle [Thu, 16 Aug 2018 19:42:37 +0000 (21:42 +0200)]
splitmuxsink: Implement split-after

The behaviour of split-now is to output the current GOP after
starting a new file.

The newly-added split-after signal will output the current GOP
to the old file if possible once a new GOP is opened.

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

6 years agoflvmux: Don't leak codec_data buffer
Seungha Yang [Thu, 20 Sep 2018 03:12:55 +0000 (12:12 +0900)]
flvmux: Don't leak codec_data buffer

Use gst_buffer_replace() to prevent buffer leak

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

6 years agoqtmux: Set Closed Caption track width/height to that of the first video track
Sebastian Dröge [Tue, 18 Sep 2018 15:13:52 +0000 (18:13 +0300)]
qtmux: Set Closed Caption track width/height to that of the first video track

Otherwise software like Premiere or Final Cut Pro won't like our files.

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

6 years agomeson: add glib-checks option to disable API guards and such
Tim-Philipp Müller [Wed, 19 Sep 2018 10:45:59 +0000 (11:45 +0100)]
meson: add glib-checks option to disable API guards and such

We want this enabled by default, also in releases, but people
may want to disable this for performance-critical workloads or
on embedded devices.

6 years agomeson: fix missing closing bracket in option descriptions
Tim-Philipp Müller [Wed, 19 Sep 2018 10:45:00 +0000 (11:45 +0100)]
meson: fix missing closing bracket in option descriptions

6 years agoqtmux: Initialize caption track language code to 0 instead of "und"
Sebastian Dröge [Thu, 6 Sep 2018 17:10:30 +0000 (20:10 +0300)]
qtmux: Initialize caption track language code to 0 instead of "und"

Without this, Final Cut considers it "non-standard" and 0 (english) is a
good default for closed captions.

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

6 years agov4l2videodec: Add HEVC decoder support
Song Bing [Thu, 13 Sep 2018 03:16:32 +0000 (03:16 +0000)]
v4l2videodec: Add HEVC decoder support

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

6 years agov4l2videodec: Move capture probe after input format is set
Nicolas Dufresne [Thu, 13 Sep 2018 02:35:39 +0000 (02:35 +0000)]
v4l2videodec: Move capture probe after input format is set

This is to support Amlogic CODEC driver which does not provide a full
list of formats when the driver is initially opened. GStreamer does
not strictly need this full list initially, but only later, in order
to negotiate with downstream if multiple format can be selected.

With this change, we will no longer probe twice the device, since the
probed list can be directly used for negotation.

6 years agov4l2: Add a debug message beforing waiting for codec stop
Ezequiel Garcia [Tue, 11 Sep 2018 19:46:34 +0000 (16:46 -0300)]
v4l2: Add a debug message beforing waiting for codec stop

Add a debug message right before waiting for the driver.
This is useful in order to debug drivers without a properly
implemented decoder or encoder stop command.

6 years agov4l2: Add a debug message indicating probe operation
Ezequiel Garcia [Mon, 10 Sep 2018 16:18:45 +0000 (13:18 -0300)]
v4l2: Add a debug message indicating probe operation

It's useful to see the v4l2 element running the probe
operation, to confirm it's turned on and working.

6 years agov4l2: Add JPEG encoding support
Ezequiel Garcia [Mon, 10 Sep 2018 16:18:30 +0000 (13:18 -0300)]
v4l2: Add JPEG encoding support

This commit adds the support for V4L JPEG stateful encoders.

6 years agov4l2: Add FWHT codec support
Ezequiel Garcia [Mon, 10 Sep 2018 19:20:52 +0000 (16:20 -0300)]
v4l2: Add FWHT codec support

The recently added vicodec (virtual codec) V4L driver
uses the Fast Walsh-Hadamard Transform for encoding
and decoding.

Add support for it.

6 years agov4l2: Sync kernel header with linuxtv tree
Nicolas Dufresne [Thu, 13 Sep 2018 01:28:24 +0000 (21:28 -0400)]
v4l2: Sync kernel header with linuxtv tree

This notably add HEVC and FWHT support, and VP8/9 profiles are now an
enumeration and their control exposed as a menu.

6 years agoqtmux: Allow up to 1 trak timescale unit of lateness in prefill mode
Vivia Nikolaidou [Wed, 12 Sep 2018 14:24:00 +0000 (17:24 +0300)]
qtmux: Allow up to 1 trak timescale unit of lateness in prefill mode

For 59.94 FPS, it's common to set 60000 as timescale. For that
timescale, if the audio is late by as little as 0:00:00.000016666
(definitely less than one audio sample), lateness gets rounded to 1.
Added a safeguard that allows lateness up to 1 sample with the specific
trak's timescale, to make sure that values less than e.g. one audio
sample won't break the prefill mode. What will happen in this case is
that the audio will get squeezed back to the video's timestamp, which in
practice means that the audio will be 0.000016666 seconds early (with
the patch).

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

6 years agov4l2object: Fix indentation
Nicolas Dufresne [Tue, 11 Sep 2018 00:20:39 +0000 (20:20 -0400)]
v4l2object: Fix indentation

6 years agov4l2object: Protect against zero PAR num/demu
Nicolas Dufresne [Tue, 11 Sep 2018 00:18:32 +0000 (00:18 +0000)]
v4l2object: Protect against zero PAR num/demu

This fixes an assertion when the driver implement CROPCAP but does
not set the PAR.

6 years agowavpackparse: fix handling of correction streams
Tim-Philipp Müller [Tue, 11 Sep 2018 23:52:19 +0000 (00:52 +0100)]
wavpackparse: fix handling of correction streams

Accept wavpack correction streams (.wvc) on sink pad, so
that wavpackparse can also be used to packetise correction
streams.

Fix parsing of subblock ID tags - the higher bits are
flags and are not part of the ID. This resulted in
correction blocks not being recognised properly and
the output not having the right (correction) caps.

6 years agomeson: Explicitly pass -DWIN32 while building speex
Nirbheek Chauhan [Fri, 7 Sep 2018 13:17:22 +0000 (18:47 +0530)]
meson: Explicitly pass -DWIN32 while building speex

The speex headers assume that WIN32 will always be defined when
building on Windows, but this is only true by default on MinGW.
Always set it explicitly.

6 years agoflvmux: Don't omit streamheader from caps on downstream reconfigure
Seungha Yang [Thu, 6 Sep 2018 04:13:19 +0000 (13:13 +0900)]
flvmux: Don't omit streamheader from caps on downstream reconfigure

The reconfigured downstream elements (e.g., dynamically added sink element)
most likely require the flv streamheader

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

6 years agomatroskamux: don't store used UIDs
Martin Kelly [Wed, 5 Sep 2018 23:11:00 +0000 (16:11 -0700)]
matroskamux: don't store used UIDs

Currently, whenever we generate a 128-bit UID, we store it in a list and
return 0 if we ever encounter a collision. This is so mathematically
improbable that it's not worth checking for, so we can save memory and
time by not tracking the UID. Even if a collision happened, a list of
only 10 UIDs would be unlikely to detect it.

This article has a good description of how improbable a collision is:
https://en.wikipedia.org/wiki/Universally_unique_identifier#Collisions

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

6 years agoqtmux: Use existing helper function to create "und" language code
Sebastian Dröge [Thu, 6 Sep 2018 17:06:10 +0000 (20:06 +0300)]
qtmux: Use existing helper function to create "und" language code

6 years agomeson: Don't skip plugins that don't build with MSVC
Nirbheek Chauhan [Wed, 5 Sep 2018 14:45:57 +0000 (20:15 +0530)]
meson: Don't skip plugins that don't build with MSVC

We now have options for all plugins, so we will just disable these in
the cerbero recipe instead. These require external deps, so they won't
affect gst-build either.

6 years agompg123: Remove ssize_t fallback, not needed anymore
Nirbheek Chauhan [Mon, 3 Sep 2018 10:34:33 +0000 (16:04 +0530)]
mpg123: Remove ssize_t fallback, not needed anymore

The mpg123 headers now contain a definition for ssize_t and building
with MSVC fails because of a redefinition for ssize_t

6 years agoqtdemux: Keep sample data from the current fragment only (push mode)
Alicia Boya García [Tue, 31 Jul 2018 10:52:36 +0000 (12:52 +0200)]
qtdemux: Keep sample data from the current fragment only (push mode)

This patch clears the sample table whenever the demuxing of a new
fragment begins. This avoids increasing memory usage for long videos.
This behavior was already present when upstream_format_is_time; this
patch extends it to all push mode operation (e.g. Media Source
Extensions).

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

6 years agomeson: Fix osxaudio build on iOS
Nirbheek Chauhan [Sat, 1 Sep 2018 04:00:23 +0000 (09:30 +0530)]
meson: Fix osxaudio build on iOS

Must define HAVE_IOS, and use appleframeworks dependency to ensure the
right frameworks are picked up.

6 years agov4l2object: Only offer MMAP/DMABUF pool
Nicolas Dufresne [Wed, 22 Aug 2018 19:23:53 +0000 (19:23 +0000)]
v4l2object: Only offer MMAP/DMABUF pool

The propose allocation was offering a pool even in DMABUF_IMPORT or
USERPTR mode. These pool are internal only.