[FFmpeg-cvslog] Revert "avformat/utils: Discard huge timestamps which would cause overflows if used in basic computations "

Michael Niedermayer git at videolan.org
Wed Oct 26 21:35:58 EEST 2016


ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Wed Oct 26 20:31:04 2016 +0200| [38e5a4f3bbe9c79928e70f2834ab0c7b27349d41] | committer: Michael Niedermayer

Revert "avformat/utils: Discard huge timestamps which would cause overflows if used in basic computations"

Some people seem to oppose this the patch seems to have been missed for a month on the ML
lets restart the discussion and solve this after the release

This reverts commit e936c8d176efd1a0a41e22df24564b1178c79ea9.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=38e5a4f3bbe9c79928e70f2834ab0c7b27349d41
---

 libavformat/utils.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/libavformat/utils.c b/libavformat/utils.c
index 74eed47..70dbfa1 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -833,12 +833,6 @@ int ff_read_packet(AVFormatContext *s, AVPacket *pkt)
             continue;
         }
 
-        if (   (pkt->dts != AV_NOPTS_VALUE && (pkt->dts <= INT64_MIN/2 || pkt->dts >= INT64_MAX/2))
-            || (pkt->pts != AV_NOPTS_VALUE && (pkt->pts <= INT64_MIN/2 || pkt->pts >= INT64_MAX/2))) {
-            av_log(s, AV_LOG_WARNING, "Ignoring huge timestamps %"PRId64" %"PRId64"\n", pkt->dts, pkt->pts);
-            pkt->dts = pkt->pts = AV_NOPTS_VALUE;
-        }
-
         st = s->streams[pkt->stream_index];
 
         if (update_wrap_reference(s, st, pkt->stream_index, pkt) && st->pts_wrap_behavior == AV_PTS_WRAP_SUB_OFFSET) {



More information about the ffmpeg-cvslog mailing list