diff options
author | Guillaume Desmottes <guillaume.desmottes@collabora.com> | 2019-02-21 08:48:31 +0100 |
---|---|---|
committer | Guillaume Desmottes <guillaume.desmottes@collabora.com> | 2019-02-21 08:56:34 +0100 |
commit | 1d293764e55fb9c3901c9dfa903fa5ae0021ab10 (patch) | |
tree | 3662149be0532119ea87ccb69887f10a09d5e15c | |
parent | 1e01f2764bb99a58471dfacce54f8abaf76553f9 (diff) | |
download | gst-libav-1d293764e55fb9c3901c9dfa903fa5ae0021ab10.tar.gz |
avdemux: fix negative pts if start_time is bigger than the ts
The start time is supposed to be the ts of the first frame.
FFmpeg uses fractions to represent timestamps and the start time may use a
different base than the frame pts. So we may end up having the start
time bigger than the pts because of rounding when converting to gst ts.
See https://gitlab.freedesktop.org/gstreamer/gst-libav/issues/51
for details.
-rw-r--r-- | ext/libav/gstavdemux.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/ext/libav/gstavdemux.c b/ext/libav/gstavdemux.c index 30e0360..3b74f92 100644 --- a/ext/libav/gstavdemux.c +++ b/ext/libav/gstavdemux.c @@ -1473,8 +1473,14 @@ gst_ffmpegdemux_loop (GstFFMpegDemux * demux) goto drop; #endif - if (GST_CLOCK_TIME_IS_VALID (timestamp)) - timestamp -= demux->start_time; + if (GST_CLOCK_TIME_IS_VALID (timestamp)) { + /* start_time should be the ts of the first frame but it may actually be + * higher because of rounding when converting to gst ts. */ + if (demux->start_time >= timestamp) + timestamp = 0; + else + timestamp -= demux->start_time; + } /* check if we ran outside of the segment */ if (demux->segment.stop != -1 && timestamp > demux->segment.stop) |