Mathieu Duponchelle [Thu, 1 Jun 2017 00:03:27 +0000 (02:03 +0200)]
rtspsrc: uniquify stream ids
https://bugzilla.gnome.org/show_bug.cgi?id=783307
Thibault Saunier [Wed, 7 Jun 2017 16:47:59 +0000 (12:47 -0400)]
meson: Do not use path separator in test names
Avoiding warnings like:
WARNING: Target "elements/audioamplify" has a path separator in its name.
Nicolas Dufresne [Tue, 6 Jun 2017 15:29:29 +0000 (11:29 -0400)]
Fix v4l2 example
Jimmy Ohn [Mon, 5 Jun 2017 07:55:13 +0000 (16:55 +0900)]
qtdemux: remove not needed code
remove not needed code about res variable.
https://bugzilla.gnome.org/show_bug.cgi?id=783422
Nicolas Dufresne [Fri, 2 Jun 2017 18:01:17 +0000 (14:01 -0400)]
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
Tim-Philipp Müller [Fri, 2 Jun 2017 10:30:15 +0000 (11:30 +0100)]
rtph265depay: fix caps leak
Tim-Philipp Müller [Fri, 26 May 2017 15:30:06 +0000 (16:30 +0100)]
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.
Juan Navarro [Tue, 30 May 2017 20:23:10 +0000 (22:23 +0200)]
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
Edward Hervey [Fri, 26 May 2017 15:52:19 +0000 (17:52 +0200)]
v4l2videoenc: Don't leak VideoCodecState
CID #1409852
Edward Hervey [Fri, 26 May 2017 15:48:01 +0000 (17:48 +0200)]
dvdemux: Remove un-needed variable check
if pad wasn't present by now everything would have broken before
CID #1409854
Piotr Drąg [Thu, 25 May 2017 13:26:37 +0000 (15:26 +0200)]
po: update POTFILES
https://bugzilla.gnome.org/show_bug.cgi?id=783093
Nicolas Dufresne [Wed, 24 May 2017 18:19:27 +0000 (14:19 -0400)]
v4l2videoenc: Remove unused function
Nicolas Dufresne [Sun, 21 May 2017 13:29:11 +0000 (15:29 +0200)]
v4l2: Don't redefine __bitwise if already set
https://bugzilla.gnome.org/show_bug.cgi?id=728438
Ayaka [Tue, 23 May 2017 18:40:56 +0000 (14:40 -0400)]
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
Nicolas Dufresne [Tue, 23 May 2017 18:36:37 +0000 (14:36 -0400)]
v4l2videodec: Remove unused forward declaration
https://bugzilla.gnome.org/show_bug.cgi?id=728438
Ayaka [Mon, 5 Oct 2015 15:30:46 +0000 (16:30 +0100)]
v4l2pool: Fix wrong error message
https://bugzilla.gnome.org/show_bug.cgi?id=728438
Ayaka [Mon, 5 Oct 2015 15:20:19 +0000 (16:20 +0100)]
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
Tim-Philipp Müller [Wed, 24 May 2017 15:32:30 +0000 (16:32 +0100)]
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.
Tim-Philipp Müller [Wed, 24 May 2017 15:14:54 +0000 (16:14 +0100)]
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.
Tim-Philipp Müller [Wed, 24 May 2017 11:57:10 +0000 (12:57 +0100)]
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.
vijay [Wed, 24 May 2017 06:03:05 +0000 (11:33 +0530)]
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
Tim-Philipp Müller [Sun, 21 May 2017 16:45:34 +0000 (17:45 +0100)]
meson: don't need config.h.meson any longer
George Kiagiadakis [Mon, 24 Apr 2017 13:55:22 +0000 (16:55 +0300)]
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
Tim-Philipp Müller [Sun, 21 May 2017 09:37:19 +0000 (10:37 +0100)]
meson: make C++ compiler optional
It's only needed for the taglib plugin which is optional.
Tim-Philipp Müller [Sun, 21 May 2017 09:33:43 +0000 (10:33 +0100)]
multifile: remove some cruft
Josep Torra [Sat, 20 May 2017 15:09:52 +0000 (17:09 +0200)]
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.
Ravi Kiran K N [Sun, 6 Sep 2015 15:19:59 +0000 (20:49 +0530)]
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
Tim-Philipp Müller [Sat, 20 May 2017 13:53:42 +0000 (14:53 +0100)]
meson: add options to set package name and origin
https://bugzilla.gnome.org/show_bug.cgi?id=782172
Luis de Bethencourt [Sat, 20 May 2017 10:40:33 +0000 (11:40 +0100)]
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.
Josep Torra [Sat, 20 May 2017 09:13:40 +0000 (11:13 +0200)]
osxvideo: fix macOS 10.12 deprecation warnings
Add #defines to allow older versions of macOS to use the new constant names.
Edward Hervey [Sat, 13 May 2017 07:05:57 +0000 (09:05 +0200)]
isomp4: Safely ignore [skip] atoms
Instead of warning about them
Sebastian Dröge [Thu, 18 May 2017 12:10:30 +0000 (15:10 +0300)]
souphttpsrc: Make session sharing thread-safe on our side
https://bugzilla.gnome.org/show_bug.cgi?id=780140
Tim-Philipp Müller [Thu, 18 May 2017 09:43:19 +0000 (10:43 +0100)]
Fix up package name and origin in some plugins
Sebastian Dröge [Mon, 15 May 2017 16:51:47 +0000 (19:51 +0300)]
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
Sebastian Dröge [Mon, 15 May 2017 11:22:34 +0000 (14:22 +0300)]
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
Nicolas Dufresne [Sat, 29 Apr 2017 03:05:35 +0000 (23:05 -0400)]
pulse: Accept MPEG 1 layer 3 version 2.5
https://bugzilla.gnome.org/show_bug.cgi?id=781929
Nicolas Dufresne [Tue, 16 May 2017 17:50:16 +0000 (13:50 -0400)]
Remove plugin specific static build option
Static and dynamic plugins now have the same interface. The standard
--enable-static/--enable-shared toggle are sufficient.
Vivia Nikolaidou [Fri, 12 May 2017 14:53:57 +0000 (17:53 +0300)]
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
Vivia Nikolaidou [Tue, 16 May 2017 09:56:15 +0000 (12:56 +0300)]
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
Sebastian Dröge [Thu, 11 May 2017 18:01:15 +0000 (20:01 +0200)]
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.
Sebastian Dröge [Thu, 11 May 2017 17:21:22 +0000 (19:21 +0200)]
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.
Sebastian Dröge [Wed, 10 May 2017 13:58:41 +0000 (15:58 +0200)]
qtmux: Lateness is in QT timescale, diff in GstClockTime
Print the right one in debug output to get meaningful numbers.
Sebastian Dröge [Wed, 10 May 2017 12:31:40 +0000 (14:31 +0200)]
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
Sebastian Dröge [Wed, 10 May 2017 09:42:09 +0000 (11:42 +0200)]
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.
Sebastian Dröge [Wed, 10 May 2017 09:12:23 +0000 (11:12 +0200)]
qtmux: Error out directly if sending filler data results in a flow error
CID 1405994
Sebastian Dröge [Tue, 9 May 2017 14:02:43 +0000 (16:02 +0200)]
qtmux: In prefill mode, handle the case when only the first chunk was ever used
Sebastian Dröge [Thu, 16 Mar 2017 13:12:07 +0000 (15:12 +0200)]
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
Sebastian Dröge [Thu, 16 Mar 2017 11:58:41 +0000 (13:58 +0200)]
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
Sebastian Dröge [Thu, 9 Mar 2017 08:15:34 +0000 (10:15 +0200)]
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
Sebastian Dröge [Wed, 29 Mar 2017 11:01:25 +0000 (14:01 +0300)]
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
Sebastian Dröge [Tue, 9 May 2017 09:41:25 +0000 (11:41 +0200)]
vpxdec: Set fb->priv to NULL after freeing just in case
https://bugzilla.gnome.org/show_bug.cgi?id=782359
Dustin Spicuzza [Mon, 8 May 2017 15:22:00 +0000 (15:22 +0000)]
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
Tim-Philipp Müller [Mon, 8 May 2017 15:05:45 +0000 (15:05 +0000)]
vpx: fix build against older libvpx versions
Such as 1.3.0 as on raspbian.
Nirbheek Chauhan [Wed, 3 May 2017 17:53:10 +0000 (23:23 +0530)]
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
Nirbheek Chauhan [Wed, 3 May 2017 18:01:00 +0000 (23:31 +0530)]
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
Tim-Philipp Müller [Sat, 6 May 2017 21:30:20 +0000 (22:30 +0100)]
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.
Sebastian Dröge [Thu, 4 May 2017 15:59:14 +0000 (18:59 +0300)]
Back to development
Sebastian Dröge [Thu, 4 May 2017 12:38:34 +0000 (15:38 +0300)]
Release 1.12.0
Sebastian Dröge [Thu, 4 May 2017 12:07:27 +0000 (15:07 +0300)]
Update .po files
Sebastian Dröge [Thu, 4 May 2017 10:47:20 +0000 (13:47 +0300)]
po: Update translations
Seungha Yang [Tue, 2 May 2017 01:32:30 +0000 (10:32 +0900)]
qtdemux: Fix crash on mss stream caused by invalid stsd entry access
Since mss has no moov, default stsd entry should be created with media-caps.
https://bugzilla.gnome.org/show_bug.cgi?id=782042
Sebastian Dröge [Thu, 27 Apr 2017 14:29:58 +0000 (17:29 +0300)]
Release 1.11.91
Sebastian Dröge [Thu, 27 Apr 2017 12:58:47 +0000 (15:58 +0300)]
Update .po files
Sebastian Dröge [Thu, 27 Apr 2017 12:28:02 +0000 (15:28 +0300)]
po: Update translations
Sebastian Dröge [Thu, 27 Apr 2017 09:56:27 +0000 (12:56 +0300)]
qtdemux: Don't crash in debug output if stream==NULL
That case is correctly handled below but not in the debug output.
https://bugzilla.gnome.org/show_bug.cgi?id=781270
Sebastian Dröge [Tue, 25 Apr 2017 14:11:27 +0000 (17:11 +0300)]
qtdemux: Don't perform seeks with inconsistent seek values
If gst_segment_do_seek() fails, we shouldn't try seeking on that
resulting segment but just error out. Crashes further down the line
otherwise.
Tim-Philipp Müller [Mon, 24 Apr 2017 19:27:49 +0000 (20:27 +0100)]
Automatic update of common submodule
From 60aeef6 to 48a5d85
Tim-Philipp Müller [Mon, 24 Apr 2017 16:31:04 +0000 (17:31 +0100)]
tests: rtp-payloading: add test for rtph264depay avc/byte-stream output
Make sure avc output doesn't contain SPS/PPS inline, but
byte-stream output does.
Tim-Philipp Müller [Mon, 24 Apr 2017 16:29:37 +0000 (17:29 +0100)]
rtph264depay: don't insert SPS/PPS inline for AVC output
SPS/PPS are in the caps in this case and shouldn't be in
the stream data.
Sebastian Dröge [Fri, 21 Apr 2017 18:09:14 +0000 (19:09 +0100)]
rtspsrc: Chain up to the parent class' provide_clock() implementation
If no clock was provided directly by rtspsrc. This behaviour was removed
by
f8013487c91a6ffc552a4b25aa1a70f0bd5377f8 and results in rtspsrc not
providing the system clock via the rtpjitterbuffer.
As a result, if another element like an audio sink, provides a clock,
the pipeline would select that (when going to PAUSED/PLAYING again later).
Audio clocks usually don't progress in PAUSED, and thus our live source
won't be able to use the clock to produce data, making the sink never
preroll and everything is stuck.
Jürgen Sachs [Thu, 20 Apr 2017 09:22:15 +0000 (11:22 +0200)]
qtdemux: reset sample_description_id to default
Fixes stream where sample_description_id is specified in the tfhd
https://bugzilla.gnome.org/show_bug.cgi?id=778337
Sebastian Dröge [Thu, 20 Apr 2017 12:16:24 +0000 (13:16 +0100)]
splitmuxsink: Don't use an explicit name for requesting audio pads
... unless the muxer uses the same audio pad template name as
splitmuxsink. We can't request a pad called "audio_0" on a muxer that
wants pads to be "sink_%d".
ChangBok Chae [Thu, 23 Feb 2017 00:31:36 +0000 (09:31 +0900)]
flvdemux: remove duplicated segment initialization
It's also done in gst_flv_demux_cleanup().
https://bugzilla.gnome.org/show_bug.cgi?id=779106
Xavier Claessens [Thu, 20 Apr 2017 10:17:35 +0000 (20:17 +1000)]
splitmuxsink: Correctly catch FLUSH events in probes
https://bugzilla.gnome.org/show_bug.cgi?id=767498
Tim-Philipp Müller [Wed, 19 Apr 2017 11:28:12 +0000 (12:28 +0100)]
Revert "rtpbin: pipeline gets an EOS when any rtpsources byes"
This reverts commit
eeea2a7fe88a17b15318d5b6ae6e190b2f777030.
It breaks EOS in some sender pipelines, see
https://bugzilla.gnome.org/show_bug.cgi?id=773218#c20
Edward Hervey [Fri, 14 Apr 2017 15:01:49 +0000 (17:01 +0200)]
qtdemux: Reset adapter in more discontinuity cases
In push mode we process as much as possible in the adapter. When we receive
a DISCONT buffer which we can't match to an actual sample (based on the existing
sample table) and there is still data remaining in the incoming adapter,there is
one of two cases happening:
1) We are doing reverse playback, in which case we should flush out all pending
data
2) We have leftover data from the previous incoming buffer... which we can't do
anything about.
For the second case, make sure we flush out the remaining data so that we can start
parsing again from scratch.
https://bugzilla.gnome.org/show_bug.cgi?id=781319
Edward Hervey [Fri, 14 Apr 2017 08:56:41 +0000 (10:56 +0200)]
rtspsrc: Use GST_ELEMENT_ERROR_WITH_DETAILS
Allows the application to know the exact status code that was returned
by the server in a programmatic fashion.
https://bugzilla.gnome.org/show_bug.cgi?id=781304
Seungha Yang [Sun, 16 Apr 2017 09:47:56 +0000 (18:47 +0900)]
qtdemux: Fix leak on QtDemuxStreamStsdEntry
Fix unit test failure
https://bugzilla.gnome.org/show_bug.cgi?id=781362
Sebastian Dröge [Fri, 14 Apr 2017 10:38:53 +0000 (13:38 +0300)]
qtmux: Fix timescale of timecode tracks
They should have ideally the same timescale of the video track, which we
can't guarantee here as in theory timecode configuration and video
framerate could be different. However we should set a correct timescale
based on the framerate given in the timecode configuration, and not just
use the framerate numerator.
Edward Hervey [Thu, 13 Apr 2017 11:25:06 +0000 (13:25 +0200)]
qtdemux: Properly reset demuxer when all streams are EOS
Make sure offset and neededbytes are properly resetted when all
streams are EOS in push-mode.
Avoids cases when some data might still be pushed by upstream (because
it didn't yet see the resulting GST_FLOW_EOS yet) and qtdemux gets
completely lost.
https://bugzilla.gnome.org/show_bug.cgi?id=781266
Edward Hervey [Thu, 13 Apr 2017 06:00:30 +0000 (08:00 +0200)]
souphttpsrc: Make more usage of error macro
And make sure we actually use the provided soup_msg argument in the macro
Nirbheek Chauhan [Wed, 12 Apr 2017 13:16:53 +0000 (18:46 +0530)]
meson: Print message when disabling taglib on MSVC
Edward Hervey [Wed, 12 Apr 2017 11:26:59 +0000 (13:26 +0200)]
qtmux: Don't forget to update pad->last_buf
buf is the current pad->last_buf value. If ever it gets copied/unreffed,
we need to make sure to write back the new pointer to the last_buf
variable.
Fixes using wrong pointer values in the case of decrasing DTS value
Edward Hervey [Wed, 12 Apr 2017 09:33:05 +0000 (11:33 +0200)]
tests: Add vp9enc to gitignore
Jürgen Sachs [Tue, 11 Apr 2017 11:41:48 +0000 (13:41 +0200)]
qtdemux: fix: sample description index override in tfhd not evaluated
https://bugzilla.gnome.org/show_bug.cgi?id=778337
Edward Hervey [Wed, 12 Apr 2017 09:03:24 +0000 (11:03 +0200)]
qtdemux: Add out-of-bound check
Make sure we don't read invalid memory
Thiago Santos [Wed, 27 Apr 2016 15:17:37 +0000 (12:17 -0300)]
qtdemux: move parsing of tkhd out of stsd entry loop
It needs only to be read once.
Thiago Santos [Thu, 7 Apr 2016 15:23:35 +0000 (12:23 -0300)]
qtdemux: check for a different stsd entry before pushing a sample
Before pushing a sample, check if there was a change in the current
stsd entry. This patch also assumes that the first stsd entry is
used as default for the first sample. It might cause an uneeded
caps renegotiation when this isn't the case.
Thiago Santos [Wed, 6 Apr 2016 15:55:18 +0000 (12:55 -0300)]
qtdemux: parse all stsd entries
stsd can have multiple format entries, parse them all.
This is required to play DVB DASH profile that uses multiple entries
to identify the different available bitrates/options on dash streams
The stream format-specific data is not stored into QtDemuxStreamStsdEntry
Thiago Santos [Tue, 5 Apr 2016 17:34:00 +0000 (14:34 -0300)]
qtdemux: rework stsd sample entries access
Instead of using the stsd as a base pointer, use the actual stsd
entry as the stsd can have multiple entries. This is rarely used
for file playback but is a possible profile with in DVB DASH specs.
This still doesn't support stsd with multiple entries but makes it
easier to do so.
Thiago Santos [Tue, 5 Apr 2016 21:00:10 +0000 (18:00 -0300)]
qtdemux: get stsd child by index instead of type
There might be multiple children with the same type
George Kiagiadakis [Fri, 7 Apr 2017 13:33:18 +0000 (16:33 +0300)]
tests/check/rtprtx: add checks for rtprtxqueue's max-size-{time,packets} properties
https://bugzilla.gnome.org/show_bug.cgi?id=780867
George Kiagiadakis [Tue, 4 Apr 2017 14:33:31 +0000 (17:33 +0300)]
rtprtxqueue: implement handling of the max-size-time property
https://bugzilla.gnome.org/show_bug.cgi?id=780867
Tim-Philipp Müller [Mon, 10 Apr 2017 22:49:06 +0000 (23:49 +0100)]
Automatic update of common submodule
From 39ac2f5 to 60aeef6
Todor Tomov [Mon, 10 Apr 2017 08:56:00 +0000 (08:56 +0000)]
v4l2object: Copy timestamp when importing buffers
This is needed for V4L2_OUTPUT interface, and is harmless of
V4L2_CAPTURE interfaces. This will fix timestamp in cases like:
v4l2src io-mode=dmabuf ! v4l2videoNenc output-io-mode=dmabuf-import ! ...
Same apply for userptr.
https://bugzilla.gnome.org/show_bug.cgi?id=781119
Sebastian Dröge [Mon, 10 Apr 2017 12:55:30 +0000 (15:55 +0300)]
qtmux: Fix last_dts tracking for raw audio and similar formats
Accumulate the durations directly and don't scale yet another time by
the number of samples.
Vincent Penquerc'h [Fri, 7 Apr 2017 09:48:50 +0000 (10:48 +0100)]
tests: fix leak in splitmux test
https://bugzilla.gnome.org/show_bug.cgi?id=781025
Lyon Wang [Fri, 7 Apr 2017 07:29:43 +0000 (15:29 +0800)]
scaletempo: Scale GAP event timestamp and duration like for buffers
https://bugzilla.gnome.org/show_bug.cgi?id=781008
Thibault Saunier [Fri, 17 Feb 2017 13:01:08 +0000 (10:01 -0300)]
v4l2dec: Fix race when going from PAUSED to READY
Running `gst-validate-launcher -t validate.file.playback.change_state_intensive.vorbis_vp8_1_webm`
on odroid XU4 (s5p-mfc v4l2 driver) often leads to:
ERROR:../subprojects/gst-plugins-good/sys/v4l2/gstv4l2videodec.c:215:gst_v4l2_video_dec_stop: assertion failed: (g_atomic_int_get (&self->processing) == FALSE)
This happens when the following race happens:
- T0: Main thread
- T1: Upstream streaming thread
- T2. v4l2dec processing thread)
[The decoder is in PAUSED state]
T0. The validate scenario runs `Executing (36/40) set-state: state=null repeat=40`
T1- The decoder handles a frame
T2- A decoded frame is push downstream
T2- Downstream returns FLUSHING as it is already flushing changing state
T2- The decoder stops its processing thread and sets `->processing = FALSE`
T1- The decoder handles another frame
T1- `->process` is FALSE so the decoder restarts its streaming thread
T0- In v4l2dec-> stop the processing thread is stopped
NOTE: At this point the processing thread loop never started.
T0- assertion failed: (g_atomic_int_get (&self->processing) == FALSE)
Here I am removing the whole ->processing logic to base it all on the
GstTask state to avoid duplicating the knowledge.
https://bugzilla.gnome.org/show_bug.cgi?id=778830
Sebastian Dröge [Fri, 7 Apr 2017 13:31:56 +0000 (16:31 +0300)]
Release 1.11.90