2016-09-25 20:59 GMT+02:00 Michael Niedermayer <mich...@niedermayer.cc>: > On Sat, Sep 24, 2016 at 03:28:36PM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch hopefully fixes ticket #5136. >> >> Please review, Carl Eugen > >> utils.c | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> 519f209901429efaf9dffec381290a6a83573deb >> 0001-lavf-utils-Do-not-overflow-in-update_initial_timesta.patch >> From 09a31f8f6584f0c97443fd6edc7fcf7142b6a6ff Mon Sep 17 00:00:00 2001 >> From: Carl Eugen Hoyos <ceho...@ag.or.at> >> Date: Sat, 24 Sep 2016 15:26:21 +0200 >> Subject: [PATCH] lavf/utils: Do not overflow in >> update_initial_timestamps(). >> >> Fixes ticket #5136. >> --- >> libavformat/utils.c | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) > > probably ok > it might be ultimately easier to reject demuxer output that has > timestamps close to INT64 MAX/MIN instead of protecting all > computations one by one though
I don't disagree but I am probably unable to improve the patch. Do you want me to apply? Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
