Tim-Philipp Müller [Mon, 3 Jul 2017 19:27:29 +0000 (20:27 +0100)]
imagefreeze: fix use-after-free on seek event
Get seqnum before unreffing the seek event.
https://bugzilla.gnome.org/show_bug.cgi?id=784486
Sebastian Dröge [Thu, 29 Jun 2017 15:59:58 +0000 (18:59 +0300)]
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
Sebastian Dröge [Thu, 22 Jun 2017 08:38:56 +0000 (11:38 +0300)]
rtspsrc: Actually use the receive lock when receiving, not the send lock
Sebastian Dröge [Tue, 20 Jun 2017 09:06:22 +0000 (12:06 +0300)]
Release 1.12.1
Sebastian Dröge [Tue, 20 Jun 2017 08:20:12 +0000 (11:20 +0300)]
Update .po files
Sebastian Dröge [Tue, 20 Jun 2017 08:08:32 +0000 (11:08 +0300)]
po: Update translations
Vivia Nikolaidou [Tue, 13 Jun 2017 14:40:19 +0000 (17:40 +0300)]
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
Sebastian Dröge [Thu, 15 Jun 2017 07:40:51 +0000 (10:40 +0300)]
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.
Sebastian Dröge [Thu, 15 Jun 2017 08:50:44 +0000 (11:50 +0300)]
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.
Sebastian Dröge [Tue, 13 Jun 2017 21:09:25 +0000 (00:09 +0300)]
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
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
Tim-Philipp Müller [Fri, 2 Jun 2017 10:30:15 +0000 (11:30 +0100)]
rtph265depay: fix caps leak
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
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 [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 [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
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
Sebastian Dröge [Fri, 7 Apr 2017 12:18:11 +0000 (15:18 +0300)]
Update .po files
Sebastian Dröge [Fri, 7 Apr 2017 12:06:30 +0000 (15:06 +0300)]
po: Update translations
Edward Hervey [Thu, 6 Apr 2017 10:01:00 +0000 (12:01 +0200)]
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
Nicolas Dufresne [Wed, 5 Apr 2017 13:46:31 +0000 (09:46 -0400)]
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.
Philipp Zabel [Wed, 16 Mar 2016 15:22:48 +0000 (16:22 +0100)]
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
Philipp Zabel [Wed, 16 Mar 2016 15:24:55 +0000 (16:24 +0100)]
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
Olivier Crête [Wed, 23 Nov 2016 17:17:55 +0000 (12:17 -0500)]
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
Jan Schmidt [Wed, 5 Apr 2017 05:55:20 +0000 (15:55 +1000)]
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.
Edward Hervey [Tue, 4 Apr 2017 15:55:13 +0000 (17:55 +0200)]
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.
Sebastian Dröge [Tue, 4 Apr 2017 12:22:25 +0000 (15:22 +0300)]
qtmux: Simplify buffer refcounting in add_buffer() and remove unneeded NULL checks
Sebastian Dröge [Tue, 4 Apr 2017 12:08:33 +0000 (15:08 +0300)]
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.
Sebastian Dröge [Tue, 4 Apr 2017 12:07:40 +0000 (15:07 +0300)]
qtmux: Error out immediately if a timecode is to be written but downstream return not-OK
Edward Hervey [Mon, 3 Apr 2017 09:34:49 +0000 (11:34 +0200)]
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.
Douglas Bagnall [Thu, 23 Mar 2017 11:11:13 +0000 (00:11 +1300)]
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
Douglas Bagnall [Thu, 23 Mar 2017 10:56:31 +0000 (23:56 +1300)]
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
Thibault Saunier [Tue, 28 Mar 2017 17:23:16 +0000 (14:23 -0300)]
meson: Use get_pkgconfig_variable instead of calling pkg-config ourself
It is avalaible in meson 0.36 which is now are requirement
Thibault Saunier [Tue, 28 Mar 2017 17:22:41 +0000 (14:22 -0300)]
pkgconfig: Do not ever build an installed .pc file
Thibault Saunier [Tue, 28 Mar 2017 14:15:53 +0000 (11:15 -0300)]
meson: test: Fix environment object usage
Thibault Saunier [Tue, 28 Mar 2017 14:14:47 +0000 (11:14 -0300)]
pkgconfig: Generate the pkg-config with meson too
Sebastian Dröge [Mon, 27 Mar 2017 18:52:00 +0000 (21:52 +0300)]
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.
Sebastian Dröge [Mon, 27 Mar 2017 08:43:31 +0000 (11:43 +0300)]
atomsrecovery: Error out when fseek() fails instead of silently ignoring
CID 1403262
Carlos Rafael Giani [Thu, 23 Mar 2017 21:13:05 +0000 (22:13 +0100)]
v4l2object: Also add videometa if there is padding to the right and bottom
https://bugzilla.gnome.org/show_bug.cgi?id=780478
George Kiagiadakis [Tue, 21 Mar 2017 10:54:27 +0000 (12:54 +0200)]
rtpmux: fix output segment and buffer DTS to correspond to the flattened PTS
https://bugzilla.gnome.org/show_bug.cgi?id=780347
Sebastian Dröge [Thu, 23 Mar 2017 15:53:19 +0000 (17:53 +0200)]
qtmux: Remove some unused variables
Sebastian Dröge [Thu, 23 Mar 2017 13:01:16 +0000 (15:01 +0200)]
qtmux: Remove a couple of unneeded levels of indentation
Enrique Ocaña González [Wed, 22 Mar 2017 18:18:40 +0000 (18:18 +0000)]
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
Sebastian Dröge [Wed, 22 Mar 2017 17:15:09 +0000 (19:15 +0200)]
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.
Sebastian Dröge [Wed, 22 Mar 2017 16:52:51 +0000 (18:52 +0200)]
qtmux: Do timecode handling per track, not per muxer instance
There could be multiple video tracks with timecodes.
Jan Schmidt [Tue, 21 Mar 2017 13:38:51 +0000 (00:38 +1100)]
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.
Sebastian Dröge [Tue, 21 Mar 2017 12:55:32 +0000 (14:55 +0200)]
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.
Tim-Philipp Müller [Mon, 20 Mar 2017 17:03:32 +0000 (17:03 +0000)]
docs: update two references to the removed 'mad' plugin
https://bugzilla.gnome.org/show_bug.cgi?id=776140
George Kiagiadakis [Mon, 20 Mar 2017 10:03:29 +0000 (12:03 +0200)]
rtprtxqueue: add basic documentation and example pipelines
Mostly explaining the difference between rtprtxqueue and rtprtxsend.
Nicolas Dufresne [Sat, 18 Mar 2017 00:58:28 +0000 (20:58 -0400)]
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).
George Kiagiadakis [Thu, 16 Mar 2017 16:20:54 +0000 (18:20 +0200)]
rtprtxreceive: fix example pipelines and improve the documentation
https://bugzilla.gnome.org/show_bug.cgi?id=771383
Vincent Penquerc'h [Fri, 17 Mar 2017 14:10:40 +0000 (14:10 +0000)]
flacparse: fix playback if sample number does not start at 0
This reverts commit
29b807685d3c962bbe8afe351c5dca97d59eb5e0, while
fixing the original breaking tests/check/pipelines/flacdec.
Vincent Penquerc'h [Fri, 17 Mar 2017 11:30:04 +0000 (11:30 +0000)]
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.
George Kiagiadakis [Thu, 16 Mar 2017 15:44:41 +0000 (17:44 +0200)]
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)
George Kiagiadakis [Thu, 16 Mar 2017 15:42:27 +0000 (17:42 +0200)]
rtprtxsend: convert LOG message to TRACE
This is printed too often (for every chained buffer!) and just clutters the logs.